Deprecated GTK Function Replacements
Michael McConville
mmcconville at mykolab.com
Tue Jul 7 22:15:04 EDT 2015
On Tue, Jul 07, 2015 at 10:08:59PM -0400, Michael McConville wrote:
> On Tue, Jul 07, 2015 at 10:02:57PM -0400, Elliott Sales de Andrade wrote:
> > On 7 July 2015 at 21:54, Ethan Blanton <elb at pidgin.im> wrote:
> > > Ethan Blanton spake unto us the following wisdom:
> > >> Michael McConville spake unto us the following wisdom:
> > >> > On Tue, Jul 07, 2015 at 07:49:38PM -0400, Michael McConville wrote:
> > >> > > I also just realized that I committed an incomplete version of the
> > >> > > patch. :P Using three different test machines is quite an experience.
> > >> > >
> > >> > > Eion and I are discussing adding a GTK compat function to reduce the
> > >> > > amount of boilerplate. It sounds like a good idea to me. I'll check back
> > >> > > in when I have one committed.
> > >> >
> > >> > https://hg.pidgin.im/soc/2015/mmcc/main/rev/9415ac64b6fa
> > >> >
> > >> > I still need to test it with 3.16.x.
> > >>
> > >> This looks much better to me. I like it.
> > >
> > > Actually, let me qualify that -- I think this should be
> > > pidgin_label_set_alignment, not gtk_label_set_alignment. But that's a
> > > small detail.
> >
> > On the one hand, this one is not a strict one-to-one compatibility
> > function, but on the other hand, everything in gtk3compat.h is not
> > intended to be public.
>
> Yeah, I was going off of gtk_grid_attach_defaults() and
> gtk_grid_attach_full(), which are defined in pidgin/gtk3compat.h and
> don't seem to be existing GTK functions. I can change their names to
> pidgin_* too. Objections?
To clarify, I think changing them to pidgin_* is a good idea so that
people don't go looking for them in the GTK API docs.
More information about the Devel
mailing list