[FFmpeg-devel] [PATCH] fix 'warning: dereferencing type-punned pointer will break strict-aliasing rules' warning
Wed Mar 5 15:11:38 CET 2008
On Wed, Mar 5, 2008 at 11:27 PM, Michael Niedermayer <michaelni at gmx.at>
> On Wed, Mar 05, 2008 at 08:08:54PM +0800, Zuxy Meng wrote:
> > Hi,
> > 2008/3/5, Michael Niedermayer <michaelni at gmx.at>:
> > > On Wed, Mar 05, 2008 at 12:51:04PM +0100, Michael Niedermayer wrote:
> > > > On Wed, Mar 05, 2008 at 09:47:26PM +1100, mark cox wrote:
> > > > > Here is a patch to fix the 'warning: dereferencing type-punned
> pointer will
> > > > > break strict-aliasing rules' warning.
> > > >
> > > > patch rejected
> > >
> > > Just to clarify even though it should be clear to every C programmer.
> > > The warnings point to bugs, the bugs are not fixed by this patch.
> > Someone willing to tell me the history about the "declared as uint64_t
> > and cast to short" thing? Looks to me as a workaround for some
> > compiler bug.
> Yes, old versions of gcc (dont ask which i dunno) ignored align attributes
> for stack variables entirely, but they did align uint64_t to 8 bytes even
> when it was on the stack (given that the stack was properly aligned).
What would be the correct way to fix this then? or is it unfixable while we
still support old gcc versions?
> Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
> Dictatorship naturally arises out of democracy, and the most aggravated
> form of tyranny and slavery out of the most extreme liberty. -- Plato
> -----BEGIN PGP SIGNATURE-----
> Version: GnuPG v1.4.6 (GNU/Linux)
> -----END PGP SIGNATURE-----
> ffmpeg-devel mailing list
> ffmpeg-devel at mplayerhq.hu
More information about the ffmpeg-devel