Rick, your patch looks almost correct to me, it's certainly the right start.
Do you want to make a branch and propose it for merging?
Looking at the diff there's the same bug in the other implementation of __eq__
that was added. Also, there should be a couple of doctests added comparing
with None, but I think an LP dev will add tests for you if you propose the
change.
On Fri Dec 11 05:14:03 UTC 2009 Rick Spencer wrote:
> Taking the liberty of setting to confirmed, as seb128 also experienced
> this issue
Unsurprisingly
http:// bazaar. launchpad. net/~lazr- developers/ lazr.restfulcli ent/trunk/ revision/ 79
looks to be the cause.
Rick, your patch looks almost correct to me, it's certainly the right start.
Do you want to make a branch and propose it for merging?
Looking at the diff there's the same bug in the other implementation of __eq__
that was added. Also, there should be a couple of doctests added comparing
with None, but I think an LP dev will add tests for you if you propose the
change.
Thanks,
James