Ihor Radchenko writes: > No Wayman writes: > >> The attached patch satisfies all but two checkdoc warnings. >> I left those two for someone more familiar with the package. > > Thanks! > >> Subject: [PATCH] * lisp/ob-lilypond.el: satisfy checkdoc > > _S_atisfy. > >> (org-babel-lilypond-compile-post-tangle): fix docstring >> (org-babel-lilypond-display-pdf-post-tangle): fix docstring >> (org-babel-lilypond-play-midi-post-tangle): fix docstring >> (org-babel-execute:lilypond): fix docstring >> (org-babel-lilypond-tangle): fix docstring >> (org-babel-lilypond-process-basic): fix docstring >> (org-babel-lilypond-parse-line-num): fix docstring >> (org-babel-lilypond-set-header-args): fix docstring > > It is enough to say "Fix docstring" just for the last line and > leave all > other : ... empty. Fixed in attached patch. >> (defun org-babel-lilypond-set-header-args (mode) >> - "Set org-babel-default-header-args:lilypond >> -dependent on ORG-BABEL-LILYPOND-ARRANGE-MODE." >> + "Set lilypond babel header args from >> ORG-BABEL-LILYPOND-ARRANGE-MODE." > > What is ORG-BABEL-LILYPOND-ARRANGE-MODE? Looks like it's a variable that would be better served by a minor-mode. #+begin_src emacs-lisp (defvar org-babel-lilypond-arrange-mode nil "Non-nil turns Arrange mode on. In Arrange mode the following settings are altered from default: :tangle yes, :noweb yes :results silent :comments yes. In addition lilypond block execution causes tangling of all lilypond blocks.") #+end_src A minor mode would also obviate the need for `org-babel-lilypond-toggle-arrange-mode'. Let's do that in a separate patch.