[FFmpeg-devel] [PATCH] Document ff_put_string()

Stefano Sabatini stefano.sabatini-lala
Sat Apr 11 15:37:57 CEST 2009


On date Saturday 2009-04-11 15:15:41 +0200, Michael Niedermayer encoded:
> On Sat, Apr 11, 2009 at 03:10:28PM +0200, Stefano Sabatini wrote:
[...]
> > So I suppose the problem is "non-zero". Check again (also please be
> > more explicit the next time).
> 
> the problem is that you repeat the word zero while one refers to the value
> of a parameter and one to the name of the parameter, you still do that

Well I don't think it is such a big problem, since it should be clear
that put_zero is the name of the param, also I can't find a better way
to express it, maybe changing the name of the param would be an option
but definitively overkill.

So I'm adding the words "if it is different from zero" to make the
phrase more explicit.

Regards.
-- 
FFmpeg = Foolish & Frightening MultiPurpose Extended Gem
-------------- next part --------------
A non-text attachment was scrubbed...
Name: bs-document-ff-put-string.patch
Type: text/x-diff
Size: 703 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090411/689f6005/attachment.patch>



More information about the ffmpeg-devel mailing list