> Angus, did you check in all places that the values for positions, etc
> are always positive? Currently LyXParagraph::size_type is signed, but
> this will probably change later. This has the potential for creating
> problems: in particular when comparing signed and unsigned values.
>
> So, I'd rather be sure you doublechecked it before applying the patch
> (in fact this was the reason why I did not make the changes myself :).

No! Will do.

Incidentally, should I use LyXParagraph::size_type or should I really use 
size_t?

Angus

Reply via email to