Re: Status fvwm-menu-desktop

2012-01-02 Thread Thomas Adam
On Tue, Dec 20, 2011 at 09:50:54PM -0500, Dan Espen wrote: Thomas Adam tho...@fvwm.org writes: Timings with the patch attached applied are here: http://xteddy.org/~n6tadam/fvwm/nytprof-updated/ That's what I call impressive improvments! I applied the patch, works well for me.

Re: Status fvwm-menu-desktop

2011-12-31 Thread Thomas Adam
On Fri, Dec 30, 2011 at 08:34:41PM -0500, Dan Espen wrote: Thomas Adam tho...@fvwm.org writes: On Fri, Dec 30, 2011 at 07:27:30PM -0500, Dan Espen wrote: If you'd like a challenge, there is a significant usability problem with fvwm-menu-desktop. It only occurs on first use but it's

Re: Status fvwm-menu-desktop

2011-12-30 Thread Dan Espen
Thomas Adam tho...@fvwm.org writes: On Tue, Dec 20, 2011 at 09:50:54PM -0500, Dan Espen wrote: Thomas Adam tho...@fvwm.org writes: Timings with the patch attached applied are here: http://xteddy.org/~n6tadam/fvwm/nytprof-updated/ That's what I call impressive improvments! I

Re: Status fvwm-menu-desktop

2011-12-30 Thread Dan Espen
Thomas Adam tho...@fvwm.org writes: On Fri, Dec 30, 2011 at 07:27:30PM -0500, Dan Espen wrote: If you'd like a challenge, there is a significant usability problem with fvwm-menu-desktop. It only occurs on first use but it's difficult to tell what fvwm is doing when you cursor past or click

Re: Status fvwm-menu-desktop

2011-12-21 Thread Thomas Adam
On Tue, Dec 20, 2011 at 09:50:54PM -0500, Dan Espen wrote: Thomas Adam tho...@fvwm.org writes: Timings with the patch attached applied are here: http://xteddy.org/~n6tadam/fvwm/nytprof-updated/ That's what I call impressive improvments! I applied the patch, works well for me. Cool

Re: Status fvwm-menu-desktop

2011-12-20 Thread Thomas Adam
On Mon, Dec 19, 2011 at 06:11:57PM -0500, Dan Espen wrote: I think TA mentioned it was slow. I don't know if he measured it or not. Yeah: http://xteddy.org/~n6tadam/fvwm/nytprof/ Note interpret_menu() is the culprit -- the scan_AppDir() and read_desktop_entry() subroutines don't help, but I

Re: Status fvwm-menu-desktop

2011-12-20 Thread Dan Espen
Thomas Adam tho...@fvwm.org writes: On Mon, Dec 19, 2011 at 06:11:57PM -0500, Dan Espen wrote: I think TA mentioned it was slow. I don't know if he measured it or not. Yeah: http://xteddy.org/~n6tadam/fvwm/nytprof/ Nice! Note interpret_menu() is the culprit -- the scan_AppDir() and

Re: Status fvwm-menu-desktop

2011-12-20 Thread Dan Espen
Thomas Adam tho...@fvwm.org writes: On Tue, Dec 20, 2011 at 01:57:37PM -0500, Dan Espen wrote: Thomas Adam tho...@fvwm.org writes: On Mon, Dec 19, 2011 at 06:11:57PM -0500, Dan Espen wrote: I think TA mentioned it was slow. I don't know if he measured it or not. Yeah:

Re: Status fvwm-menu-desktop

2011-12-20 Thread Thomas Adam
On Tue, Dec 20, 2011 at 06:14:54PM -0500, Dan Espen wrote: After removing the ones we don't need (above), there's not many calls left. We only deal with a few directories. We could readdir and build a hash. Yes, but the recusiveness of those subroutines before the hashing I added in the patch

Re: Status fvwm-menu-desktop

2011-12-20 Thread Dan Espen
Thomas Adam tho...@fvwm.org writes: Timings with the patch attached applied are here: http://xteddy.org/~n6tadam/fvwm/nytprof-updated/ That's what I call impressive improvments! I applied the patch, works well for me. -- Dan Espen

Status fvwm-menu-desktop

2011-12-19 Thread Dan Espen
We had a recent patch for fvwm-menu-desktop. I never got around to taking a look. Did it get committed? I recently installed Fedora 16 and fvwm-menu-desktop stopped working. It looks like it never finds the .desktop files in /usr/share/applications. I made a quick fix by adding

Re: Status fvwm-menu-desktop

2011-12-19 Thread Thomas Adam
On Mon, Dec 19, 2011 at 04:48:00PM -0500, Dan Espen wrote: We had a recent patch for fvwm-menu-desktop. I never got around to taking a look. Did it get committed? Yes, by me, AFAIK. I'll double-check for you since sometimes commiting to CVS for me can be hit and miss. -- Thomas Adam --

Re: Status fvwm-menu-desktop

2011-12-19 Thread Thomas Funk
Nachricht- Von: Dan Espen des...@verizon.net Gesendet: 19.12.2011 22:48:00 An: fvwm-workers@fvwm.org Betreff: Status fvwm-menu-desktop We had a recent patch for fvwm-menu-desktop. I never got around to taking a look. Did it get committed? I recently installed Fedora 16 and fvwm-menu-desktop

Re: Status fvwm-menu-desktop

2011-12-19 Thread Thomas Adam
On Mon, Dec 19, 2011 at 11:11:48PM +0100, Thomas Funk wrote: If anyone can help me to get an idea how I can implement this merging in a good an nice way shout. Please do not conflate a missing XDG path with your own attempts at reimplementing f-m-d. The two are completely distinct, and if you