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

Reply via email to