Description: Undo doesn't restore image anchor position Steps to Reproduce: 1. Open the attached file 2. Place cursor after the yellow marked text (so after "current") 3. Press and hold backspace. Say until 'setup' in line above 4. Press and hold undo Actual Results: Image has gone up & anchor is at 'setup' after undo Expected Results: Should be back to area with yellow marking Reproducible: Always User Profile Reset: No Additional Info: Version: 7.3.0.0.alpha0+ (x64) / LibreOffice Community Build ID: 36efb384a66b6dd645e0ae80fd7df68370a9dc8b CPU threads: 4; OS: Windows 6.3 Build 9600; UI render: Skia/Raster; VCL: win Locale: nl-NL (nl_NL); UI: en-US Calc: CL
Created attachment 174198 [details] Example file
Also in Version: 6.1.0.0.beta2+ Build ID: 22c451df33b733440f24c1feb6380d31240d55e6 CPU threads: 4; OS: Windows 6.3; UI render: GL; Locale: nl-NL (nl_NL); Calc: CL and in Versie: 4.4.7.2 Build ID: f3153a8b245191196a4b6b9abd1d0da16eead600 Locale: nl_NL fine in Version: 4.3.0.4 Build ID: 62ad5818884a2fc2e5780dd45466868d41009ec0
confirm in Version: 7.3.0.0.alpha0+ (x64) / LibreOffice Community Build ID: ac80ec817eb07c77a51bc0729985a473c734182e CPU threads: 16; OS: Windows 10.0 Build 19042; UI render: Skia/Raster; VCL: win Locale: ru-RU (ru_RU); UI: en-US Calc: CL
I done a bibisection using the win32-4.4 repository and got https://git.libreoffice.org/core/commit/99c48d6cc739fd542b33023e1ef51db3df169586 Adding Cc: to Stephan Bergmann
(In reply to Henrik Palomäki from comment #4) > I done a bibisection using the win32-4.4 repository and got > > https://git.libreoffice.org/core/commit/ > 99c48d6cc739fd542b33023e1ef51db3df169586 (Please always also state the git commit in the bibisect repo, not only the corresponding git commit in the source repo.) You apparently identified > commit 3c8197bed2a798f6683bffce2c6d75489ac84474 (tag: source-hash-99c48d6cc739fd542b33023e1ef51db3df169586) > Author: buildslave <buildslave@tb51.libreoffice.org> > Date: Wed Mar 30 12:54:19 2016 +0200 > > source-hash-99c48d6cc739fd542b33023e1ef51db3df169586 > > commit 99c48d6cc739fd542b33023e1ef51db3df169586 > Author: Stephan Bergmann <sbergman@redhat.com> > AuthorDate: Mon Oct 6 18:13:51 2014 +0200 > Commit: Stephan Bergmann <sbergman@redhat.com> > CommitDate: Mon Oct 6 18:13:51 2014 +0200 > > loplugin:unreffun > > Change-Id: I00240e0d17e40e187b82875bbd25a0630120c4d7 which, relative to its parent > commit d03b9722ae0e866ad8ba2bc7cdbb4d5e05d53e77 (tag: source-hash-195674627cb887b1e0d92d756a285ad48ef1d362) > Author: buildslave <buildslave@tb51.libreoffice.org> > Date: Wed Mar 30 04:59:04 2016 +0200 > > source-hash-195674627cb887b1e0d92d756a285ad48ef1d362 > > commit 195674627cb887b1e0d92d756a285ad48ef1d362 > Author: Lionel Elie Mamane <lionel@mamane.lu> > AuthorDate: Mon Oct 6 12:22:03 2014 +0200 > Commit: Lionel Elie Mamane <lionel@mamane.lu> > CommitDate: Mon Oct 6 12:23:22 2014 +0200 > > fdo#84658 on reexec, cancel modification before resetting IsModification state > > Change-Id: I878bff4d36ca1c2dd9474c937b03d0b366a6e911 encompasses 24 source commits, any of which can be the problematic one (and my test-file--only 99c48d6cc739fd542b33023e1ef51db3df169586 "loplugin:unreffun" is more than unlikely it): > $ git log --stat 195674627cb887b1e0d92d756a285ad48ef1d362..99c48d6cc739fd542b33023e1ef51db3df169586 > commit 99c48d6cc739fd542b33023e1ef51db3df169586 > Author: Stephan Bergmann <sbergman@redhat.com> > Date: Mon Oct 6 18:13:51 2014 +0200 > > loplugin:unreffun > > Change-Id: I00240e0d17e40e187b82875bbd25a0630120c4d7 > > sw/qa/core/uwriter.cxx | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > commit bd1be89c1a5fd3457b6bfe8ca78789c047820f17 > Author: Caolán McNamara <caolanm@redhat.com> > Date: Mon Oct 6 16:16:40 2014 +0100 > > encrypt sf_508f4e1..rtf to avoid it triggering virus detection > > use name CVE-pseudo-2009-0238-1.rtf to record its detected as > Virus:Trojan.Mdropper and pseudo because its just a fuzzed > file from some random .rtf and not an original reproducer for > CVE-2009-0238 > > Change-Id: I85f6a72e0d32c14ea27cbe03bc4da3c2306fd1f5 > > writerfilter/qa/cppunittests/rtftok/data/pass/CVE-pseudo-2009-0238-1.rtf | Bin 0 -> 11289 bytes > writerfilter/qa/cppunittests/rtftok/data/pass/sf_508f4e169fb76c80745d3541bd01b0a2-73462-minimized.rtf | Bin 11289 -> 0 bytes > 2 files changed, 0 insertions(+), 0 deletions(-) > > commit 76d4db3181dfd1524620377bb12a8e0332ae8ab0 > Author: Caolán McNamara <caolanm@redhat.com> > Date: Mon Oct 6 16:00:35 2014 +0100 > > Resolves: fdo#80101 inserting non-chart as chart -> unhandled exception > > Change-Id: I4d5438292d9a5c49f1336304ebd3dc4e155dd212 > > sc/source/ui/view/tabvwshb.cxx | 5 +++++ > 1 file changed, 5 insertions(+) > > commit f0859c4eae0f567393811c308074723329f1faab > Author: Caolán McNamara <caolanm@redhat.com> > Date: Mon Oct 6 15:59:09 2014 +0100 > > Resolves: fdo#84588 loop in TableInfo::getNextNode > > Change-Id: I5bb44dc94041c3b3cd9d80ba125594a48c9c921f > > sw/source/filter/ww8/wrtww8.cxx | 8 +++++++- > 1 file changed, 7 insertions(+), 1 deletion(-) > > commit 1a725ff0b42a6a97720f4663b9c5426fd5919079 > Author: Caolán McNamara <caolanm@redhat.com> > Date: Mon Oct 6 15:48:08 2014 +0100 > > cast to size_t altogether > > Change-Id: I93cb374bd9743c45c4f3a98799a5f448dbb46fd8 > > sw/qa/core/uwriter.cxx | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > commit 5c1482c3e37502bce2b3be106d1a70938ce8b30c > Author: Caolán McNamara <caolanm@redhat.com> > Date: Mon Oct 6 15:36:26 2014 +0100 > > Related; #i125163# rename IsAdaptedToNewFillProperties > > to supportsFullDrawingLayerFillAttributeSet so as to rename our equivalent > solution of d3de12fcfc85feb36eda3b0495807ef11d8fe772 > "Resolves: fdo#81277 missing table cell background color in doc/html import" > with that of 36cc153d32bc262c2285d333a1369f6ca36a7fba > "i125163 Corrected FillStyle handling for tables in Writer" > > Change-Id: If8ecbc712e01d823d034ff3771bf38a4df16368d > > sw/inc/format.hxx | 2 +- > sw/inc/frmfmt.hxx | 2 +- > sw/inc/swtblfmt.hxx | 6 +++--- > sw/source/core/attr/format.cxx | 14 +++++++------- > sw/source/core/inc/frame.hxx | 2 +- > sw/source/core/layout/atrfrm.cxx | 12 ++++++------ > sw/source/core/layout/findfrm.cxx | 4 ++-- > sw/source/core/layout/frmtool.cxx | 2 +- > sw/source/core/layout/paintfrm.cxx | 2 +- > sw/source/core/table/swtable.cxx | 6 +++--- > 10 files changed, 26 insertions(+), 26 deletions(-) > > commit 0a4d77f35e96e4dfdf69cc5ceb788ddaa849084c > Author: Miklos Vajna <vmiklos@collabora.co.uk> > Date: Mon Oct 6 15:26:35 2014 +0200 > > SwTxtNode::Update: don't move anchors at insert position > > Use cases: > > 1) User of the XTextAppend API's uses appendTextPortion(), then > appendTextContent(), then appendTextPortion() again. It's expected that > the text content will be in the middle of the paragraph, but before, it > was at the end of it. > > It's possible to work this around by inserting the text contents only at > the end of the paragraph (like it's done for ODF import in > XMLParaContext::~XMLParaContext()), but probably it's not a good idea to > copy&paste this workaround in all the import filters. > > 2) User types some characters in a new document, then inserts a picture, > and sets anchor type to at-char. Now the anchor point is set to the end > of the paragraph, but it's impossible to type anything after that > character. > > To be fair, now it's not possible to type something *before* the anchor > point, if it's at the first character, but hopefully that's less of an > issue, since setting the anchor type to at-char sets it to the end of > the paragraph, not at the start of it. > > Change-Id: Ia8b33e8880aae86f62fae09cd5ac1cf82c83320f > > sw/qa/extras/ooxmlexport/data/anchor-position.docx | Bin 0 -> 14599 bytes > sw/qa/extras/ooxmlexport/ooxmlexport.cxx | 9 +++++++++ > sw/qa/extras/ooxmlexport/ooxmlexport5.cxx | 2 +- > sw/qa/extras/ooxmlexport/ooxmlsdrexport.cxx | 2 +- > sw/source/core/txtnode/ndtxt.cxx | 16 ++++++++++++++++ > 5 files changed, 27 insertions(+), 2 deletions(-) > > commit 54e86f708cf0274ec6b2536200802cf6f406101d > Author: Stephan Bergmann <sbergman@redhat.com> > Date: Mon Oct 6 16:49:26 2014 +0200 > > Missing include > > Change-Id: I47ddb95018109e5e771324b699dfdc10f529dfc4 > > sd/source/ui/remotecontrol/AvahiNetworkService.cxx | 1 + > 1 file changed, 1 insertion(+) > > commit 946ea643185af680508a0333f63c086ed3f454a3 > Author: Caolán McNamara <caolanm@redhat.com> > Date: Mon Oct 6 15:27:37 2014 +0100 > > Resolves: fdo#84649 can have negative values from ConvertToModelPosition > > Change-Id: I570ee54e15a4cd4b11da26004164d9e6a1d8cdb9 > > sw/source/core/bastyp/breakit.cxx | 3 +++ > 1 file changed, 3 insertions(+) > > commit de78b2ca3c24a822ad9b8456323b9d2210ae0487 > Author: Caolán McNamara <caolanm@redhat.com> > Date: Mon Oct 6 15:26:13 2014 +0100 > > when size_t == unsigned int > > Change-Id: If8cd3632d69f6456b0e1351efac0e428e0093e3b > > comphelper/source/misc/random.cxx | 4 ++++ > include/comphelper/random.hxx | 4 ++++ > 2 files changed, 8 insertions(+) > > commit 2c6d17b02529f0bc8bfcb1fc16f9e1481ea71829 > Author: Matthew J. Francis <mjay.francis@gmail.com> > Date: Sun Oct 5 13:09:03 2014 +0800 > > Refactor giant SwDocTest::testModelToViewHelper() into smaller tests > > This is to clear the way for following bugfixes which will add to / > amend these tests > > Change-Id: Ie97aba72f1a4c7b1be37577b9bcab2b2f0750d17 > Reviewed-on: https://gerrit.libreoffice.org/11808 > Reviewed-by: David Tardon <dtardon@redhat.com> > Tested-by: David Tardon <dtardon@redhat.com> > > sw/qa/core/uwriter.cxx | 490 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++-------------------------------------------------------------------------------------------------------------------------------- > 1 file changed, 292 insertions(+), 198 deletions(-) > > commit e41597eade24f04893752a4339112b13419fefdd > Author: Robert Antoni Buj i Gelonch <robert.buj@gmail.com> > Date: Fri Oct 3 17:07:28 2014 +0200 > > scripting: line break (code formatting) > > Change-Id: Ib1a8d7e4bdaad2f4eae822d9af280fd5b6f9dd7b > Reviewed-on: https://gerrit.libreoffice.org/11794 > Reviewed-by: David Tardon <dtardon@redhat.com> > Tested-by: David Tardon <dtardon@redhat.com> > > scripting/java/Framework/com/sun/star/script/framework/security/SecurityDialog.java | 246 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++----------------------------------------------------------------------------------------- > scripting/java/com/sun/star/script/framework/browse/DialogFactory.java | 95 ++++++++++++++++++++++++++++++++++++++++++++++++++++----------------------------------------- > scripting/java/com/sun/star/script/framework/browse/ParcelBrowseNode.java | 86 +++++++++++++++++++++++++++++++++++++++++++----------------------------------------- > scripting/java/com/sun/star/script/framework/browse/PkgProviderBrowseNode.java | 6 +++--- > scripting/java/com/sun/star/script/framework/browse/ProviderBrowseNode.java | 70 ++++++++++++++++++++++++++++++++++----------------------------------- > scripting/java/com/sun/star/script/framework/browse/ScriptBrowseNode.java | 114 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++---------------------------------------------- > scripting/java/com/sun/star/script/framework/container/DeployedUnoPackagesDB.java | 8 ++++++-- > scripting/java/com/sun/star/script/framework/container/Parcel.java | 63 ++++++++++++++++++++++++++++++++++++++------------------------ > scripting/java/com/sun/star/script/framework/container/ParcelContainer.java | 177 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++---------------------------------------------------------------------- > scripting/java/com/sun/star/script/framework/container/ParcelDescriptor.java | 15 ++++++--------- > scripting/java/com/sun/star/script/framework/container/ParsedScriptUri.java | 3 +-- > scripting/java/com/sun/star/script/framework/container/ScriptEntry.java | 9 ++++++--- > scripting/java/com/sun/star/script/framework/container/ScriptMetaData.java | 52 +++++++++++++++++++++++++-------------------------- > scripting/java/com/sun/star/script/framework/container/UnoPkgContainer.java | 100 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++------------------------------------- > scripting/java/com/sun/star/script/framework/container/XMLParser.java | 2 +- > scripting/java/com/sun/star/script/framework/container/XMLParserFactory.java | 55 ++++++++++++++++++++++++++++++++++-------------------- > scripting/java/com/sun/star/script/framework/io/UCBStreamHandler.java | 47 ++++++++++++++++++++++++++-------------------- > scripting/java/com/sun/star/script/framework/io/XInputStreamImpl.java | 19 ++++++++++++------- > scripting/java/com/sun/star/script/framework/io/XInputStreamWrapper.java | 4 ++-- > scripting/java/com/sun/star/script/framework/io/XOutputStreamWrapper.java | 24 +++++++++++------------- > scripting/java/com/sun/star/script/framework/io/XStorageHelper.java | 53 ++++++++++++++++++++++++++++++++++------------------ > scripting/java/com/sun/star/script/framework/log/LogUtils.java | 2 +- > scripting/java/com/sun/star/script/framework/provider/ClassLoaderFactory.java | 5 +++-- > scripting/java/com/sun/star/script/framework/provider/EditorScriptContext.java | 16 ++++++++++------ > scripting/java/com/sun/star/script/framework/provider/NoSuitableClassLoaderException.java | 5 +++-- > scripting/java/com/sun/star/script/framework/provider/PathUtils.java | 10 ++++++---- > scripting/java/com/sun/star/script/framework/provider/ScriptContext.java | 24 ++++++++++++++++-------- > scripting/java/com/sun/star/script/framework/provider/ScriptEditor.java | 2 +- > scripting/java/com/sun/star/script/framework/provider/ScriptProvider.java | 239 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++------------------------------------------------------------------------------------------------------------------- > scripting/java/com/sun/star/script/framework/provider/SwingInvocation.java | 3 ++- > scripting/java/com/sun/star/script/framework/provider/beanshell/PlainSourceView.java | 6 +++--- > scripting/java/com/sun/star/script/framework/provider/beanshell/ScriptEditorForBeanShell.java | 26 +++++++++++++------------- > scripting/java/com/sun/star/script/framework/provider/beanshell/ScriptProviderForBeanShell.java | 116 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++---------------------------------------------- > scripting/java/com/sun/star/script/framework/provider/beanshell/ScriptSourceModel.java | 3 ++- > scripting/java/com/sun/star/script/framework/provider/beanshell/ScriptSourceView.java | 2 +- > scripting/java/com/sun/star/script/framework/provider/java/Resolver.java | 10 ++++------ > scripting/java/com/sun/star/script/framework/provider/java/ScriptDescriptor.java | 20 ++++---------------- > scripting/java/com/sun/star/script/framework/provider/java/ScriptProviderForJava.java | 124 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++----------------------------------------------------------- > scripting/java/com/sun/star/script/framework/provider/java/ScriptProxy.java | 14 +++++--------- > scripting/java/com/sun/star/script/framework/provider/java/StrictResolver.java | 33 ++++++++++++++++++--------------- > scripting/java/com/sun/star/script/framework/provider/javascript/ScriptEditorForJavaScript.java | 24 ++++++++++-------------- > scripting/java/com/sun/star/script/framework/provider/javascript/ScriptProviderForJavaScript.java | 79 ++++++++++++++++++++++++++++++++++++++++++------------------------------------ > scripting/java/org/openoffice/netbeans/modules/office/actions/DeployParcelAction.java | 8 ++++---- > 43 files changed, 1136 insertions(+), 883 deletions(-) > > commit b1d48f0d9a3442c11adae6e36325a463d89ce44e > Author: Takeshi Abe <tabe@fixedpoint.jp> > Date: Sun Oct 5 11:40:03 2014 +0900 > > fdo#75757: remove inheritance to std::vector > > from PptSlidePersistList. > > This also puts in assertions that every entry in a PptSlidePersistList > is non-null. > Cf. 9536a28ede89e8c29940e8ba225215a45667062c > > Change-Id: Iea35532ed668e58a3f02d3f3433a66c2d4e1fe2a > Reviewed-on: https://gerrit.libreoffice.org/11807 > Reviewed-by: David Tardon <dtardon@redhat.com> > Tested-by: David Tardon <dtardon@redhat.com> > > filter/source/msfilter/svdfppt.cxx | 101 ++++++++++++++++++++++++++++++++++++++++++++++++++--------------------------------------------------- > include/filter/msfilter/svdfppt.hxx | 25 +++++++++++++++---------- > sd/source/filter/ppt/pptin.cxx | 29 ++++++++++++++--------------- > 3 files changed, 79 insertions(+), 76 deletions(-) > > commit 4108bd9b7a41eaa0f3bf8b8173f27f57e009ee34 > Author: Justin Luth <justin_luth@sil.org> > Date: Tue Sep 30 14:37:59 2014 -0400 > > fdo#71248 Excel VBA: ActiveDocument is not tracking currently selected Doc > > Change-Id: I86606646206ec26e2822a6e27888537d44351b7b > Reviewed-on: https://gerrit.libreoffice.org/11725 > Reviewed-by: Samuel Mehrbrodt <s.mehrbrodt@gmail.com> > Tested-by: Samuel Mehrbrodt <s.mehrbrodt@gmail.com> > > sfx2/source/doc/objxtor.cxx | 2 +- > sw/inc/unoprnms.hxx | 2 +- > 2 files changed, 2 insertions(+), 2 deletions(-) > > commit 76b50b7b0d58e73b973b79566e227a22b831fcbb > Author: Caolán McNamara <caolanm@redhat.com> > Date: Mon Oct 6 14:41:43 2014 +0100 > > Resolves: fdo#84698 crash in autoreplacement > > Change-Id: I44d5dacb21870f77ca993b24f3a31433edb17c6d > > editeng/source/misc/svxacorr.cxx | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > commit 17dd074b3f775326b6787c0fc77d4048e541e220 > Author: Caolán McNamara <caolanm@redhat.com> > Date: Mon Oct 6 12:50:20 2014 +0100 > > drop connectivity testmoz workben > > Change-Id: I09b387606dee76605ad60dcf8cfb3fc68057d7be > > connectivity/workben/testmoz/initUNO.cxx | 37 ----------------- > connectivity/workben/testmoz/main.cxx | 663 -------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- > connectivity/workben/testmoz/makefile.mk | 61 ---------------------------- > connectivity/workben/testmoz/mozthread.cxx | 454 ----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- > 4 files changed, 1215 deletions(-) > > commit 8f436d3de7e99268a8862664d2cb2574231c3b18 > Author: Caolán McNamara <caolanm@redhat.com> > Date: Fri Oct 3 17:12:23 2014 +0100 > > use comphelper::rng::uniform_*_distribution everywhere > > and automatically seed from time on first use > > coverity#1242393 Don't call rand > coverity#1242404 Don't call rand > coverity#1242410 Don't call rand and additionally allow 0xFF as a value > coverity#1242409 Don't call rand > coverity#1242399 Don't call rand > coverity#1242372 Don't call rand > coverity#1242377 Don't call rand > coverity#1242378 Don't call rand > coverity#1242379 Don't call rand > coverity#1242382 Don't call rand > coverity#1242383 Don't call rand > coverity#1242402 Don't call rand > coverity#1242397 Don't call rand > coverity#1242390 Don't call rand > coverity#1242389 Don't call rand > coverity#1242388 Don't call rand > coverity#1242386 Don't call rand > coverity#1242384 Don't call rand > coverity#1242394 Don't call rand > > Change-Id: I241feab9cb370e091fd6ccaba2af941eb95bc7cf > > basegfx/CppunitTest_basegfx.mk | 1 + > basegfx/test/boxclipper.cxx | 4 ++-- > basic/source/runtime/methods.cxx | 4 ++-- > comphelper/source/misc/random.cxx | 58 +++++++++++++++++++++++++++++++++++++++++++++------------- > cui/source/customize/cfg.cxx | 4 ++-- > desktop/source/deployment/misc/lockfile.cxx | 8 +++----- > drawinglayer/source/primitive2d/polygonprimitive2d.cxx | 16 +++++++++++----- > drawinglayer/source/texture/texture.cxx | 9 ++++----- > include/comphelper/random.hxx | 21 +++++++++++++++++---- > include/drawinglayer/tools/converters.hxx | 2 ++ > oox/source/export/chartexport.cxx | 4 ++-- > sc/Library_scopencl.mk | 1 + > sc/source/core/data/global.cxx | 2 -- > sc/source/core/data/table3.cxx | 5 +++-- > sc/source/core/opencl/opencl_device.cxx | 4 ++-- > sc/source/core/tool/interpr1.cxx | 2 +- > sc/source/core/tool/interpr3.cxx | 3 ++- > sc/source/ui/StatisticsDialogs/SamplingDialog.cxx | 13 ++----------- > scaddins/idl/com/sun/star/sheet/addin/XAnalysis.idl | 2 ++ > scaddins/source/analysis/analysis.cxx | 9 +++------ > sd/source/core/CustomAnimationPreset.cxx | 7 ++++--- > sd/source/ui/remotecontrol/AvahiNetworkService.cxx | 5 +++-- > slideshow/source/engine/OGLTrans/generic/OGLTrans_TransitionImpl.cxx | 4 +++- > slideshow/source/inc/tools.hxx | 5 ++--- > svx/source/svdraw/sdrpagewindow.cxx | 6 +++++- > svx/source/svdraw/sdrpaintwindow.cxx | 32 +++++--------------------------- > sw/qa/core/uwriter.cxx | 16 +++++++++------- > sw/source/filter/ww8/docxattributeoutput.cxx | 9 +++++---- > vbahelper/source/vbahelper/vbacommandbarhelper.cxx | 4 ++-- > vcl/source/window/debugevent.cxx | 3 ++- > xmloff/source/text/txtlists.cxx | 4 ++-- > 31 files changed, 149 insertions(+), 118 deletions(-) > > commit e5ab3685550cf35c3eb9cb47530044f2d86433d5 > Author: Caolán McNamara <caolanm@redhat.com> > Date: Mon Oct 6 09:19:18 2014 +0100 > > Related: coverity#704049 Unchecked return value > > surely this should be IsLess not IsEqual, i.e. > see > commit 86a3fe47a66950e26d23d7d7f2680fa7d4fb0839 > Date: Sat Oct 4 21:36:27 2014 -0500 > coverity#704049 Unchecked return value > and > commit 3edb1dcdc82304c418fc32212bb0bdf2c92e414e > Date: Wed Jun 26 08:32:38 2002 +0000 > #99877# Phonetic sorting for Asian languages > > Change-Id: I6c78e388e986577c34b37643a877ab7f5c17b173 > > sw/source/core/tox/txmsrt.cxx | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > commit 58ff30d1a1b9404255bb0dbca5ed52e8355292a4 > Author: Caolán McNamara <caolanm@redhat.com> > Date: Mon Oct 6 09:14:21 2014 +0100 > > convert to floating point before divide > > Change-Id: I0328ca6594b3809210ea124dcb744bad3b5eb92e > > sdext/source/presenter/PresenterHelpView.cxx | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > commit d73cee7858edc04944578f52a6d838d866aafb61 > Author: Michael Stahl <mstahl@redhat.com> > Date: Mon Oct 6 13:32:24 2014 +0200 > > gbuild: enable -Wnon-virtual-dtor for all GCC platforms > > It was disabled globally at a time when GCC did not yet support #pragma > GCC diagnostic. > > Change-Id: I062f6b437e6119203f2b0824a25238b45b4f4e79 > > solenv/gbuild/platform/IOS_ARM_GCC.mk | 4 ---- > solenv/gbuild/platform/com_GCC_defs.mk | 1 + > solenv/gbuild/platform/macosx.mk | 4 ---- > solenv/gbuild/platform/mingw.mk | 4 ---- > solenv/gbuild/platform/solaris.mk | 1 - > solenv/gbuild/platform/unxgcc.mk | 1 - > 6 files changed, 1 insertion(+), 14 deletions(-) > > commit c2056bd31caa61df9f9b6ffd26479ccd85ea8f1e > Author: Michael Stahl <mstahl@redhat.com> > Date: Mon Oct 6 13:19:45 2014 +0200 > > gbuild: -Wnon-virtual-dtor and strict aliasing workaround for GCC < 4.6 > > ... is obsolete, remove that. > > Change-Id: I541de32ac4389d8049c25c98b63a69d084d1594d > > solenv/gbuild/platform/solaris.mk | 15 +-------------- > solenv/gbuild/platform/unxgcc.mk | 28 +--------------------------- > 2 files changed, 2 insertions(+), 41 deletions(-) > > commit 89a606b4649932bd811cfa789ff92bf3055032e7 > Author: Michael Stahl <mstahl@redhat.com> > Date: Mon Oct 6 13:23:29 2014 +0200 > > configure: another GCC version check that is always true > > Change-Id: Iafbc0dadc1aba4267414e2c6653bf306e4d84f4c > > configure.ac | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > commit 076cd220c28bf118f85bd0bfcd02dcec08847f3e > Author: Michael Stahl <mstahl@redhat.com> > Date: Mon Oct 6 13:09:15 2014 +0200 > > configure: for C++11 support, at least GCC 4.6 is required > > Change-Id: I0b61d738a85fc0d874fa9e18b4af4ebde5a40385 > > configure.ac | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > commit 969941f996385fa48f405f51f97a39454ef2b79a > Author: Caolán McNamara <caolanm@redhat.com> > Date: Wed Oct 1 12:17:38 2014 +0100 > > Related: fdo#58277 if the SwFrm is not IsValid so are its portions > > i.e. they don't necessarily match the SwTxtNode anymore, e.g. > content deleted from node, SwFrm is invalid, old portions > refer to old offsets not new ones yet > > Change-Id: I755051cd647aa7bb203a6e1f815193fb6ec39191 > > sw/source/core/text/txtfrm.cxx | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-)
(In reply to Stephan Bergmann from comment #5) > > commit 0a4d77f35e96e4dfdf69cc5ceb788ddaa849084c > > Author: Miklos Vajna <vmiklos@collabora.co.uk> > > Date: Mon Oct 6 15:26:35 2014 +0200 > > > > SwTxtNode::Update: don't move anchors at insert position > > > > Use cases: > > > > 1) User of the XTextAppend API's uses appendTextPortion(), then > > appendTextContent(), then appendTextPortion() again. It's expected that > > the text content will be in the middle of the paragraph, but before, it > > was at the end of it. > > > > It's possible to work this around by inserting the text contents only at > > the end of the paragraph (like it's done for ODF import in > > XMLParaContext::~XMLParaContext()), but probably it's not a good idea to > > copy&paste this workaround in all the import filters. > > > > 2) User types some characters in a new document, then inserts a picture, > > and sets anchor type to at-char. Now the anchor point is set to the end > > of the paragraph, but it's impossible to type anything after that > > character. > > > > To be fair, now it's not possible to type something *before* the anchor > > point, if it's at the first character, but hopefully that's less of an > > issue, since setting the anchor type to at-char sets it to the end of > > the paragraph, not at the start of it. > > > > Change-Id: Ia8b33e8880aae86f62fae09cd5ac1cf82c83320f > > > > sw/qa/extras/ooxmlexport/data/anchor-position.docx | Bin 0 -> 14599 bytes > > sw/qa/extras/ooxmlexport/ooxmlexport.cxx | 9 +++++++++ > > sw/qa/extras/ooxmlexport/ooxmlexport5.cxx | 2 +- > > sw/qa/extras/ooxmlexport/ooxmlsdrexport.cxx | 2 +- > > sw/source/core/txtnode/ndtxt.cxx | 16 ++++++++++++++++ > > 5 files changed, 27 insertions(+), 2 deletions(-) Miklos, I wild guess that the above might be the relevant commit?
Yes, probably I forgot to adapt undo/redo to do the opposite of the new behavior of SwTextNode::Update().
> (Please always also state the git commit in the bibisect repo, not only the > corresponding git commit in the source repo.) OK, thanks for the feedback.
Miklos Vajna committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/1734e97222324c137ecd084ad2464abdff2698d1 tdf#143819 sw: fix undo not restoring adjusted anchor It will be available in 7.3.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.