[Gnucash-changes] r12989 - gnucash/trunk/src/report - Check for invalid options tables when loading reports.

[Chris Shoemaker]

On Fri, Jan 27, 2006 at 01:15:33AM -0500, Chris Shoemaker wrote:
> Author: chris
> Date: 2006-01-27 01:15:31 -0500 (Fri, 27 Jan 2006)
> New Revision: 12989
> Trac: http://svn.gnucash.org/trac/changeset/12989
> 
> Modified:
>    gnucash/trunk/src/report/report-gnome/window-report.c
>    gnucash/trunk/src/report/report-system/report.scm
> Log:
>    Check for invalid options tables when loading reports.
>    Also, plug memory leak in my last commit.
> 
> 
> Modified: gnucash/trunk/src/report/report-system/report.scm
> ===================================================================
> --- gnucash/trunk/src/report/report-system/report.scm	2006-01-27 05:27:08 UTC (rev 12988)
> +++ gnucash/trunk/src/report/report-system/report.scm	2006-01-27 06:15:31 UTC (rev 12989)
> @@ -269,7 +269,7 @@
>      (if (>= id *gnc:_report-next-serial_*)
>          (set! *gnc:_report-next-serial_* (+ id 1)))
>      (gnc:report-add id r)
> -    id))
> +    #f))
>  

*sigh* This was a mistake.  I convinced myself that we didn't depend
on the return value, and we didn't used to, but we do now.  New-style
saved-reports won't restore until I'll fix this in my next commit.

-chris