tree 11fbd868ebd9d6f3db0d7706ae6940bd893c5ac3
parent 61bc2485555c5001b02debad0909000a8e2faf3b
author David Lawrence Ramsey <pooka109@gmail.com> 1483822309 -0600
committer Benno Schulenberg <bensberg@justemail.net> 1483892308 +0100

weeding: remove another unnecessary setting of openfile->current_y

The setting of current_y in copy_from_filestruct() also appears to be
a holdover from the days of a more-common STATIONARY scrolling mode.

do_cut_text() uses the above function when copying text (uncutting
text again right after cutting it).  Since the text is effectively
the same afterward, current_y doesn't need to change.

do_uncut_text(), however, does need current_y up to date in one case:
when uncutting a full screen or less' worth of lines, focusing will be
FALSE, and it uses edit_refresh(), so it will use STATIONARY scrolling
mode then.  Take a cue from do_insertfile() and call reset_cursor() to
get an updated current_y.

(Note that the check for a full screen or less' worth of lines uses
incorrect values when in softwrap mode, but that's a separate problem.)

undo_cut(), do_redo(), and backup_lines() do not need set current_y
because they all result in edit_refresh() with focusing = TRUE, so
they do a CENTERING scroll which does not need current_y.