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

2018-12-13 Thread Masanobu SAITOH
Oops, sorry. Wrong mail address. On 2018/12/13 20:54, Masanobu SAITOH wrote:  Please pullup the following change to netbsd-8 branch. On 2018/12/12 15:36, SAITOH Masanobu wrote: Module Name:    src Committed By:    msaitoh Date:    Wed Dec 12 06:36:13 UTC 2018 Modified Files:

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

2018-12-13 Thread Masanobu SAITOH
Please pullup the following change to netbsd-8 branch. On 2018/12/12 15:36, SAITOH Masanobu wrote: Module Name:src Committed By: msaitoh Date: Wed Dec 12 06:36:13 UTC 2018 Modified Files: src/external/bsd/file/lib: Makefile Log Message: Use DPSRCS for magic.h. OK'd

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()

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

2014-01-18 Thread Justin Cormack
On Sat, Jan 18, 2014 at 10:33 AM, Martin Husemann mar...@duskware.de 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

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

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-time

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 to

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

2014-01-17 Thread Alan Barrett
On Thu, 16 Jan 2014, Joerg Sonnenberger wrote: Module Name:src Committed By: joerg Date: Thu Jan 16 23:36:52 UTC 2014 Modified Files: src/external/bsd/file/dist/src: file.c Log Message: Only use __dead if it exists. I think the code would be easier to read with

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

2014-01-17 Thread David Laight
On Fri, Jan 17, 2014 at 08:24:20PM +, Martin Husemann wrote: Module Name: src Committed By: martin Date: Fri Jan 17 20:24:20 UTC 2014 Modified Files: src/external/bsd/file: Makefile.inc Log Message: Make a gcc 4.8 warning non-fatal (couldn't find a way to avoid it,