On 11/24/10 11:59 AM, John Fischer wrote:
Ginnie,

The code looks syntactically correct. But will it write the output
to the logger in the case when the data does not contain
x0a or x0d?

86 tmp = str(self.data)
87 # Once the output contains a line feed or
88 # carriage return, strip it and log it
89 if '\x0a' in tmp or '\x0d' in tmp:
90 tmp = tmp.rstrip('\x0a\x0d')
91 self.trans_logger.debug('%s' % (tmp))
92 self.data = ''

I'd also ask why the hexadecimal encoded characters are being used here instead of "\r" and "\n".

And why it's checking the string using "in" when it's only interested in trailing newlines or carriage returns.

This code also makes the assumption that output will never include unicode.

-Shawn
_______________________________________________
caiman-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/caiman-discuss

Reply via email to