Re: Fwd: Re: [PATCH][IN PROGRESS] gbuild conversion: module cpputools

2012-04-11 Thread Caolán McNamara
On Wed, 2012-04-11 at 12:51 +0200, Michael Stahl wrote: > (in Writer you can actually find places where optimization has been > disabled in 1995 or so, and apparently nobody ever checked if it's still > necessary...) Yeah, its incredibly annoying to just disable optimization for something without

Re: Fwd: Re: [PATCH][IN PROGRESS] gbuild conversion: module cpputools

2012-04-11 Thread Michael Stahl
On 11/04/12 12:02, Matúš Kukan wrote: > On 8 April 2012 10:29, David Tardon wrote: >> >> 4. regsingleton sets an additional compiler flag for MSC. You can use >> gb_Executable_add_cxxflags to do that in gbuild land. > > Hmm, what's this -Ob0 ? Isn't it some kind of optimization flag ? > It seem

Re: Fwd: Re: [PATCH][IN PROGRESS] gbuild conversion: module cpputools

2012-04-11 Thread Matúš Kukan
On 8 April 2012 10:29, David Tardon wrote: > > 4. regsingleton sets an additional compiler flag for MSC. You can use >   gb_Executable_add_cxxflags to do that in gbuild land. Hmm, what's this -Ob0 ? Isn't it some kind of optimization flag ? It seems to be default one [1]. I don't think we need to

Re: Fwd: Re: [PATCH][IN PROGRESS] gbuild conversion: module cpputools

2012-04-08 Thread David Tardon
On Wed, Apr 04, 2012 at 06:33:44AM +0200, David Tardon wrote: > On Wed, Apr 04, 2012 at 12:50:47AM +0200, David Ostrovsky wrote: > > Hi David, > > > > thank you for your comments. Here is another try. Hi, I looked at the patch and there are still some problems to correct. I could easily do it my

Re: Fwd: Re: [PATCH][IN PROGRESS] gbuild conversion: module cpputools

2012-04-03 Thread David Tardon
On Wed, Apr 04, 2012 at 12:50:47AM +0200, David Ostrovsky wrote: > Hi David, > > thank you for your comments. Here is another try. > As you haven't merged feature/gbuild_components branch to master > I let "gb_Executable_add_api" in place: Good, I will look at the patch and push it (with appropr