[FFmpeg-devel] [PATCH]Add one CRLF to http headers if necessary

[Nicolas George]

Le primidi 21 ventôse, an CCXXIII, Michael Niedermayer a écrit :
> what is the problem ?
> can you elaborate?

The problem raised in this trac ticket is that windows users seem unable to
put control characters in their command lines. I thought it was a
shortcoming of the windows command line interpreter, but I have come to
suspect it may be a deeper flaw in the way command-line arguments are passed
to programs. I have submitted the question to a friend who knows windows
well.

> theres a bug in OUR code it should be fixed in our code.

No, there is no bug in our code.

The issue is definitely not a bug in lavf: lavf takes a C string, and except
for \0, a C string can contain any character without any problem. And this
API is not even as bad as some people claim: passing a list of values is
always clumsy.

It may be a shortcoming in our command-line interface IF AND ONLY IF it is
established that some OSes can not pass arbitrary C string as arguments to
main(). A shortcoming, not a bug.

> The line ending characters differ between platforms and users do in
> general not know about these differences nor should they need to.

You seem to be under the misapprehension the issue is about different line
ending encodings. It is not, it never was.

> IMO
> User interfaces should be designed so they do what the user wants with
> minimun effort on the users part.

True, but this "minimum effort" must be measured globally, not locally. It
may seem convenient to make a special case for this or that option. If you
only look at the effort required to invoke that particular option, that is
true. But you also have to count the effort to remember which option has a
special case.

When you have hundreds of options like FFmpeg, you want, as much as
possible, that ALL OPTIONS BEHAVE EXACTLY THE SAME. Any other consideration
is secondary.

Regards,

-- 
  Nicolas George
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20150311/4daebb24/attachment.asc>