Hi John, thanks for the patch -- some comments below.
John Hendy <jw.he...@gmail.com> writes: > Header arguments: clarify that `:exports' is only applicable to > blocks, not inline code This first line should include "org.texi:" to tell that the change happens in org.texi. See other commits for examples. > * doc/org.texi (Exporting code blocks): add clarification; relevant for > blocks ony. Start with a capitalized letter: "Add clarification." Don't use semi-column. "Clarify" is enough IMO. > * doc/org.texi (exports): add clarification; relevant for blocks only. When two changes get the same description use this: * doc/org.texi (Exporting code blocks, exports): Clarify. In general, you can make it easier for you by reviewing the patch (hit `C-x v =' in the modified org.texi buffer), and then hit `C-x 4 a' on each change to create a Changelog buffer that you can safely edit. > The documentation change resulted from the expectation that > `:exports' could be used with inline code blocks, which is not the > case. Clarification was added to avoid future confusion on this > Babel block specific header argument. Please use auto-fill and fill-column to something between 72 and 80. > TINYCHANGE Thanks again for the patch, HTH, -- Bastien