Merge lp:~mvngu/igraph/doc into lp:~igraph/igraph/0.6-main-old
Status: | Merged |
---|---|
Merged at revision: | 2647 |
Proposed branch: | lp:~mvngu/igraph/doc |
Merge into: | lp:~igraph/igraph/0.6-main-old |
Diff against target: |
25 lines (+2/-2) 2 files modified
src/structural_properties.c (+1/-1) src/type_indexededgelist.c (+1/-1) |
To merge this branch: | bzr merge lp:~mvngu/igraph/doc |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Gábor Csárdi | Approve | ||
Review via email: mp+95828@code.launchpad.net |
Description of the change
Here is a description of the first patch. With trunk #2646, building the HTML version of the documentation resulted in the following error:
if [ "x.." != "x.." ]; then cp ../doc/fdl.xml ../doc/gpl.xml ../doc/
xmlto -x ../doc/gtk-doc.xsl -o html xhtml igraph-docs.xml \
&& touch html/stamp
xmlto: /home/mvngu/
xmlto: Fix document syntax or use --skip-validation option
structural.
structural.
structural.
structural.
Document /home/mvngu/
make: *** [html/stamp] Error 13
This is due to using the HTML tag <em></em> in the documentation of the function igraph_linegraph(). A fix is to use doxygen's syntax for emphasizing words, i.e. the tag \em in doxygen.
Here is a description of the second patch. In the function igraph_get_eid(), the documentation for the parameter error is misleading. The current documentation for error implies that if error is set to false and the required edge is not found, then the function will return -1. This is clearly not the behaviour of the function. If the boolean error is set to false and the required edge is not found, then the parameter eid will be assigned value -1. My second patch makes this change in documentation.
All clean.