show zero accounts option patch

[Derek Atkins]

Try CVS now.  I think I've got a fix in there, and it seems to work
for me.  Is this what you were looking for?

-derek

Derek Atkins <warlord at MIT.EDU> writes:

> Fabien COELHO <fabien at coelho.net> writes:
>
>>> ok, this is in CVS now.
>>
>> Thanks.
>>
>> I had a "segmentation fault" yesterday night that may have been triggered
>> by this patch on gnucash 1.8.8. It may be so that it explores "untested"
>> code areas. If I can reproduce it, I'll send a test case.
>
> Hmm, I don't see a SegV, but I do see a Stack Overflow due to the
> recursive (but not tail-recursive) calls between use-acct?,
> show-acct?, my-get-balance, and my-get-balance-nosub in
> html-utilities.scm.  It seems poorly designed in the face of
> show-zero-entries? being false.
>
> I'm not exactly sure what the right approach is to fix it.  Somehow
> these four functions need to be disentangled, or you need to
> precompute all your values and use a lookup table rather than the
> silly algorithm currently in place to compute balances on the fly for
> every account over and over and over again.
>
> I'll try to think of a way to fix it, but feel free to continue
> working on it yourself.
>
> -derek
>
> -- 
>        Derek Atkins, SB '93 MIT EE, SM '95 MIT Media Laboratory
>        Member, MIT Student Information Processing Board  (SIPB)
>        URL: http://web.mit.edu/warlord/    PP-ASEL-IA     N1NWH
>        warlord at MIT.EDU                        PGP key available
> _______________________________________________
> gnucash-devel mailing list
> gnucash-devel at gnucash.org
> https://lists.gnucash.org/mailman/listinfo/gnucash-devel
>
>

-- 
       Derek Atkins, SB '93 MIT EE, SM '95 MIT Media Laboratory
       Member, MIT Student Information Processing Board  (SIPB)
       URL: http://web.mit.edu/warlord/    PP-ASEL-IA     N1NWH
       warlord at MIT.EDU                        PGP key available