Re: [FFmpeg-devel] [PATCHv2] avformat/movenc: suppress -Wstrict-overflow warnings

2015-10-17 Thread Ganesh Ajjanagadde
On Sat, Oct 17, 2015 at 3:57 PM, Mark Harris wrote: >> -if (cluster_idx >= track->entry) >> +/* GCC 5.2 wants to "optimize" cluster_idx >= track->entry to the below >> + * expression. We actually mean cluster_idx >= track->entry. */ >> +if (cluster_idx - track->entry >= 0) >>

Re: [FFmpeg-devel] [PATCHv2] avformat/movenc: suppress -Wstrict-overflow warnings

2015-10-17 Thread Mark Harris
> -if (cluster_idx >= track->entry) > +/* GCC 5.2 wants to "optimize" cluster_idx >= track->entry to the below > + * expression. We actually mean cluster_idx >= track->entry. */ > +if (cluster_idx - track->entry >= 0) > return 0; On Sat, Oct 17, 2015 at 11:04 AM, Ganesh Aj

Re: [FFmpeg-devel] [PATCHv2] avformat/movenc: suppress -Wstrict-overflow warnings

2015-10-17 Thread Ganesh Ajjanagadde
On Sat, Oct 17, 2015 at 1:51 PM, Ronald S. Bultje wrote: > Hi, > > On Fri, Oct 16, 2015 at 8:45 PM, Ganesh Ajjanagadde > wrote: > >> On Fri, Oct 16, 2015 at 8:34 PM, Ronald S. Bultje >> wrote: >> > Hi, >> > >> > On Fri, Oct 16, 2015 at 8:18 PM, Ganesh Ajjanagadde >> > wrote: >> > >> >> On Fri,

Re: [FFmpeg-devel] [PATCHv2] avformat/movenc: suppress -Wstrict-overflow warnings

2015-10-17 Thread Ronald S. Bultje
Hi, On Fri, Oct 16, 2015 at 8:45 PM, Ganesh Ajjanagadde wrote: > On Fri, Oct 16, 2015 at 8:34 PM, Ronald S. Bultje > wrote: > > Hi, > > > > On Fri, Oct 16, 2015 at 8:18 PM, Ganesh Ajjanagadde > > wrote: > > > >> On Fri, Oct 16, 2015 at 8:05 PM, Ronald S. Bultje > >> wrote: > >> > Hi, > >> > >

Re: [FFmpeg-devel] [PATCHv2] avformat/movenc: suppress -Wstrict-overflow warnings

2015-10-16 Thread Ganesh Ajjanagadde
On Fri, Oct 16, 2015 at 8:34 PM, Ronald S. Bultje wrote: > Hi, > > On Fri, Oct 16, 2015 at 8:18 PM, Ganesh Ajjanagadde > wrote: > >> On Fri, Oct 16, 2015 at 8:05 PM, Ronald S. Bultje >> wrote: >> > Hi, >> > >> > On Fri, Oct 16, 2015 at 5:48 PM, Ganesh Ajjanagadde >> > wrote: >> > >> >> On Fri,

Re: [FFmpeg-devel] [PATCHv2] avformat/movenc: suppress -Wstrict-overflow warnings

2015-10-16 Thread Ronald S. Bultje
Hi, On Fri, Oct 16, 2015 at 8:18 PM, Ganesh Ajjanagadde wrote: > On Fri, Oct 16, 2015 at 8:05 PM, Ronald S. Bultje > wrote: > > Hi, > > > > On Fri, Oct 16, 2015 at 5:48 PM, Ganesh Ajjanagadde > > wrote: > > > >> On Fri, Oct 16, 2015 at 5:45 PM, Hendrik Leppkes > >> wrote: > >> > On Fri, Oct 1

Re: [FFmpeg-devel] [PATCHv2] avformat/movenc: suppress -Wstrict-overflow warnings

2015-10-16 Thread Ganesh Ajjanagadde
On Fri, Oct 16, 2015 at 8:05 PM, Ronald S. Bultje wrote: > Hi, > > On Fri, Oct 16, 2015 at 5:48 PM, Ganesh Ajjanagadde > wrote: > >> On Fri, Oct 16, 2015 at 5:45 PM, Hendrik Leppkes >> wrote: >> > On Fri, Oct 16, 2015 at 11:39 PM, Ganesh Ajjanagadde >> > wrote: >> >> On Wed, Oct 14, 2015 at 10:

Re: [FFmpeg-devel] [PATCHv2] avformat/movenc: suppress -Wstrict-overflow warnings

2015-10-16 Thread Ronald S. Bultje
Hi, On Fri, Oct 16, 2015 at 5:48 PM, Ganesh Ajjanagadde wrote: > On Fri, Oct 16, 2015 at 5:45 PM, Hendrik Leppkes > wrote: > > On Fri, Oct 16, 2015 at 11:39 PM, Ganesh Ajjanagadde > > wrote: > >> On Wed, Oct 14, 2015 at 10:05 PM, Ganesh Ajjanagadde > >> wrote: > >>> This patch results in iden

Re: [FFmpeg-devel] [PATCHv2] avformat/movenc: suppress -Wstrict-overflow warnings

2015-10-16 Thread Ganesh Ajjanagadde
On Fri, Oct 16, 2015 at 5:45 PM, Hendrik Leppkes wrote: > On Fri, Oct 16, 2015 at 11:39 PM, Ganesh Ajjanagadde > wrote: >> On Wed, Oct 14, 2015 at 10:05 PM, Ganesh Ajjanagadde >> wrote: >>> This patch results in identical behavior of movenc, and suppresses >>> -Wstrict-overflow >>> warnings obs

Re: [FFmpeg-devel] [PATCHv2] avformat/movenc: suppress -Wstrict-overflow warnings

2015-10-16 Thread Hendrik Leppkes
On Fri, Oct 16, 2015 at 11:39 PM, Ganesh Ajjanagadde wrote: > On Wed, Oct 14, 2015 at 10:05 PM, Ganesh Ajjanagadde > wrote: >> This patch results in identical behavior of movenc, and suppresses >> -Wstrict-overflow >> warnings observed in GCC 5.2: >> http://fate.ffmpeg.org/log.cgi?time=201509262

Re: [FFmpeg-devel] [PATCHv2] avformat/movenc: suppress -Wstrict-overflow warnings

2015-10-16 Thread Ganesh Ajjanagadde
On Wed, Oct 14, 2015 at 10:05 PM, Ganesh Ajjanagadde wrote: > This patch results in identical behavior of movenc, and suppresses > -Wstrict-overflow > warnings observed in GCC 5.2: > http://fate.ffmpeg.org/log.cgi?time=20150926231053&log=compile&slot=x86_64-archlinux-gcc-threads-misc, > "warning:

[FFmpeg-devel] [PATCHv2] avformat/movenc: suppress -Wstrict-overflow warnings

2015-10-14 Thread Ganesh Ajjanagadde
This patch results in identical behavior of movenc, and suppresses -Wstrict-overflow warnings observed in GCC 5.2: http://fate.ffmpeg.org/log.cgi?time=20150926231053&log=compile&slot=x86_64-archlinux-gcc-threads-misc, "warning: assuming signed overflow does not occur when assuming that (X - c) >

[FFmpeg-devel] [PATCHv2] avformat/movenc: suppress -Wstrict-overflow warnings

2015-09-26 Thread Ganesh Ajjanagadde
This patch results in identical behavior of movenc, and suppresses -Wstrict-overflow warnings observed in GCC 5.2: http://fate.ffmpeg.org/log.cgi?time=20150926231053&log=compile&slot=x86_64-archlinux-gcc-threads-misc, "warning: assuming signed overflow does not occur when assuming that (X - c) >