Opened 10 years ago

Closed 9 years ago

#3634 closed enhancement (invalid)

Worker thread to fill Mount menu

Reported by: jonas.kirilla Owned by: axeld
Priority: normal Milestone: R1
Component: Applications/Deskbar Version: R1/pre-alpha1
Keywords: Cc:
Blocked By: Blocking:
Has a Patch: no Platform: All

Description

I had the impression the Mount menu was slow to open - it used to be - so I created this worker thread thing. See patch. But now it seems I don't have any prolonged pause before the Mount menu opens, so perhaps this is unnecessary.

The code may be shite anyway, and it would only help Deskbar's Mount menu and not that of Tracker. Put here for safe keeping. (Better here than on the mailinglist?)

Comments?

Attachments (1)

BeMenu.cpp.diff (1017 bytes) - added by jonas.kirilla 10 years ago.
BeMenu.cpp.diff

Download all attachments as: .zip

Change History (3)

Changed 10 years ago by jonas.kirilla

Attachment: BeMenu.cpp.diff added

BeMenu.cpp.diff

comment:1 Changed 10 years ago by axeld

That used to be a problem in BeOS, it really shouldn't be one in Haiku, as its device API works differently, and always knows the current status quo.

In any case, the patch attached seems to have some locking issues (ie. it doesn't lock anything, but the menu structures are read/changed from different threads).

comment:2 Changed 9 years ago by jackburton

Resolution: invalid
Status: newclosed

Not necessary in Haiku.

Note: See TracTickets for help on using tickets.