------------------------------------------------------------------------------
To reply, visit https://hellosplat.com/s/beanbag/tickets/4371/
------------------------------------------------------------------------------
New update by smacleod
For Beanbag, Inc. > Review Board > Ticket #4371
Reply:
The problem is within algorithm of detecting a group move.
From opcode_generator.py#~350
# The way we do that is to find each removed line that matches
# this inserted line, and for each of those find out if there's
# an existing move range that the found removed line
# immediately follows. If there is, we update the existing
# range.
If there is a MoveRange which starts with the same (2+) lines of code last
choice is prefered.
I think that the solution could work along the following lines:
* create a temp MoveRange
* continue adding to it until the following lines are the same
* if the range is longer than existing one replace them entirely.
--
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.
