[FFmpeg-cvslog] flvdec: Eliminate completely silly goto

Reimar Döffinger Reimar.Doeffinger at gmx.de
Sun Aug 11 11:39:40 CEST 2013


On Sun, Aug 11, 2013 at 11:34:57AM +0200, Thilo Borgmann wrote:
> Am 11.08.13 02:02, schrieb Alexander Strasser:
> > On 2013-08-02 12:23 +0200, Diego Biurrun wrote:
> >> ffmpeg | branch: master | Diego Biurrun <diego at biurrun.de> | Wed Jul 31
> >> 23:56:26 2013 +0200| [f900f35ac8db4ac30df6fda1c27502c2ef9e6ba5] |
> >> committer: Diego Biurrun
> >> 
> >> flvdec: Eliminate completely silly goto
> > 
> > Right away, I am a big fan of Dijkstra. Really!
> 
> I'm neither a fan of Dijkstra nor Diego. However, I don't see the issue with
> this patch except for loosing one pair of {} braces. Please elaborate.

Simple: If someone adds a malloc they'll either add back the goto (making
this patch useless history clutter) or even more likely they'll forget
to add the free in half the cases, causing a memleak we won't detect
since we don't properly test error cases.
Or in other words: because it increases the likelihood of future bugs
for no real benefit.


More information about the ffmpeg-cvslog mailing list