[FFmpeg-devel] [PATCH 1/5] lavc/htmlsubtitles: improve handling broken garbage

Michael Niedermayer michael at niedermayer.cc
Sun Jul 30 18:06:34 EEST 2017


On Sun, Jul 30, 2017 at 12:31:57PM +0200, Nicolas George wrote:
> Le duodi 12 thermidor, an CCXXV, Michael Niedermayer a écrit :
> > maybe but the compiler doesnt complain about them
> > i think the difference is the number of nested {}
> 
> Yes and no. This construct is valid, otherwise the gcc developers would
> not have removed the warning. The construct with the braces is also
> valid and more natural in specific cases, but less consistent for
> various types.

this is true for many warnings
still warnings like this can "hide" real issues by one not seeing them
in a wood of meaningless warnings.
That is i think we should disable this type of warning or if we cant
then do the generally trivial and rare change to avoid the warning


[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Modern terrorism, a quick summary: Need oil, start war with country that
has oil, kill hundread thousand in war. Let country fall into chaos,
be surprised about raise of fundamantalists. Drop more bombs, kill more
people, be surprised about them taking revenge and drop even more bombs
and strip your own citizens of their rights and freedoms. to be continued
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20170730/aba2d312/attachment.sig>


More information about the ffmpeg-devel mailing list