libxfce4menu filesystem monitoring

Brian J. Tarricone bjt23 at cornell.edu
Mon Oct 1 20:56:00 CEST 2007


Jannis Pohlmann wrote:
> Am Mon, 1 Oct 2007 00:08:03 -0700
> schrieb "Brian J. Tarricone" <bjt23 at cornell.edu>:
>  
> The current monitor system state is just another step to really useful
> monitoring. Remember we talked about telling a menu which file has
> changed rather than just regenerating the whole menu? That will also
> fix caching. It will work like this:
> 
>  1. libxfce4menu asks xfdesktop/thunar to monitor a file/directory
>  2. a changed event occurs => xfdesktop/thunar tell libxfce4menu that a
>     certain monitor handle has changed
>  3. libxfce4menu looks up the XfceMenu and filename for this handle and
>     asks the XfceMenu to update that filename
>  4. the XfceMenu asks the cache to invalidate and reload the
>     XfceMenuItem for this filename

Ah, ok, I didn't quite remember if you'd decided that you actually 
wanted to make it possible to update the XfceMenu 'in place' like that. 
  Cool.

>> Also, I get a bunch of these messages every time the menu changes
>> (might be happening when I unref the XfceMenu too):
>>
>> (xfdesktop:14288): thunar-vfs-CRITICAL **: thunar_vfs_monitor_remove:
>> assertion `g_list_find (monitor->handles, handle) != NULL' failed
>>
>> It doesn't happen for every call to _remove_monitor(), though, just
>> some of them.
>>
>> I've committed the code to xfdesktop trunk (with some
>> potentially-useful debug output with --enabke-debug); maybe you could
>> take a look?
> 
> Yeah, I'll debug everything and hopefully find out what's wrong. 

Great, thanks!

>> P.S. Can you bump libxfce4menu's package version so xfdesktop can
>> depend on the new version?
> 
> Do you mean libxfce4menu_verinfo? The pkg-config module version is
> updated with every commit.

Well, I guess verinfo should be updated too at some point (not really 
necessary pre-release, I guess).  But I was talking just about what 
pkg-config's --modversion reports.  Didn't realise you have the svn rev 
encoded in there, so I can just use that.

	-brian



More information about the Xfce4-dev mailing list