Patch Re: Bug 625193 To move utils to core

Mike Evans mikee at saxicola.idps.co.uk
Tue Aug 17 16:39:28 EDT 2010


On Tuesday August 17 2010 20:54:42 Christian Stimming wrote:
> Am Monday 16 August 2010 schrieb Mike Evans:
> > I've attached a patch to move utils.c utils.h to src/engine as discussed
> > in Bug 625193.
> 
> Committed to SVN, thanks.
> 
> One suggestion for the function naming (in gncIDsearch.h): The functions
> which belong to a hypothetical "core gnucash library" should all begin
> with "gnc_", and the next word may or may not point to the main object on
> which this function operates. In this case, I'd suggest one of the
> following namings: * gnc_search_customer_on_id
> * gnc_book_search_by_id_customer
> * gnc_book_search_customer_by_id

Will do.  I wasn't sure about naming conventions.  I  also realised (just 
after posting) that much re-factoring is required and that I missed vendors 
out, so I will have to re-patch anyway.

Thanks and Regards MikeE




-- 
GPG Key: 1024D/050895C2
Keyserver: http://pgp.mit.edu/          
Search String: 0x050895C2


More information about the gnucash-devel mailing list