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