Hi Forest, all three patches are online as r17829. Thanks a lot!
As usual, I cannot really help with report changes, I simply checked the obvious things and hope that Josh and all the others will contribute better feedback now. Regarding white space in .scm files, I have got the feeling that it is a mess. Personally, I would prefer space-only, emacs style indented files, but committing massive whitespace changes makes backports a lot harder. One thing I observed though is that opening parentheses on single lines after lets look strange to me :-) Ciao, -- andi5 Forest Bond wrote: > Hi Andreas, > > Attached are two patches correcting issues with my recently submitted > budget-related reports. As requested, these patches apply on top of my > previously-sent patch. > > As always, your feedback is appreciated. > > The whitespace conventions for .scm files are not entirely clear to me. The > whitespace in my scheme code is quirky. It appears that this is not entirely > unusual in the existing report code, but I may follow up with a patch to > correct > this, unless such patches are generally not preferred. > > Thanks for your time. > > Thanks, > Forest
signature.asc
Description: OpenPGP digital signature
_______________________________________________ gnucash-devel mailing list gnucash-devel@gnucash.org https://lists.gnucash.org/mailman/listinfo/gnucash-devel
