[FFmpeg-cvslog] r11889 - trunk/libavutil/mem.h

Rich Felker dalias
Mon Feb 11 19:35:24 CET 2008


On Mon, Feb 11, 2008 at 07:17:18PM +0100, Reimar D?ffinger wrote:
> Hello,
> On Mon, Feb 11, 2008 at 12:51:08PM -0500, Rich Felker wrote:
> > On Mon, Feb 11, 2008 at 11:45:38AM +0100, Reimar D?ffinger wrote:
> > > Huh? With my suggestion the gcc variant is the fallback case...
> > 
> > The fallback case needs to be C, not "GNU C" or "MS Visual C++" or
> > whatever they call it now. A plain C build has no need for alignment
> > beyond what the compiler provides, nor any need for asm constants
> > since there's no asm.
> 
> I'll agree at once if you change configure so this is the case.
> But the fallback case being "plain C but with asm that is sure to crash
> because of missing alignment" is idiocy.

Umm, does configure really detect the support of asm without actually
checking for it?? Somehow I doubt this. It should at least look for
the __GNUC stuff or attempt to compile a file with inline asm..

Rich




More information about the ffmpeg-cvslog mailing list