[GnomeMeeting-devel-list] [PATCH] generalizes the use of gconf widgets extensions



Hi,

I spend quite some time digging through the results of "grep -r src/ -e
gconf > gconf.log". The goal was to check if the very nice functions
that are defined in lib/gconf_widgets_extensions.h were used everywhere.
They were not, and that patch tries to fix that.

I erased from gconf.log every line I had fixed (ie: every place where
gconf is used through a wrapper), so what is left is the list of the
places where gconf is used directly ; I join it for information, that
may be useful in some time.

It was quite a pain to browse through that file, because:
i) the functions of the wrapper have names similar to those of gconf
itself;
ii) the variable names in gconf_* are... not rare;
iii) there are a lot of comments with "gconf" in it too;
so I'm wonder if it wouldn't be better to have the wrapped functions
named differently (say gm_conf_* instead of gconf_*).

That patch is quite big, but should be easy to read...

Snark

Attachment: gconf.log.bz2
Description: application/bzip

Attachment: gconf.patch.bz2
Description: application/bzip



[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]