On Tue, Mar 20, 2018 at 06:48:43AM +0000, Radostin Stoyanov wrote:
Changes since v1
(
https://www.redhat.com/archives/libvir-list/2018-March/msg01001.html)
01 - Fixed typo.
03 - Added whitespace after comma in: value = value +
re.sub("^(\d+)U$","\\1", token[1])
05 and 06 - Use printf style format and break strings everywhere there is a newline.
07 - Use try-except instead of an explicit check.
10 - Use isalnum() instead of a regex match, and a few more simplifications were added.
20 - Use try-except instead of an explicit check.
23 - Don't slice `line` in the loop condition.
New patches: 9, 24, 25
The patches have been tested with `make check`, `make syntax-check`.
The output of apibuild.py was tested with `cd docs/` followed by:
srcdir=. builddir=. /usr/bin/python2 ./apibuild.py; sha1sum *.xml | sha1sum
and
srcdir=. builddir=. /usr/bin/python3 ./apibuild.py; sha1sum *.xml | sha1sum
before and after the patches were applied.
The apibuild.py script was borrowed from libxml2 IIRC. I don't know what
the python3-compatibility status is over there, but it could benefit
from some of the cleanups if you're feeling adventurous ;)
Jan