Ihor Radchenko <yanta...@posteo.net> writes:

> Jack Kamm <jackk...@gmail.com> writes:
>
>> The following patch seems to fix the issue -- does something like this
>> seem reasonable?
>>
>> ...
>> +(defun org-babel-previous-src-block-or-inline ()
>> +  "Jump to previous source block or inline block."
>> +  (re-search-backward (rx (or (regexp org-babel-src-block-regexp)
>> +                              ;; copied from 
>> `org-element-inline-src-block-parser'.
>> +                              (regexp "\\_<src_\\([^ \t\n[{]+\\)[{[]")))))
>
> Yes, in general. But,
> 1. Probably better make it internal function for more flexibility in
>    future
> 2. You need to consult `org-element-context' to make sure that we are
>    really at inline src block, but not inside some kind of verbatim
>    markup.
> 3. Rather than copying regexps around, please factor it out and create a
>    constant in org-element.el

Thanks very much for the feedback -- I will work on updating the patch
to incorporate your suggestions as well as other improvements (such as
unit tests).

Reply via email to