[PATCH] Proper error message when specifying empty commit message

uws uws at xs4all.nl
Fri Sep 9 00:03:51 BST 2005


På Wed, Sep 07, 2005 at 06:46:52PM +0200, Tristan Seligmann skrev:
> * Nir Soffer <nirs at actcom.net.il> [2005-09-07 15:42:47 +0300]:
> 
> > On 7 Sep, 2005, at 14:55, Robert Collins wrote:
> > 
> > >I agree with Michaels suggestion of len(msg.strip()) == 0 as the test.
> > 
> > A more readable test is msg.isspace()
> 
> Not quite; "".isspace() returns False

What about this:

  if not msg or msg.isspace():
      throw...

  mvrgr, Wouter

-- 
:wq                                                       mail uws at xs4all.nl
                                                      web http://uwstopia.nl

no more kill :: no more kill i'm in love      -- black rebel motorcycle club
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
Url : https://lists.ubuntu.com/archives/bazaar/attachments/20050909/191d3650/attachment.pgp 


More information about the bazaar mailing list