[vsipl++] [patch] Document div, mul, and sub elementwise functions

Stefan Seefeld stefan at codesourcery.com
Wed Aug 6 13:36:28 UTC 2008


Jules Bergmann wrote:
> This patch documents the div,mul, and sub elementwise functions, along 
> the same lines as the add function.
> 
> It also removes the arguments from the man page title ('Add' instead of 
> 'Add(A, B)') and adds a section on operator syntax (stating that 'add(A, 
> B)' is equivalent to 'A + B').
> 
> I've put a generated manual here: ~jules/tmp/manual.pdf
> 
> Ok to apply?

This looks good (literally so ! :-) ). I have a couple of (small) 
issues, though:

> +<section id="functionref_div">
> + <title><literal>div</literal></title>

What is the reason you use the 'literal' element in the above ? Is it to 
force a specific output style ? If you really want to mark this up 
specially, I'd suggest any of the more expressive markups, such as 
'code'. (The same issue came up in a patch from Mike.)


> +  <title>Elementwise Functions</title>
>    <xi:include xmlns:xi="http://www.w3.org/2001/XInclude" href="add.xml" />
> +  <xi:include xmlns:xi="http://www.w3.org/2001/XInclude" href="div.xml" />
> +  <xi:include xmlns:xi="http://www.w3.org/2001/XInclude" href="mul.xml" />
> +  <xi:include xmlns:xi="http://www.w3.org/2001/XInclude" href="sub.xml" />

I would suggest we put the namespace declaration (the 'xmlns:xi' 
attribute) into the root element of each xml file, so we don't need to 
repeat it each time we use xi:include.

Finally, I noticed in your pdf manual the same thing Mike complained 
about in his output: the formatting of the function synopsis is wrong, 
as the function parameter types are declared outside the function 
prototype. As I mentioned in a reply to Mike, the fix for this is to 
customize the html and fo - generating stylesheets so ansi-style 
formatting is used, not k&r.
(I can submit a fix for this in a separate patch, as it affects the 
csl-docbook module only.)


Thanks,
		Stefan

-- 
Stefan Seefeld
CodeSourcery
stefan at codesourcery.com
(650) 331-3385 x718



More information about the vsipl++ mailing list