Teste rapido com valgrind, retirando erros no gerenciamento de memória

perry.werneck@gmail.com
1 parent 4a8e884a
Showing 5 changed files with 26 additions and 4 deletions Show diff stats
pw3270.cbp
src/gtk/dialog.c
src/gtk/v3270/widget.c
src/gtk/valgrind.suppression
ui/00default.xml
@@ -134,6 +134,7 @@
 		<Unit filename="src/gtk/v3270/widget.c">
 			<Option compilerVar="CC" />
 		</Unit>
+		<Unit filename="src/gtk/valgrind.suppression" />
 		<Unit filename="src/include/lib3270.h" />
 		<Unit filename="src/include/lib3270/action_table.h" />
 		<Unit filename="src/include/lib3270/actions.h" />
@@ -274,6 +274,7 @@
 		if(encattr)
 			g_free(encattr);
+		trace("Removing dialog %p",dialog);
 		gtk_widget_destroy(dialog);
 	}
@@ -934,7 +934,12 @@ void v3270_set_font_family(GtkWidget *widget, const gchar *name)
 	terminal = GTK_V3270(widget);
 	if(terminal->font_family)
+	{
+		if(!g_strcasecmp(terminal->font_family,name))
+			return;
 		g_free(terminal->font_family);
+		terminal->font_family = NULL;
+	}
 	if(!name)
 	{
@@ -942,11 +947,11 @@ void v3270_set_font_family(GtkWidget *widget, const gchar *name)
 		name = "courier new";
 	}
-	trace("%s(%s)",__FUNCTION__,name);
-
 	terminal->font_family = g_strdup(name);
 	terminal->font_weight = lib3270_get_toggle(terminal->host,LIB3270_TOGGLE_BOLD) ? CAIRO_FONT_WEIGHT_BOLD : CAIRO_FONT_WEIGHT_NORMAL;
+	trace("%s: %s (%p)",__FUNCTION__,terminal->font_family,terminal->font_family);
+
 	g_signal_emit(widget,v3270_widget_signal[SIGNAL_UPDATE_CONFIG], 0, "font-family", name);
 	v3270_reload(widget);
@@ -958,7 +963,6 @@ void v3270_set_font_family(GtkWidget *widget, const gchar *name)
 const gchar	* v3270_get_font_family(GtkWidget *widget)
 {
 	g_return_val_if_fail(GTK_IS_V3270(widget),NULL);
-
 	return GTK_V3270(widget)->font_family;
 }
@@ -94,3 +94,10 @@
    ...
 }
+{
+   gdk_window_get_root_coords
+   Memcheck:Cond
+   fun:gdk_window_get_root_coords
+   ...
+}
+
@@ -32,9 +32,18 @@
 	<menubar name='topmenu' topmenu='yes'>
 		<menu name='FileMenu' label='_File' >
+
+			<!---
+				Extended attributes for 'save' actions:
+
+				* overwrite='yes' to always overwrite file
+				* filename='PATH' to save to a predefined file (no user save dialog)
+
+			--->
 			<menuitem action='save' src='all' group='online' label='Save screen' />
 			<menuitem action='save' src='selected' group='selection' label='Save selected' />
 			<menuitem action='save' src='copy' group='clipboard' label='Save copy' />
+
 			<separator/>
 			<menuitem action='print' src='all' key='Print' icon='print' label='Print' />
 			<menuitem action='print' src='selected' group='selection' label='Print selected' />
@@ -176,7 +185,7 @@
 		<separator />
 		<menuitem action='backtab' label='Previous field' />
 		<menuitem action='tab' label='Next field'/>
-		<menuitem name="return" action='activate' label='Return' key='return' />
+		<menuitem name="return" action='activate' label='Return' key='Return' />
 		<separator />
 		<menuitem action='Quit'/>
	@@ -274,6 +274,7 @@		@@ -274,6 +274,7 @@
274	if(encattr)	274	if(encattr)
275	g_free(encattr);	275	g_free(encattr);
276		276
		277	+ trace("Removing dialog %p",dialog);
277	gtk_widget_destroy(dialog);	278	gtk_widget_destroy(dialog);
278	}	279	}
279		280
	@@ -94,3 +94,10 @@		@@ -94,3 +94,10 @@
94	...	94	...
95	}	95	}
96		96
		97	+{
		98	+ gdk_window_get_root_coords
		99	+ Memcheck:Cond
		100	+ fun:gdk_window_get_root_coords
		101	+ ...
		102	+}
		103	+