On Wed, Sep 16, 2009 at 04:18:23PM +0100, Michael Hanselmann wrote:
> 2009/9/16 Iustin Pop <[email protected]>:
> > On Wed, Sep 16, 2009 at 03:55:45PM +0200, Michael Hanselmann wrote:
> >> --- a/lib/cmdlib.py
> >> +++ b/lib/cmdlib.py
> >> -      if result.failed or result.fail_msg:
> >> +      if result.RemoteFailMsg():
> >
> > why not leave it like result.fail_msg?
> 
> I checked other code in cmdlib.py and some of it uses fail_msg
> directly, other uses RemoteFailMsg(), and I didn't change it back.
> RemoteFailMsg() just returns fail_msg. Can it go away, too?

Sure - and thanks :)

iustin

Reply via email to