Request for review: patch for bug 499496

Daniel Harding dharding at
Thu Dec 27 04:04:07 EST 2007

Derek Atkins wrote:
> Hi,
> Daniel Harding <dharding at> writes:
>> Hello,
>> I have put together a patch for a bug I filed a few weeks back.  This is 
>> my first attempt to make any real changes to the GnuCash sources, so I'd 
>> appreciate feedback on both my approach to addressing the bug, as well 
>> as the format/style of the changes themselves.  I've attached the patch, 
>> along with a (hopefully clear) description of what it is trying to do, 
>> to the bug report:
> Thanks.  Committed to trunk as r16730.

Realized a small problem with my patch.  Got the logic reversed for an
#if which was intended to disable a warning which currently gets hit all
the time.  Here is a patch that fixes that (alternatively, the whole
chunk of code behind the #if could simply be removed).
Apologies for that,


-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: disable_warning.patch

More information about the gnucash-devel mailing list