No problem, any extra pair of eyes is always appreciated, thanks!

On 10/02/2012 04:23 PM, Eduardo Bacchi Kienetz wrote:
I have only checked for sanity, not actually run any of it,
unfortunately :)

On 10/02/2012 03:02 PM, Chris Evich wrote:
@@ -66,17 +67,16 @@ class TempXMLFile(file):
param: mode: file access mode
param: buffer: size of buffer in bytes, 1: line buffered

param: buffsz ^


Thanks, I'll look into it.

@@ -177,6 +186,7 @@ class XMLTreeFile(ElementTree.ElementTree,
XMLBackup):
# Test if xml is a valid filename
self.sourcebackupfile = open(xml, "rb")

Kind of wondering why we're using open here ^ and file elsewhere. Why
not using open() everywhere? Am I missing something? :)


Nope, you're spot on.  I'll fix it up to be consistent.  Thanks!

--
Chris Evich, RHCA, RHCE, RHCDS, RHCSS
Quality Assurance Engineer
e-mail: cevich + `@' + redhat.com o: 1-888-RED-HAT1 x44214

_______________________________________________
Autotest-kernel mailing list
[email protected]
https://www.redhat.com/mailman/listinfo/autotest-kernel

Reply via email to