Patch, patch submission questions and general configure/make questions

[Tim M]

On Mon, Jun 6, 2011 at 4:15 AM, Christian Stimming
<christian at cstimming.de> wrote:
> The code under lib/... is copied from somewhere else and is not developed
> within gnucash. This means we don't want to touch it, except if there are
> obvious bugs that might need to be fixed.
>

Good point, thanks for the clarification.

>> I am wondering if it is desired to fix the build errors which are
>> appearing due to my configure options or if I should simply change
>> those options?  I have limited C++ experience so currently small
>> patches such as this are easy for me and probably a good use of my
>> time getting familiar with the code, but let me know if this is not
>> desired.  I do have some work I would like to accomplish on
>> new/improved budgeting features _eventually_ in the future if my
>> ability increases to the point that I can manage it, but I will only
>> have time and ability for smaller patches for the time being.
>
> Both are fine. Feel free to work on whatever you prefer for yourself right
> now. Feel free to submit patches for "simple" things such as comment clean-up
> or even code clean-up for ISO C89, as long as it doesn't make the code less
> readable. We are looking forward to seeing your patches!
>

I will avoid the --enable-iso-c option.

Cheers,
-Tim M