John Kitchin <jkitc...@andrew.cmu.edu> writes: > I am not sure what you mean for this. Let me know if it isn't fixed in > the attached patches. I thought I had squashed everything into a concise > history.
No worries. Let's just apply the 21 patches. > I think this code below (which should be in the patches) handles the > option correctly. > > (defun org--open-file-link (path app) It should, but I didn't see it in the previous patch, hence my remark. > -(defvar org-store-link-functions nil > - "List of functions that are called to create and store a link. > Each function will be called in turn until one returns a non-nil > -value. Each function should check if it is responsible for creating > -this link (for example by looking at the major mode). > -If not, it must exit and return nil. > -If yes, it should return a non-nil value after a calling > -`org-store-link-props' with a list of properties and values. > -Special properties are: > +value. Each function should check if it is responsible for > +creating this link (for example by looking at the major mode). If > +not, it must exit and return nil. If yes, it should return a > +non-nil value after a calling `org-store-link-props' with a list > +of properties and values. Special properties are: Missing a space after the full stop above. > +(defun org--open-file-link (path app) > + "Open PATH using APP. > + > +PATH is from a file link, and can have the following syntax: > + [[file:~/code/main.c::255]] > + [[file:~/xx.org::My Target]] > + [[file:~/xx.org::*My Target]] > + [[file:~/xx.org::#my-custom-id]] > + [[file:~/xx.org::/regexp/]] > + > +APP is '(4) to open the PATH in Emacs, or 'system to use a system > application." Maybe something like: Called it with \\[universal-argument] to open PATH in Emacs. If ARG is `system', use a system application instead. Regards,
