[FFmpeg-devel] [PATCH] sbgdec: dont set slide to a uninitialized value

Nicolas George nicolas.george at normalesup.org
Thu Nov 8 00:01:26 CET 2012


Le sextidi 16 brumaire, an CCXXI, Reimar Döffinger a écrit :
> Not that it matters, but I had thought of
> f = *fr;
> in order to better match the name and only make it update those two
> values. Of course that's only based on a completely unfounded
> guess on what the author thought the semantics should be.

It does not matter much: this is quite a simple function, private and used
only once in the parser. If someday someone want to extend the syntax (very
unlikely), they can and will change the function to suit their needs, and
the result may be one of the versions suggested here or something else
entirely. I do not think it is any use speculating about it.

Regards,

-- 
  Nicolas George
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20121108/a041215a/attachment.asc>


More information about the ffmpeg-devel mailing list