------------------------------------------------------------------------------
To reply, visit https://hellosplat.com/s/beanbag/tickets/4371/
------------------------------------------------------------------------------
New update by smacleod
For Beanbag, Inc. > Review Board > Ticket #4371
Reply:
I have a patch that still needs some additional testing, but if you could
give it a try and see if it works for any cases you've hit (and standard cases
that appeared to work fine), that would really help :)
I did test it with the review request in the description above, and it
seems to produce the right results (I haven't inspected every character of move
range but I did look for the obvious problems that were previously hit).
The attached patch contains the test data you supplied as well. However, I
noticed that one of the lines in the test case you supplied was still pointing
to an incorrect move line (`33 -> 2671` should have been `33 -> 2615`). I've
fixed that in the attached diff.
Files:
- 0001-Fix-reuse-of-previously-used-move-detection-ranges.patch
<https://hellosplat.com/s/beanbag/attachments/1224/>
--
You received this message because you are subscribed to the Google Groups
"reviewboard-issues" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To post to this group, send email to [email protected].
Visit this group at https://groups.google.com/group/reviewboard-issues.
For more options, visit https://groups.google.com/d/optout.
