Re: another shutdown patch



On Mon, 2002-11-11 at 16:23, Jeroen Zwartepoorte wrote:
> Hi Dave,
> 
> This patch fixes a lot of the issues the older patch had. I've tested it
> thoroughly and it works fine over here.
> 
> Any comments?
> 
> Thanks,
> 
> Jeroen

+       GtkWidget *button;
+
+       if (!anjuta_document_get_uri (ANJUTA_DOCUMENT (current))) {
+               char *label;
+
+               label = g_strdup (get_doc_label (docman, current));
+               filename = g_path_get_basename (label);
+       } else {
+               filename = g_strdup (anjuta_document_get_uri
(ANJUTA_DOCUMENT (current)));
+       }
+

Looks like you leak label here.

Other than that it looks ok, please commit.

-- 
Dave Camp <dave ximian com>




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