Re: gnucash master: Multiple changes pushed

2017-02-21 Thread John Ralls
> On Feb 21, 2017, at 9:16 AM, David T. wrote: > > >> On Feb 21, 2017, at 10:06 PM, John Ralls wrote: >> >> >>> On Feb 21, 2017, at 8:43 AM, Derek Atkins wrote: >>> >>> It's weird that this isn't caught by the *compiler* -- isn't

Re: Missing Guile Bindings and False Positive Unit tests

2017-02-21 Thread Ted Creedon
FYI there is a swig problem in OpenSuse leap 4.2. Its not the latest version. Tedc From: gnucash-devel on behalf of Derek Atkins Sent: Tuesday, February 21, 2017 8:59:27 AM To:

Re: Eclipse CDT IDE

2017-02-21 Thread Ted Creedon
Derick No I just thought you might be interested. I use my own builder shell script, not the built ins for heimdal + oafs. That solved many problems. The FIXME suggestions are a great help in cleaning up old code. ted From: Geert Janssens

Re: gnucash master: Multiple changes pushed

2017-02-21 Thread David T. via gnucash-devel
> On Feb 21, 2017, at 10:06 PM, John Ralls wrote: > > >> On Feb 21, 2017, at 8:43 AM, Derek Atkins wrote: >> >> It's weird that this isn't caught by the *compiler* -- isn't an unused >> parameter a warning/error? > > It is by default, but we turn it off

Re: Eclipse CDT IDE

2017-02-21 Thread Geert Janssens
Op dinsdag 21 februari 2017 16:38:10 CET schreef Ted Creedon: > Just a suggestion, > > I've finally succeeded in getting openafs up under the IBM eclipse IDE. > > Its my opinion that the task/team/remote debugging feature would reduce > support effort considerably. > > I encourage

Re: gnucash master: Multiple changes pushed

2017-02-21 Thread John Ralls
> On Feb 21, 2017, at 8:43 AM, Derek Atkins wrote: > > Geert Janssens > writes: > >> Hi John, >> >> Op maandag 20 februari 2017 19:06:19 CET schreef John Ralls: >> >>> @@ -246,7 +246,7 @@ inline GncRational

Re: gnucash master: Multiple changes pushed

2017-02-21 Thread Geert Janssens
Op dinsdag 21 februari 2017 11:43:35 CET schreef Derek Atkins: > Geert Janssens writes: > > Hi John, > > > > Op maandag 20 februari 2017 19:06:19 CET schreef John Ralls: > >> @@ -246,7 +246,7 @@ inline GncRational operator+(GncRational a, GncInt128 > >> b) } > >> >

Re: Missing Guile Bindings and False Positive Unit tests

2017-02-21 Thread Derek Atkins
Hi Chad, Please remember to CC gnucash-user on all replies so that everyone can remain "in the loop". Also, the gnc-modules should properly export swig-exported functions without a re-export. If it stopped doing that, that would be a bug IMHO. Note that these unit tests are probably 18 years

Re: Eclipse CDT IDE

2017-02-21 Thread Derek Atkins
Ted, Ted Creedon writes: > Just a suggestion, > > I've finally succeeded in getting openafs up under the IBM eclipse IDE. > > Its my opinion that the task/team/remote debugging feature would > reduce support effort considerably. > > I encourage experimenting with it.

Re: gnucash master: Multiple changes pushed

2017-02-21 Thread Derek Atkins
Geert Janssens writes: > Hi John, > > Op maandag 20 februari 2017 19:06:19 CET schreef John Ralls: > >> @@ -246,7 +246,7 @@ inline GncRational operator+(GncRational a, GncInt128 b) >> } >> inline GncRational operator+(GncInt128 a, GncRational b) >> { >> -return

Eclipse CDT IDE

2017-02-21 Thread Ted Creedon
Just a suggestion, I've finally succeeded in getting openafs up under the IBM eclipse IDE. Its my opinion that the task/team/remote debugging feature would reduce support effort considerably. I encourage experimenting with it. Tedc ___

Re: gnucash master: Multiple changes pushed

2017-02-21 Thread Geert Janssens
Op dinsdag 21 februari 2017 07:07:58 CET schreef John Ralls: > Interesting. I didn't get that particular failure on Travis; I got it on my > local Ubuntu 14.04 VM which I use to debug Travis failures. It didn't even > occur to me that the exception (which isn't the one the standard requires > and

Re: cpp branch

2017-02-21 Thread Geert Janssens
Op dinsdag 21 februari 2017 11:33:08 CET schreef Robert Fewell: > Geert, > > Just an observation, when I do an import and select the date column first > with an invalid default format, the view goes red and the error message > tells me what is wrong. At the same time I ended up with 20 WARN

Re: gnc-numeric

2017-02-21 Thread Geert Janssens
Op dinsdag 21 februari 2017 11:18:38 CET schreef Robert Fewell: > Hi, > > Just built from master with all the new changes but had to change > gnc-numeric.cpp line 86 from isnan(d) to std::isnan(d), gcc version 4.9.4 > > After change all built successfully. Interestingly, I don't need to do

Re: gnucash master: Multiple changes pushed

2017-02-21 Thread Geert Janssens
Hi John, Op maandag 20 februari 2017 19:06:19 CET schreef John Ralls: > @@ -246,7 +246,7 @@ inline GncRational operator+(GncRational a, GncInt128 b) > } > inline GncRational operator+(GncInt128 a, GncRational b) > { > -return b + GncRational(a, 1); > +return GncRational(a, 1) + a; > }

Re: gnc-numeric

2017-02-21 Thread Matthew Forbis via gnucash-devel
On a similiar but slightly unrelated note, on commit https://github.com/Gnucash/gnucash/commit/e322457e , under functioninline GncRational operator+(GncInt128 a, GncRational b), I think it possibly should be | - return b + GncRational(a, 1); | | + return GncRational(a, 1) + b; instead of

Re: cpp branch

2017-02-21 Thread Robert Fewell
Geert, Just an observation, when I do an import and select the date column first with an invalid default format, the view goes red and the error message tells me what is wrong. At the same time I ended up with 20 WARN messages, one for each line in the trace file. I was wondering if

gnc-numeric

2017-02-21 Thread Robert Fewell
Hi, Just built from master with all the new changes but had to change gnc-numeric.cpp line 86 from isnan(d) to std::isnan(d), gcc version 4.9.4 After change all built successfully. Bob ___ gnucash-devel mailing list gnucash-devel@gnucash.org