Comment 1 for bug 325935

Revision history for this message
Chris S. (cszikszoy) wrote :

The problem here is that the manlookup function has a _very_ hungry method for SupportsItem.

When you type man in the first pane, do has to run this function on every type of SupportedItem in the universe, which in this case is an ApplicationItem, ManualPageItem, or TextItem.

I think the logical thing to do here is move this function and don't check every supported item in the universe right away. This method could be moved to Perform, so that when the man page action runs, it will have to check if that application has a supported manual.

Worst case scenario: the plugin will try to find a man page for an application that doesn't have one. Here we could just politely inform the user that the is no man page for that application.

I'll mess around with it a little later.