"Manuel A. Fernandez Montecelo" <[email protected]> writes:

> In practice the %-operation would probably achieve the same, but I think
> that this way is easier to understand.

That's fair, and I agree that it should be equivalent in practice.
As a matter of style, though, I might declare comp_mask as

    static const int comp_mask;

> Patch attached.  I will try to get it into the next release.

Sounds good.

> Aaron, it would be great if you could test it and check that it fixes
> the problem.  I never stumbled upon this problem, even if I use
> multi-arch in most of my systems.

I'm not sure what specifically about my installation triggered this
behavior.  FWIW, when aptitude 0.8.5-1 hit testing three weeks ago, I
decided to try switching back to a stock build, and haven't seen any
trouble on this front since.  However, it's probably still best to
defend against the possibility, so please do proceed here.

Thanks!

-- 
Aaron M. Ucko, KB1CJC (amu at alum.mit.edu, ucko at debian.org)
http://www.mit.edu/~amu/ | http://stuff.mit.edu/cgi/finger/[email protected]

Reply via email to