! Please note that this is a snapshot of our old Bugzilla server, which is read only since May 29, 2020. Please go to gitlab.xfce.org for our new server !
Use g_type_class_add_private and parent_class in xfrun4
Status:
RESOLVED: FIXED
Product:
Xfce-utils

Comments

Description Nick Schermer editbugs 2006-07-30 19:41:14 CEST
User-Agent:       Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.8.0.5) Gecko/20060727 Firefox/1.5.0.5 (Swiftfox)
Build Identifier: rev 22562

Some fixes in xfrun4

Reproducible: Always

Steps to Reproduce:
Comment 1 Nick Schermer editbugs 2006-07-30 19:42:27 CEST
Created attachment 690 
Patch

* Addde g_type_class_add_private
* Added NETK_APPLICATION_GET_PRIVATE definition
* Added parent_class and finalize this one
Comment 2 Nick Schermer editbugs 2006-07-30 19:50:00 CEST
Btw i think this also fixes the memory leak you were talking about Brain.
Comment 3 Brian J. Tarricone (not reading bugmail) 2006-07-30 21:19:38 CEST
Gah, I always forget to chain up in _finalize()!  Thanks for this; I can stop tearing out what little hair I have left.

BTW, why are you calling the macro NETK_*?  It's not a netk function...  Anyway, since it's only called once, no need for a macro at all.

Also, no need to _peek_parent().  The G_DEFINE_TYPE() macro already creates xfrun_dialog_parent_class and does that for you.

Anyway... fixed.
Comment 4 Nick Schermer editbugs 2006-07-30 21:37:44 CEST
(In reply to comment #3)
> BTW, why are you calling the macro NETK_*?  It's not a netk function... 
> Anyway, since it's only called once, no need for a macro at all.

Because i copied the lines from netk_application... stupid me! (Patch from
earlier this day).

> 
> Also, no need to _peek_parent().  The G_DEFINE_TYPE() macro already creates
> xfrun_dialog_parent_class and does that for you.

MM didn't know that, thanks for the info.

Bug #2091

Reported by:
Nick Schermer
Reported on: 2006-07-30
Last modified on: 2009-07-14

People

Assignee:
Brian J. Tarricone (not reading bugmail)
CC List:
0 users

Version

Attachments

Patch (1.62 KB, patch)
2006-07-30 19:42 CEST , Nick Schermer
no flags

Additional information