• David Lawrence Ramsey's avatar
    weeding: remove unnecessary settings of openfile->current_y · b1c20629
    David Lawrence Ramsey authored
    Many of the adjustments of the value of openfile->current_y appear to be
    a holdover from the days when certain functions had to account for what
    is now called STATIONARY scrolling mode, which depends on the value of
    current_y.  Remove these adjustement where they are superfluous.
    
    do_para_begin(), do_para_end(), and do_bracket_match() update the screen
    through edit_redraw(), which uses either CENTERING or FLOWING scrolling
    mode, so their setting of current_y is redundant and useless, as it will
    be ignored and then overridden by the next call to reset_cursor().
    
    findnextstr() is called by go_looking() [which calls edit_redraw(), see
    above], and by do_replace_loop() and do_int_spell_fix(), which both call
    edit_refresh(), which in this case only uses CENTERING scrolling mode
    since focusing is TRUE.
    
    (Additionally, the adjustments of current_y in findnextstr() and
    do_bracket_match() use incorrect values when in softwrap mode.)
    
    find_paragraph() doesn't need to save or restore current_y, because it
    doesn't do any screen updates.  do_justify() calls edit_refresh() with
    focusing set to TRUE, so it uses the CENTERING scrolling mode.
    
    do_alt_speller() and do_formatter() do not need to save and restore
    current_y, because they don't modify it in any way.
    
    This addresses https://savannah.gnu.org/patch/?9197.
    b1c20629
text.c 113 KB