On date Monday 2015-03-16 11:00:12 +0100, Stefano Sabatini encoded: > On date Sunday 2015-03-15 15:15:34 +0100, Clément Bœsch encoded: [...] > > To elaborate on this, the bug here is referring to an allocation check not > > done in the caller (there are many currently since I'm just introducing > > the error handling). > > I won't block this patch, but getting a crash in a specified point of > the program is more useful than failing with this: >
> A bug occurred somewhere Now this is not correct, it should be: there is a bug somewhere (since bug don't "occurr" but stay/are). Now that (the presence of a "bug") is not even a good reason to fail a program, it should had been named something more meaningful like "unexpected condition" (but again - for that a crash or an assert is just more useful to the programmer). -- FFmpeg = Foolish and Foolish Martial Portentous Exuberant Gorilla _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel