[PATCH] stdalign: relax _Alignof and tighten _Alignas test

I pushed these afterthought fixes to stdalign: * m4/stdalign.m4 (gl_STDALIGN_H): Relax the _Alignof test, as it was too strict: alignof must divide offsetof, but it need not equal offsetof. Inspired by Joseph S. Myers's comment . Conversely,

Re: Patches to README-release

Hi Jim, On 28 ม.ค. 2012, at 1:21, Jim Meyering wrote: > Reuben Thomas wrote: >> Ping? The patches still apply cleanly to HEAD. >> >> On 22 December 2011 19:54, Reuben Thomas wrote: >>> -* Ensure that the desired versions of autoconf, automake, etc. >>> - are in your PATH. See the buildreq lis

Re: [PATCH gnulib 1/4] malloca: Avoid warnings on x86_64 mingw64.

Marc-André Lureau wrote: > * lib/malloca.c: Include . > (mmalloca, freea): Use uintptr_t to convert pointers to integers. > * modules/malloca (Depends-on): Add stdint. > * modules/relocatable-prog-wrapper (Depends-on): Likewise. > With review from Bruno Haible . This patch was already committed o

Re: [libvirt] [PATCH libvirt 6/6] Cast timeval.tv_sec long to localtime expected type time_t

Marc-André Lureau wrote: > >>> Oops. struct timeval is generating a struct timeval with a 32-bit > >>> tv_sec even when time_t is 64-bits. > ... > Mingw64 reviewer said Windows actually defines it. I dig the msdn and > found this: > > http://msdn.microsoft.com/en-us/library/windows/desktop/ms7405

Re: test-stdalign.c breaks on i686 on Fedora Rawhide (gcc 4.7)

Hi Paul, > I expect this is a GCC 4.7 bug Yes, I confirm: I can reproduce it with gcc 4.7-20120121 built from source. Since this is not a regression, it will most likely not be fixed in time for GCC 4.7.0 [1]. [1] http://gcc.gnu.org/ml/gcc/2012-01/msg00337.html > I pushed the following workarou

Re: test-stdalign.c breaks on i686 on Fedora Rawhide (gcc 4.7)

Thanks for reporting this. I expect this is a GCC 4.7 bug, and have filed a bug report. I pushed the following workaround into Gnulib. If I'm wrong and it's not a GCC bug then we can revisit this. stdalign: check that alignof and offsetof are consistent * m4/stdalign.m4 (gl_STDALIGN_H): Check

Re: Patches to README-release

Reuben Thomas wrote: > Ping? The patches still apply cleanly to HEAD. > > On 22 December 2011 19:54, Reuben Thomas wrote: >> A series of patches that attempt to improve a few minor formatting >> problems and also streamline some of the steps. ... >> >> From 1949b5fc095e746bac4ab90c30b373b79aa6507d

Re: Patches to README-release

Ping? The patches still apply cleanly to HEAD. On 22 December 2011 19:54, Reuben Thomas wrote: > A series of patches that attempt to improve a few minor formatting > problems and also streamline some of the steps. > > From a7fef6319b09420f26fc6d2a508322b7c21b7997 Mon Sep 17 00:00:00 2001 > From:

Re: [libvirt] [PATCH libvirt 6/6] Cast timeval.tv_sec long to localtime expected type time_t

On Thu, Jan 26, 2012 at 4:42 PM, Marc-André Lureau wrote: > On Thu, Jan 26, 2012 at 3:01 PM, Bruno Haible wrote: >> Eric Blake wrote: >>> <_timeval.h>: >>> struct timeval >>> { >>>         long tv_sec; >>>         long tv_usec; >>> }; >>> >>> Oops.  struct timeval is generating a struct timeval w

Re: sys/types.h on Android includes stdint.h before defining time_t

On Fri, 27 Jan 2012 08:25, si...@josefsson.org said: > Great! I suppose you'll report back what the necessary patches for > GnuPG were to Werner? It would be nice to have it build unpatched. I'll take care of it. Shalom-Salam, Werner -- Die Gedanken sind frei. Ausnahmen regelt ein Bund

Re: sys/types.h on Android includes stdint.h before defining time_t

On Jan 27, 2012, at 2:25 AM, Simon Josefsson wrote: > Hans-Christoph Steiner writes: > >> On 01/26/2012 03:59 PM, Paul Eggert wrote: >>> On 01/26/2012 12:14 PM, Hans-Christoph Steiner wrote: ./stdint.h:32:3: error: invalid preprocessing directive #@ >>> >>> This looks like some sort of pr

Re: locale_charset() on MacOS X

On 27 Jan 2012, at 02:14, Bruno Haible wrote: > - On MacOS X nearly all users are working in the "C" locale. If a user >has told the OS that he's working in the French locale, the OS does >not set LC_* variables to indicate this, nor does the user usually >do so (why should he? he has

Re: update-copyright request and patch...

Jim Meyering wrote: > Joel Brobecker wrote: >>> > Yep, this is what we did for this year. I just want to go one step >>> > further and just collapse all years into one single range, based on >>> > the fact that I have verified that we have made significant changes >>> > to at least one file each ye

Re: update-copyright request and patch...

Joel Brobecker wrote: >> > Yep, this is what we did for this year. I just want to go one step >> > further and just collapse all years into one single range, based on >> > the fact that I have verified that we have made significant changes >> > to at least one file each year since 1986. >> >> I hav

Re: test-stdalign.c breaks on i686 on Fedora Rawhide (gcc 4.7)

On Fri, Jan 27, 2012 at 12:57:20AM +0100, Bruno Haible wrote: > Richard W.M. Jones wrote: > > CC test-stdalign.o > > Please, can you show the entire compiler command? Run "make V=1" so that > 'make' prints the real command. Sure, it's: depbase=`echo test-stdalign.o | sed 's|[^/]*$|.deps/&