Re: src/sys/modules/spdmem

2011-08-19 Thread Alan Barrett
On Fri, 19 Aug 2011, David Holland wrote: I think we should make no changes to appease the compiler in this case. [...] I would lean towards fixing the ones that can be fixed noninvasively; [...] The compiler is being really stupid, and I don't like making invasive changes to appease it. No

Re: src/sys/modules/spdmem

2011-08-19 Thread Martin Husemann
On Fri, Aug 19, 2011 at 08:07:11AM +0200, Alan Barrett wrote: > If it helped, then it would also be fine to change > const char * format = "%s"; > to > const char * const format = "%s"; > but it doesn't help. but: const char format[] ="%s"; works just fine. Martin

re: src/sys/modules/spdmem

2011-08-19 Thread matthew green
> On Fri, 19 Aug 2011, David Holland wrote: > >> I think we should make no changes to appease the compiler in > >> this case. [...] > > > > I would lean towards fixing the ones that can be fixed > > noninvasively; [...] > > The compiler is being really stupid, and I don't like making > invasive

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

2011-08-19 Thread Christos Zoulas
In article <20110818142524.ga26...@britannica.bec.de>, Joerg Sonnenberger wrote: >On Thu, Aug 18, 2011 at 01:47:19AM +, Christos Zoulas wrote: >> In article <20110817212805.gb16...@britannica.bec.de>, >> Joerg Sonnenberger wrote: >> >Could you please stop randomly changing 3rd party code wi

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

2011-08-19 Thread Joerg Sonnenberger
On Fri, Aug 19, 2011 at 10:55:20AM +, Christos Zoulas wrote: > Every project I know off makes changes locally first and then pushes > them upstream. It is not practical to wait for upstream to be fixed > first, specially in cases of security fixes. In some cases we > maintain many thousands of

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

2011-08-19 Thread Jim Wise
Joerg Sonnenberger writes: > On Fri, Aug 19, 2011 at 10:55:20AM +, Christos Zoulas wrote: >> Every project I know off makes changes locally first and then pushes >> them upstream. It is not practical to wait for upstream to be fixed >> first, specially in cases of security fixes. In some case

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

2011-08-19 Thread Joerg Sonnenberger
On Fri, Aug 19, 2011 at 09:19:46AM -0400, Jim Wise wrote: > As long as I remember, we've had a strict policy of submitting changes > upstream where possible, but of _not_ gating fixes on this process -- > particularly fixes which are security or correctness related (the latter > includes fixes with

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

2011-08-19 Thread Thomas Klausner
I've sent the diff to Kristaps and he included it in his repository. Thomas On Wed, Aug 17, 2011 at 11:28:05PM +0200, Joerg Sonnenberger wrote: > Could you please stop randomly changing 3rd party code without > contacting the maintainer? > > Joerg > > On Wed, Aug 17, 2011 at 05:30:14AM -0400, C

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

2011-08-19 Thread Jim Wise
Joerg Sonnenberger writes: > On Fri, Aug 19, 2011 at 09:19:46AM -0400, Jim Wise wrote: >> As long as I remember, we've had a strict policy of submitting changes >> upstream where possible, but of _not_ gating fixes on this process -- >> particularly fixes which are security or correctness related

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

2011-08-19 Thread Joerg Sonnenberger
On Fri, Aug 19, 2011 at 10:57:53AM -0400, Jim Wise wrote: > And the project has a long history of making sure all (these days: most) > code in the base distribution builds with -Wall -Werror. This isn't > because every GCC warning is right, of course -- it's because when you > turn off warnings fo

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

2011-08-19 Thread Jim Wise
Joerg Sonnenberger writes: > On Fri, Aug 19, 2011 at 10:57:53AM -0400, Jim Wise wrote: >> And the project has a long history of making sure all (these days: most) >> code in the base distribution builds with -Wall -Werror. This isn't >> because every GCC warning is right, of course -- it's becau

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

2011-08-19 Thread Thomas Klausner
On Fri, Aug 19, 2011 at 07:54:28PM +0200, Joerg Sonnenberger wrote: > When have you last checked the number of -Wno-* instances in base? > Sorry, but pessimizing code to work around clearly bogus GCC warnings is > not helpful. It doesn't make code easier to read, it doesn't improve > code quality.