ok, done.
A/
On Tue, Oct 14, 2014 at 5:33 PM, Richard Heck wrote:
> On 10/14/2014 11:17 AM, Alfredo Braunstein wrote:
>>
>> On Tue, Oct 14, 2014 at 8:51 AM, Alfredo Braunstein
>> wrote:
>>>
>>> On Mon, Oct 13, 2014 at 6:27 PM, Jean-Marc Lasgouttes
>>> wrote:
Le 13/10/2014 18:15, Alfr
On 10/14/2014 11:17 AM, Alfredo Braunstein wrote:
On Tue, Oct 14, 2014 at 8:51 AM, Alfredo Braunstein wrote:
On Mon, Oct 13, 2014 at 6:27 PM, Jean-Marc Lasgouttes
wrote:
Le 13/10/2014 18:15, Alfredo Braunstein a écrit :
The fix however seems bogus to me:
-cur.forwardPos()
On Tue, Oct 14, 2014 at 8:51 AM, Alfredo Braunstein wrote:
> On Mon, Oct 13, 2014 at 6:27 PM, Jean-Marc Lasgouttes
> wrote:
>> Le 13/10/2014 18:15, Alfredo Braunstein a écrit :
>
> The fix however seems bogus to me:
> -cur.forwardPos();
> +
On Mon, Oct 13, 2014 at 6:27 PM, Jean-Marc Lasgouttes
wrote:
> Le 13/10/2014 18:15, Alfredo Braunstein a écrit :
The fix however seems bogus to me:
>>>
>>>
>>>
-cur.forwardPos();
+cur.top().forwardPos();
>>>
>>>
>>>
>>> Why bogus? Why do you want
Le 13/10/2014 18:15, Alfredo Braunstein a écrit :
The fix however seems bogus to me:
-cur.forwardPos();
+cur.top().forwardPos();
Why bogus? Why do you want to revert it?
To me it seems wrong (for the reasons explained below). But maybe I
don't understand i
On Mon, Oct 13, 2014 at 5:39 PM, Jean-Marc Lasgouttes
wrote:
> Le 13/10/2014 14:29, Alfredo Braunstein a écrit :
>>
>> The fix however seems bogus to me:
>
>
>> -cur.forwardPos();
>> +cur.top().forwardPos();
>
>
> Why bogus? Why do you want to revert it?
To me it s
Le 13/10/2014 14:29, Alfredo Braunstein a écrit :
The fix however seems bogus to me:
-cur.forwardPos();
+cur.top().forwardPos();
Why bogus? Why do you want to revert it?
If you look at the top of the chunk, there is another
forwardPos/backwardPos that could
The problem was indeed correctly identified by JM in changeset 7c3d1d7:
"The problem is the use of cursor movement methods to update cursor.
Cursor::forwardPos() steps into insets, which is not always what we
want. The problem here is that there is a math inset just after the
accepted change, and
