Lars Gullik BjÃnnes wrote:

> Sorry for not noticing this ealier, but I think (if possible) they
> should be free functions and not class functions.

No problem.

> 
> Then they would look like this:
> 
> bool operator==(PosIterator const & lhs,
>                 PosIterator const & rhs)
> {
>         PosIteratorItem const & li = lhs.stack_.top();
>         PosIteratorItem const & ri = rhs.stack_.top();
> 
>         return li.pl == ri.pl && li.pit == ri.pit &&
>                (li.pit == li.pl->end() || li.pos == ri.pos);
> }
> 
> 
> bool operator!=(PosIterator const & lhs,
>                 PosIterator const & rhs)
> {
>         return !(lhs == rhs);
> }


okey dokey.

> And yes, I found this better.

Super, I'd commit then if for you it's ok.

Alfredo


Reply via email to