HI Mike,
> 
>   m_iInsPoint -= countChars;
>   if (m_iInsPoint < posBOD)
>   {
>    m_iInsPoint = posBOD;
>   }
> 
> inside FV_View::_charMotion takes care of it. Note that I will not commit
> this one since I'm not 100% sure that some other code might depend upon the
> current behaviour. The fix seems reasonable though.
> 
        I think you're right about posBOD and posEOD. I put the extra
checks in the earlier code to handle fields. They seemed to work.

I cannot imagine any reason why m_iInsPoint would want to be before
posBOD. I think committing it couldn't hurt.

Go ahead.

Martin





Reply via email to