[GNC-dev] gnucash maint: Multiple changes pushed

[Wm]

On 26/02/2019 11:56, Christopher Lam wrote:
> Hi Geert,
> Sharp eye indeed.
> I haven't used the new gnc:gui-error function because report.scm is being
> simulateously attacked:
> (1) refactoring in maint-scheme-progress
> (2) slowly creating nearly 100% coverage for tests
> (3) fixing an invalid code path introduced about 8 months ago, which was
> returning #f to signify failure for a report-definition-without-guid, when
> in reality the report-definition-without-guid was correctly handled by
> autogenerating guid.
> I considered gnc:gui-error to be a proper refactoring job belonging to (1)
> in my unpushed maint-scheme-progress branch but needed to create tests
> during (2) first to ensure the refactoring was safe, and the tests needed
> to handle non-gui code properly.
> I felt that completing (2) was more important than (1).
> But you're right, we could easily have used gnc:gui-error here.

I love your confidence, Cristopher Lam.

-- 
Wm