Re: [gnome-network]PATCHES for bug 114826
- From: Mason Kidd <mason_kidd mrkidd com>
- To: Emil Soleyman-Zomalan <emil nishra com>
- Cc: Rodrigo Moya <rodrigo gnome-db org>, "gnome-network-list gnome org" <gnome-network-list gnome org>
- Subject: Re: [gnome-network]PATCHES for bug 114826
- Date: Tue, 30 Dec 2003 20:18:39 -0800
This final patch was applied, right? If so, bugs #114826 and #122476
can both be marked resolved (I don't have privs to do it).
On Mon, 2003-12-08 at 18:53, Emil Soleyman-Zomalan wrote:
> On Sun, 2003-12-07 at 12:25, Rodrigo Moya wrote:
> > this one looks perfect and has now been applied to CVS, except for the
> > .png file. Could you please resend that file, so that I add it to CVS?
> You can find the PNG file at
> > > http://www.nishra.com/patches/help-and-err.patch
> > >
> > this one has some small problems. For instance, you replaced all
> > #include's with an "#include gnome.h". While this is correct, it slows
> > down compilation, since a lot of unneeded header files are used during
> > the compilation. So please don't do that, keep the headers as they are.
> I understand. My recent patch reverts this change.
> > Also, please remove the big /*** for the functions you added. Those are
> > private functions, so there is no need to have them documented as you
> > did. Just add a quick comment (/* this does this */).
> > Also, in some:
> > + title = g_strdup_printf ("Failed to create a connection");
> > + reason = g_strdup_printf ("The host cannot be found");
> > + create_error (title, reason);
> > +
> > + g_free (title);
> > + g_free (reason);
> > there is really no reason at all to allocate memory for 'reason' :-) Just
> > pass the const string to the create_error function.
> That's also done. I just thought that it was cleaner by creating a new
> variable and passing both title and reason to create_error(). But this
> also works.
> The new patch can be found at:
][Date Next] [Thread Prev