Re: [gcmd-dev] Migration to C++

[Piotr Eljasiak]

> > Magnus - I'm sending small patch for adding a few TRACE() calls around
> > code responsible for popup menu creation. Could you please apply it
> > (patch -p1 < gcmd-rmb-popup.patch) and run gcmd from console? The
> > interesting part of log comes at the end of output, for me it looks
> > like:
> > 
> 
> Now this is strange...
> When running it from the source dir, it works.
> But when i make a pckage and install (/usr/local)
> and run then it installed, then the problem is back,
> crash on right-click.
> 
> And now it gets even stranger..
> After it is installed, it dies when i run it from the
> source tree too.
> 
> Then i did an uninstall, and now it works
> just fine when i run it from the source /src
> 
> Maybe this is the reason that i am the only one having
> this problem. I usually have the stable 1.2.x installed
> in /usr, and 1.3.x installed in /usr/local
> It is just on rare occasions that i run it in the source
> tree. Foolish? Well maybe, but 1.3.x gets well tested
> since i use it everyday for normal tasks (that is when
> it works and do not crash - now i use 1.2.3 for real tasks).

Do you have the same crash when you have only one gcmd installed ? Maybe
there are some remnants of previous versions of gcmd still installed
somewhere?

> 
> Some output with your patch.
> 
> -------------------------------------------------------
> For a jpeg with default app (installed and crashing):
> 
> 
> gnome-cmd-file-popmenu.cc(544) GtkWidget* 
> gnome_cmd_file_popmenu_new(GnomeCmdFileList*) files: 0x88da40
> gnome-cmd-file-popmenu.cc(417) gchar* 
> get_default_application_action_name(GList*)       files: 0x88da40
> gnome-cmd-file-popmenu.cc(418) gchar* 
> get_default_application_action_name(GList*)       g_list_length(files): 1
> gnome-cmd-file-popmenu.cc(424) gchar* 
> get_default_application_action_name(GList*)       finfo: 0x82c010
> gnome-cmd-file-popmenu.cc(426) gchar* 
> get_default_application_action_name(GList*)       uri_str: 
> file:///home/magnus/Discovery.jpg
> gnome-cmd-file-popmenu.cc(428) gchar* 
> get_default_application_action_name(GList*)       app: 0x93e220
> gnome-cmd-file-popmenu.cc(436) gchar* 
> get_default_application_action_name(GList*)       escaped_app_name: 
> Bildvisaren gThumb
> gnome-cmd-file-popmenu.cc(439) gchar* 
> get_default_application_action_name(GList*)       retval: _Open with 
> "Bildvisaren gThumb"
> gnome-cmd-file-popmenu.cc(548) GtkWidget* 
> gnome_cmd_file_popmenu_new(GnomeCmdFileList*) open_uiinfo[0].label: 
> _Open with "Bildvisaren gThumb"

It looks like the problem is not inside
get_default_application_action_name(), but somewhere further in
gnome_cmd_file_popmenu_new(). Let's drill it out a bit more - here goes
the next version of the patch. I'm curious where it crashes.


Piotr

gcmd-rmb-popup-2.patch.bz2
Description: application/bzip