The previous code processed the input file line by line, but I think
it looks a little more straight forward to just process the whole file
at once.
This patch also explicitly closes the file after reading its contents.
Signed-off-by: Russell Bryant <russell@ovn.org>
Acked-by: Justin Pettit <jpettit@ovn.org>
def manpage_to_nroff(xml_file, subst, version=None):
- f = open(xml_file)
- content = []
- for line in f:
- for k, v in subst.iteritems():
- line = line.replace(k, v)
- content += [line]
- doc = xml.dom.minidom.parseString(''.join(content)).documentElement
+ with open(xml_file) as f:
+ content = f.read()
+ for k, v in subst.iteritems():
+ content = content.replace(k, v)
+ doc = xml.dom.minidom.parseString(content).documentElement
if version is None:
version = "UNKNOWN"