> On Dec 11, 2016, at 11:32 AM, Robert Fewell <[email protected]> wrote: > > #15 0x00007ffff6c2431f in gnc_dmy2timespec_neutral (day=day@entry=2016, > month=month@entry=12, year=year@entry=2001)
The problem seems pretty obvious... > at gnc-date.cpp:1251 > ---Type <return> to continue, or q <return> to quit--- > #16 0x00007fffe038564d in parse_date_with_year (format=<optimized out>, > date_str=0x11f1220 "2016/12/01") at gnc-csv-model.c:212 > #17 parse_date (date_str=0x11f1220 "2016/12/01", format=1) at > gnc-csv-model.c:304 Here you're telling parse_date_with_year to use "d-m-y" with a y-m-d format, and so passing an invalid day to gnc_dmy2rimespec_neutral. I suspect that this is what led Geert to ask about invalid date values a few weeks ago. We don't want exceptions to leak out of C++ modules, so gnc_dmy2timespec_neutral (and every other call to a constructor in gnu-date.cpp) should be inside a try-catch block and catch should return an invalid result; callers should test for that result. C++ code should use GncDateTime directly and handle the exception. Aaron, are you still working on replacing Timespec with time64? Regards, John Ralls _______________________________________________ gnucash-devel mailing list [email protected] https://lists.gnucash.org/mailman/listinfo/gnucash-devel
