[Glade-devel] [PATCH/Glade-2.0.0] Re-enabling signal handler's "user_data" parameter

[jeannot12 linuxmail org (Jeannot Langlois)]

This is a multi-part message in MIME format...

------------=_1054833043-9801-1
Content-Type: text/plain; charset="iso-8859-1"
Content-Disposition: inline
Content-Transfer-Encoding: 7bit

Hello,



After I've sent my feature requests for the next version of Glade yesterday, I had a look at the Glade-2.0.0 
sources and I noticed that one of the features I was asking for had already been present in Glade-0.6.4, but 
had been dropped in Glade-2.0.0 :-(.  

I've decided to re-enable it, as it is *QUITE* valuable to me (as I've said previously it avoids me the pain 
of editing "interface.c", which gets overwritten constantly).  I am sure it will be valuable to other Glade 
users too.

I DO REQUIRE to pass pointers to certain signal handlers (like "clicked" or "activate" event handlers) to 
perform gtk_widget_destroy calls properly to avoid memory leaks and perform normal widget destruction; this 
"data" passing feature allows me to do it properly with no manual editing of "interface.c", keeping 
everything modular and, most importantly, avoiding GLOBAL widget pointers called from signal handlers.

Attached to this email is a tarball containing six patches for the following six files (present in the 
"glade" subdirectory of the "glade-2.0.0" hierarchy):

save.c
save.h
glade-parser.c
glade-parser.h
property.c
gbwidget.c

After being applied on these six files, these six patches re-enable the "Data" TextEntry field on the 
"Signals" tab from the "Properties" window in Glade-2.0.0.  

I've done a few tests and everything seems stable and to be working fine.  However, if I detect any weird 
behaviors I'll post another message later.  I'm actually working on a small UI project so occasions to test 
the patch myself will be plentiful.  

Everything with this patch should be fine, as I've inspired myself of the "Object" TextEntry field which is 
somewhat similar in concept to the "Data" TextEntry field, and remainders of the "Data" TextEntry feature 
from the Glade-0.6.4 code should suffice to do the trick properly.

I know this change will force a change in the Glade-2.0.0 XML file format, but I don't think it is THAT big 
and annoying.

I hope these patches will convince you that the "Data" TextEntry field in "Properties" >> "Signals" is an 
important feature, which I strongly believe should remain present in future versions of Glade.  


Thanks all.  

I hope I've made my point clear eheh :-).


I DO repeat it, Glade is a GREAT work !

Regards,



--
Jeannot Langlois
B. Sc. Computer Science
jeannot12 AT linuxmail DOT org

(When replying please CC: me at "jeannot12 AT linuxmail DOT org" as I may not be a member of the mailing list 
for long (web mail quotas are small... :-)).
-- 
______________________________________________
http://www.linuxmail.org/
Now with e-mail forwarding for only US$5.95/yr

Powered by Outblaze

------------=_1054833043-9801-1
Content-Type: application/x-gzip;
    name="callback_data_patch_for_glade-2.0.0.tar.gz"
Content-Disposition: attachment; filename="callback_data_patch_for_glade-2
    .0.0.tar.gz"
Content-Transfer-Encoding: base64

H4sIAB1q3z4AA+2Ya2/aMBSG+Up/hccnYAk4zq0RK1LVdVM3VE2F7stWRSYx
qdeQRInpVk3777MdLm3XlV4W2DQ/QuZmHx8TvyevicZfaRgR1gk6GWbBea0C
oAGhY1k1CKHh2lA8Q6N8L3FsowZdx+EPyzYc3t+0LLMGYBXJ3GZWMJwDUEtm
0zHJnXv6kbzYREKbxYDIDESz8wqUFDRKcKz3Q8ww2APHp4NBD3Tb4DhloJhl
WZozEgKcXE3TnHRAu7uj6/pO/+7RkV+wPJxloFl+4dNkkn6iZx3xfasHROB2
W3830BGEZhc6XWjr7fY8DAhwHI9xcAFkuJPD/YPR0cf90eFrPq0YK5PjL+9N
D7nubiAavsR6/VqO6fgLCZi2fB/jgvnTNKQTGmBG08RndEpavfkC7x0rElwf
qVzuU1a7U9H1j2IcEj3DOd/eVdWAdfq3kbnSvyn0byHDVvrfBMhDGHmmhjxL
bHESFwTQCWi+4LoNplkTM5YXXLAaaIhd2Wi16pynq1ZMIuqEqAOdeZHg3dNA
FAqaRE1+ORjR+zRhJJ/ggGhgnsJL44xL0UDmLjaQZWi8QSKayFZE0/vl9MWy
vNyVqf7oTL9N4yFhH/I0ayZpSBY/hAbunrRXlVKr4Yb+z7ejf34lV/oXXkDo
31L63wSWh204v3tHwTn/KdpiG/d+uTE/QjjbXpPi4WS8rpGcXW3V/0NnpX8X
Cv9vm8r/bwTD8pxANNL/R+zCD2LKzWtBmB+k8Wya+Je0oGMaU3YFmm9H7/2D
wdFwtDT0sntLA0gDb/YHw8OlXX5utNHJKQ/2LK9h2BYKRCMXt9jqMhuZRkx4
DuOhjPRaGvjbS3jImD+Q6Pauf4EvSZXaF6z1/9fO/w7knxvIgUjpfxN4FsKe
Zcrjbb10APyF9AA+d4V4qt1wAs7j9rXn2NhzXM1zLSmnZ50bgDyYrDJryc++
L/96iOZHCB9nGUnCsqveH88mE5Jz2y7D7n1uNLi+l39X8O3vlz18HIbzCLeH
Xpuz9+DpVhP94O3faoqk/ivy/QvW+n97df53+MFf6t9V+t8EhmlhbrdK/S8q
ALjBb4vBP3OTUygUCoVCoVAoFAqFQqH4j/kJ6IdwnwAoAAA=

------------=_1054833043-9801-1--