[PATCH] bookmarks: do nothing if commit was not successful

Benoit Boissinot bboissin at gmail.com
Thu Nov 6 07:15:48 CST 2008


On Wed, Nov 5, 2008 at 10:13 PM, Dennis Brakhane
<brakhane at googlemail.com> wrote:
> On Sun, Oct 26, 2008 at 05:10:44PM +0800, Dmitriy Taychenachev wrote:
>> +            if node == None:
>> +                return None
>
> Shouldn't that be "if node is None"?

Both forms are correct, I find the later one prettier :)

regards,

Benoit


More information about the Mercurial-devel mailing list