Re: CVS commit: src/external/bsd/file

2014-01-18 Thread David Laight
On Sat, Jan 18, 2014 at 11:53:09AM +0100, Martin Husemann wrote: > On Sat, Jan 18, 2014 at 10:46:50AM +, Justin Cormack wrote: > > I believe gcc is correct here, you are only allowed to alias via a > > char pointer or the original type. Because you put a void pointer in > > instead gcc is right

Re: CVS commit: src/external/bsd/file

2014-01-18 Thread Christos Zoulas
On Jan 18, 11:33am, mar...@duskware.de (Martin Husemann) wrote: -- Subject: Re: CVS commit: src/external/bsd/file | On Fri, Jan 17, 2014 at 11:32:29PM +, David Laight wrote: | > If the problem actually caused by gcc failing to pair all the conditionals? | > Compiling with 'clazz' a compile-tim

Re: CVS commit: src/external/bsd/file

2014-01-18 Thread Martin Husemann
On Sat, Jan 18, 2014 at 10:46:50AM +, Justin Cormack wrote: > I believe gcc is correct here, you are only allowed to alias via a > char pointer or the original type. Because you put a void pointer in > instead gcc is right to complain. Right, but actually the complaint does not go away if we m

Re: CVS commit: src/external/bsd/file

2014-01-18 Thread Justin Cormack
On Sat, Jan 18, 2014 at 10:33 AM, Martin Husemann wrote: > On Fri, Jan 17, 2014 at 11:32:29PM +, David Laight wrote: >> If the problem actually caused by gcc failing to pair all the conditionals? >> Compiling with 'clazz' a compile-time constant might show things. > > I don't think so, and I'l

Re: CVS commit: src/external/bsd/file

2014-01-18 Thread Martin Husemann
On Fri, Jan 17, 2014 at 11:32:29PM +, David Laight wrote: > If the problem actually caused by gcc failing to pair all the conditionals? > Compiling with 'clazz' a compile-time constant might show things. I don't think so, and I'll leave a final fix to Christos as maintainer. > Or, if memcpy()