gEDA-dev: glib 2.8.x functions in gschem/src/x_menu.c

[Peter TB Brett]

On Thursday 10 May 2007 04:13:46 Ales Hvezda wrote:
> Hi,
>
> [snip]
>
> >This should be s_file_set_contents() in libgeda's s_basic.c, I believe. 
> > Al= es?
>
> 	I'm actually okay with leaving it in gschem, unless one of the other
> gaf programs need to use it.
>
> However, I would rather apply the other patch, the one that uses #ifdefs
> to enable/disable g_file_set_contents() instead of duplicating the code
> and then having to maintain it.  Now if we had several chunks of code
> wanting to use this function, then the other approach might prevail.

I agree: autotools even provides a mechanism for providing stuff like that 
("missing").

I'm testing the patch now, and will commit it soon.

Peter

-- 
Fisher Society committee                    http://tinyurl.com/o39w2
CUSBC novices, match and league secretary   http://tinyurl.com/mwrc9
CU Spaceflight                              http://tinyurl.com/ognu2

      09f911029d74e35bd84156c5635688c0            peter-b.co.uk
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
Url : http://www.seul.org/pipermail/geda-dev/attachments/20070510/09903be1/attachment.pgp