https://bugs.freedesktop.org/show_bug.cgi?id=80715

Matthew Francis <fdb...@neosheffield.co.uk> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |mst...@redhat.com

--- Comment #15 from Matthew Francis <fdb...@neosheffield.co.uk> ---
Within the above bibisect range, possibly this commit.

Adding Cc: to mst...@redhat.com. Could you possibly take a look at this? Thanks
(To recap, before the perf regression of this bug, replacing the 16,000 $ in
the attached file with " " takes a few seconds. After, it takes ~forever)


commit b60ce8465c8f01242354abccebe00742d164af60
Author: Michael Stahl <mst...@redhat.com>
Date:   Sat Oct 12 01:54:37 2013 +0200

    sw: fix regex search/replace of $ in selection

    SwView::Replace() nowadays first searches for the to-be-replaced text in
    the current selection, which does not work in the corner case of "$"
    when the selection starts at end of a paragraph, as it will when
    clicking "Find" to look for "$" and then clicking "Replace".

    The problem is that there is some antique code in SwPaM::Find() to
    move the cursor forward if it's at the end of a paragraph, which is
    presumably some pointless micro-optimization; the result is that
    "Replace" does not replace the selected paragraph break but the
    one in the following paragraph.

    (regression from 68a014dbca8bbd25056f75ef551fb81a0dbfb1b7)

    Change-Id: I5aae9c272d102a48166c63e01775dc6322f9f02d

-- 
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

Reply via email to