[MPlayer-dev-eng] Re: [NEW PATCH] ftp support

[Bernd Ernesti]

On Wed, Aug 13, 2003 at 08:18:47PM +0200, Alban Bedel wrote:
> Hi Bernd Ernesti,
> 
> on Wed, 13 Aug 2003 19:04:00 +0200 you wrote:
> 
> > Hi,
> > 
> > first of all, I don't understand why mplayer needs such support at all.
> The only alternative is wget ... | mplayer. But you can't seek (and need
> wget ;)
> 
> > And then, why is it enabled by default?
> 1. it doesn't depend on something special
> 2. it should work for everybody.
> 3. It must be tested

4. It simply increases the binary size for all people
5. people who need it, can always enable it
6. did you check it if there is no buffer overflow in it?
7. why do you we need functions like: FtpRmdir, FtpMkdir, FtpModDate, FtpPut, FtpRename, FtpDelete
8. 'must' is a strong word

> > Oh, and using 'nospam at nowhere.com' is not a good idea. Thats a valid
> > domain name.
> I think whatever you put it will become "a valid domain name" sooner
> or later. Anyway that's just unimportant.

Thanks for this attitude, please make it then not the default if you don't care
about faking email addresses.

Bernd