On Tue, Feb 12, 2019 at 9:41 AM Nicolas Goaziou <m...@nicolasgoaziou.fr> wrote: > > [...] > > Thank you! Some comments follow. > > > -`eval', which see.") > > +`eval', which see. And it is used as the value for > > +`lexical-binding' in buffers created by `org-edit-src-code'.") > > You need to add two spaces after full stops. > > > +(defun org-babel-emacs-lisp-lexical (lexical) > > + "Convert :lexical source block argument LEXICAL into the form > > +appropriate for `lexical-binding' and the LEXICAL argument to > > +`eval'." > > The first sentence in a docstring needs to fit on a single line. > > Could you add a test or two for that feature? Could you also add an > ORG-NEWS entry?
I did all the changes you suggested. I will send the new patch after I managed to setup and get acquainted with mail in Emacs (mu4e).
