On 2015-08-22 21:57+0100 Phil Rosenberg wrote:

> Hi All
>
> I have just pushed a commit with a change to the documentation which I
> believe reflects the things we have said. The changes are to the
> plspage and plsetchr functions in the API section.
>
> If anyone thinks further changes are needed then that's fine, but I
> thought it would be good if any changes we were making based on this
> discussion could be based on the actual documentation, rather than on
> emails that may or may not get properly documented.

Thanks for making these changes.

I have only looked at diffs so far, but so far I have no issues with
how you have restated things.

Also, "make validate" shows there are no issues from the DocBook
perspective.  Nevertheless, the

git diff d5ee62bb^..d5ee62b

command did show one strange issue right at the top:

diff --git a/doc/docbook/src/api.xml b/doc/docbook/src/api.xml
index 956fc15..706aece 100644
--- a/doc/docbook/src/api.xml
+++ b/doc/docbook/src/api.xml
@@ -1,4 +1,4 @@
-<!-- -*-  mode: nxml -*- -->
+<U+FEFF><!-- -*-  mode: nxml -*- -->

I don't think it should be necessary to insert that weird Unicode
character into api.xml.  If you agree, could you please remove it?

Alan
__________________________
Alan W. Irwin

Astronomical research affiliation with Department of Physics and Astronomy,
University of Victoria (astrowww.phys.uvic.ca).

Programming affiliations with the FreeEOS equation-of-state
implementation for stellar interiors (freeeos.sf.net); the Time
Ephemerides project (timeephem.sf.net); PLplot scientific plotting
software package (plplot.sf.net); the libLASi project
(unifont.org/lasi); the Loads of Linux Links project (loll.sf.net);
and the Linux Brochure Project (lbproject.sf.net).
__________________________

Linux-powered Science
__________________________

------------------------------------------------------------------------------
_______________________________________________
Plplot-devel mailing list
Plplot-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/plplot-devel

Reply via email to