65 + if (kw.has_key('timeout')): 66 + timeout = kw['timeout'] 67 + else: 68 + timeout = self.proc.timeout 69 + 70 + if len(args) == 1: 71 + logging.debug("expect (%d): '%s'" %(timeout, args[0])) 72 + else: 73 + logging.debug("expect (%d): '%s'" %(timeout, str(args)))
We should combine these two blocks, no need to have this ambiguous check for len(args) == 1 if we just checked whether timeout was specified.
Also, there is a small merge conflict to fix. Otherwise, still looks fine.
« Back to merge proposal
65 + if (kw.has_ key('timeout' )): debug(" expect (%d): '%s'" %(timeout, args[0])) debug(" expect (%d): '%s'" %(timeout, str(args)))
66 + timeout = kw['timeout']
67 + else:
68 + timeout = self.proc.timeout
69 +
70 + if len(args) == 1:
71 + logging.
72 + else:
73 + logging.
We should combine these two blocks, no need to have this ambiguous check for len(args) == 1 if we just checked whether timeout was specified.
Also, there is a small merge conflict to fix. Otherwise, still looks fine.