Alfredo Braunstein <[EMAIL PROTECTED]> writes: | Lars Gullik Bjønnes wrote: > >> yes + some comment. >> >> | +using boost::prior; >> >> which header file include next_prior.hpp? > | Couldn't find out. Should I add it just in case?
yeah. >> I wonder if there is a way, so that we don't need to have those >> XXX_iterator_(begin|end) functions. >> >> but ok for now. > | what about a pos_iterator_begin(Buffer &)? in PosIterator.[Ch] I don't know, let's leave it for the time being. >> | +PosIterator::PosIterator(ParIterator & parit, lyx::pos_type pos) >> | +{ >> >> Why have this here? and not int he positerator.C file? > | Because I need ParPosition which is not defined in the header file. Should I | put it on the header? forward delcare... >> why do they have to be friends? > | Because I need private members of PosIterator in the | PosIterator(Pariterator, pos) constructor. What should I be doing instead? Unprivate the members? friend the whole PosIterator class so that forward delcaration will work? -- Lgb