TextSearch should open files on specific line

[KapiX]

Currently it only does this when "Open in Pe" is enabled.

Three solutions:

• Let the user change the editor.
• Open files with /bin/open.
• Add be:line and be:column to B_REFS_RECEIVED message like /bin/open does.

IMO "Open in Pe" should be removed completely. It depends on an application that doesn't have to be in the system, and sometimes doesn't even work because of missing symbols.

[humdinger]

Doing it like "open", i.e. launching with the preferred app, won't cut it. HTML files would be opened in the browser, which probably isn't what we want. An idea would be listing all apps that handle "text/*" MIME typed files in a submenu under "Settings | Open files in >".

[axeld]

I like humdingers suggestion; be:line and be:column should be set already, or I wouldn't know how the Pe feature is supposed to work. I think StyledEdit should support this, too, so it could actually be the default.

[KapiX]

@humdinger: That rules out option 2. Still, if "Open in Pe" is disabled the file is opened in preferred app. Adding the attributes shouldn't hurt. At worst, the app will ignore them. ​http://cgit.haiku-os.org/haiku/tree/src/apps/text_search/GrepWindow.cpp#n1180

@axeld: It uses Pe-specific parameter. ​http://cgit.haiku-os.org/haiku/tree/src/apps/text_search/GrepWindow.cpp#n1511 EDIT: ... and Pe-specific message code.

[phoudoin]

I will go to the solution 3:

• launch preferred app for the file
• add be:line to a B_REFS_RECEIVED message handed to the tracker, which will open the preferred app for that file; as usual with tracker.
• add a context menu to allow to open with another app, as possible with tracker.
• and remove the "Open in Pe"

No be:column yet, as we don't have such information yet. And be:line seems to be supported by Pe too, BTW.

[phoudoin]

Part one done in hrev51529.