Code review comment for lp:~roadmr/checkbox/588539

Revision history for this message
Marc Tardif (cr3) wrote :

First, you might like to use the default parameter when declaring the timeout property in the BackendInfo class rather than create an additional entry in examples/checkbox.ini. Also, I think you want to declare it as an Int rather than a String property variable.

Second, you forgot a pdb statement in the stop message handler.

review: Needs Fixing

« Back to merge proposal