On 10/18/2017 11:28 AM, Ken Brown wrote:
> Similar considerations apply to the other public member functions of 
> SolvableVersion.  So my inclination is to go with something like my 
> patch...

...with perhaps one tweak.  Maybe we should test 'id' rather than 
'pool', since id being 0 is what's used elsewhere to characterize the 
empty package.  And if id != 0 but pool == 0, there's a bug that we want 
to catch.

Revised patch attached.

Ken