Re: CVS commit: src/sys/dev/usb

2019-02-07 Thread Michael van Elst
On Fri, Feb 08, 2019 at 07:12:28AM +0900, Rin Okuyama wrote: > Hi, > > What compiler options (or version, architecture, etc.) do you use? > I want to enable that warnings, but I cannot even with WARNS=5. The warnings came for assertions, so you need to build with DIAGNOSTIC, which is default for

Re: CVS commit: src/sys/dev/usb

2019-02-07 Thread Michael van Elst
On Fri, Feb 08, 2019 at 07:12:28AM +0900, Rin Okuyama wrote: > Hi, > > What compiler options (or version, architecture, etc.) do you use? That was just default settings for amd64 (and same for evbarm). > I want to enable that warnings, but I cannot even with WARNS=5. I suspect most warnings are

Re: CVS commit: src/sys/dev/usb

2019-02-07 Thread Rin Okuyama
Hi, What compiler options (or version, architecture, etc.) do you use? I want to enable that warnings, but I cannot even with WARNS=5. Thanks, rin On 2019/02/07 19:36, Michael van Elst wrote: Module Name:src Committed By: mlelstv Date: Thu Feb 7 10:36:20 UTC 2019 Modified Fil

Re: CVS commit: src/external/cddl/osnet/sys/sys

2019-02-07 Thread Christos Zoulas
In article <20190206201922.ga8...@homeworld.netbsd.org>, wrote: >On Wed, Feb 06, 2019 at 12:56:57PM -0500, Christos Zoulas wrote: >D.org> >> @@ -66,7 +66,7 @@ crgetgroups(cred_t *cr) >> { >> static gid_t gids[NGROUPS_MAX]; > >I feel uncomfortable seeing unnecessary use of static. >(I find i