bug#9077: coreutils-8.12: fiemap.h uses "struct fiemap_extent fm_extents[0]; "

2011-07-14 Thread Paul Eggert
On 07/14/11 18:11, Pádraig Brady wrote: > feel free to push that OK, thanks, done.

bug#9077: coreutils-8.12: fiemap.h uses "struct fiemap_extent fm_extents[0]; "

2011-07-14 Thread Pádraig Brady
On 14/07/11 23:50, Paul Eggert wrote: > On 07/14/11 04:45, Pádraig Brady wrote: >> The non standard "zero length array" syntax was copied from >> the linux kernel headers. Changing this to a C99 flexible array >> as you suggest, should be fine according to the constraints listed here: > > Won't th

bug#9077: coreutils-8.12: fiemap.h uses "struct fiemap_extent fm_extents[0]; "

2011-07-14 Thread Paul Eggert
On 07/14/11 04:45, Pádraig Brady wrote: > The non standard "zero length array" syntax was copied from > the linux kernel headers. Changing this to a C99 flexible array > as you suggest, should be fine according to the constraints listed here: Won't this break non-GCC compilers that don't support C

bug#9077: coreutils-8.12: fiemap.h uses "struct fiemap_extent fm_extents[0]; "

2011-07-14 Thread Pádraig Brady
On 14/07/11 08:03, Joachim Schmitz wrote: > Hi folks > > fiemap.h uses > > struct fiemap_extent fm_extents[0]; > > My compiler (HP NonStop) disallows that. The following would be allowed > though: > > struct fiemap_extent fm_extents[]; The non standard "zero length array" syntax was copied fr

bug#9077: coreutils-8.12: fiemap.h uses "struct fiemap_extent fm_extents[0]; "

2011-07-14 Thread Joachim Schmitz
Hi folks fiemap.h uses struct fiemap_extent fm_extents[0]; My compiler (HP NonStop) disallows that. The following would be allowed though: struct fiemap_extent fm_extents[]; Bye, Jojo