[Libreoffice-bugs] [Bug 83306] FILEOPEN: Artifacts when importing .docx files with Autoshape images (Windows only)
https://bugs.documentfoundation.org/show_bug.cgi?id=83306 Michael Stahl changed: What|Removed |Added Keywords||bisected CC||caol...@redhat.com, ||noelgran...@gmail.com --- Comment #11 from Michael Stahl --- it doesn't look like commit dcbac37efebb9877a72f7c9914b63d60f46a5656 is responsible - i can't build that old master with MSVC 2013 but a) the whole Fraction class was removed before 4.4 branch off anyway b) if i disable the WrapPolygon handling in GraphicImport.cxx completely i still get the same extra lines in the image ... the minimal group shape to demonstrate the bug that shows white image on Linux and 3 grey lines on Windows is this: ... okay if i play with strtempl.cxx i find that the responsible commit is: commit 281989007fd7dea997ed9a65f513f80b1aff67dd Author: Noel Grandin AuthorDate: Tue Jul 1 13:17:01 2014 +0200 Commit: Caolán McNamara CommitDate: Fri Jul 4 09:04:05 2014 + Use standard library optimised routines for OUString/OString ..handling where possible. urgh :( -- You are receiving this mail because: You are the assignee for the bug.___ Libreoffice-bugs mailing list Libreoffice-bugs@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs
[Libreoffice-bugs] [Bug 83306] FILEOPEN: Artifacts when importing .docx files with Autoshape images (Windows only)
https://bugs.documentfoundation.org/show_bug.cgi?id=83306
--- Comment #12 from Michael Stahl ---
the bug is triggered in compare_WithLength, in
+if (sizeof(IMPL_RTL_STRCODE) == sizeof(wchar_t))
+{
+// take advantage of builtin optimisations
+sal_Int32 nMin = std::min(nStr1Len, nStr2Len);
+sal_Int32 nRet = wcsncmp((wchar_t*)pStr1, (wchar_t*)pStr2, nMin);
+return nRet == 0 ? nStr1Len - nStr2Len : nRet;
+}
the problem is that the oox code uses strings with embedded null bytes
to identify shapes (why?), and wcsncmp stops at the first null byte
but the old code would compare them too. this causes different results.
--
You are receiving this mail because:
You are the assignee for the bug.___
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs
[Libreoffice-bugs] [Bug 83306] FILEOPEN: Artifacts when importing .docx files with Autoshape images (Windows only)
https://bugs.documentfoundation.org/show_bug.cgi?id=83306 --- Comment #13 from Caolán McNamara --- change to wmemcmp/memcmp ? -- You are receiving this mail because: You are the assignee for the bug.___ Libreoffice-bugs mailing list Libreoffice-bugs@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs
[Libreoffice-bugs] [Bug 83306] FILEOPEN: Artifacts when importing .docx files with Autoshape images (Windows only)
https://bugs.documentfoundation.org/show_bug.cgi?id=83306 Commit Notification changed: What|Removed |Added Whiteboard||target:5.3.0 -- You are receiving this mail because: You are the assignee for the bug.___ Libreoffice-bugs mailing list Libreoffice-bugs@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs
[Libreoffice-bugs] [Bug 83306] FILEOPEN: Artifacts when importing .docx files with Autoshape images (Windows only)
https://bugs.documentfoundation.org/show_bug.cgi?id=83306 --- Comment #14 from Commit Notification --- Michael Stahl committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=de7ed418e7813c85f192b558ff06e976eccaa54d tdf#83306: sal: fix compare of rtl::OUString/OString containing '\0' It will be available in 5.3.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback. -- You are receiving this mail because: You are the assignee for the bug.___ Libreoffice-bugs mailing list Libreoffice-bugs@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs
[Libreoffice-bugs] [Bug 83306] FILEOPEN: Artifacts when importing .docx files with Autoshape images (Windows only)
https://bugs.documentfoundation.org/show_bug.cgi?id=83306 Michael Stahl changed: What|Removed |Added CC||alexpik...@gmail.com --- Comment #15 from Michael Stahl --- *** Bug 82658 has been marked as a duplicate of this bug. *** -- You are receiving this mail because: You are the assignee for the bug.___ Libreoffice-bugs mailing list Libreoffice-bugs@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs
[Libreoffice-bugs] [Bug 83306] FILEOPEN: Artifacts when importing .docx files with Autoshape images (Windows only)
https://bugs.documentfoundation.org/show_bug.cgi?id=83306 Commit Notification changed: What|Removed |Added Whiteboard|target:5.3.0|target:5.3.0 target:5.2.3 -- You are receiving this mail because: You are the assignee for the bug.___ Libreoffice-bugs mailing list Libreoffice-bugs@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs
[Libreoffice-bugs] [Bug 83306] FILEOPEN: Artifacts when importing .docx files with Autoshape images (Windows only)
https://bugs.documentfoundation.org/show_bug.cgi?id=83306 Commit Notification changed: What|Removed |Added Whiteboard|target:5.3.0|target:5.3.0 target:5.2.3 Caolán McNamara changed: What|Removed |Added Status|NEW |RESOLVED Resolution|--- |FIXED -- You are receiving this mail because: You are the assignee for the bug.___ Libreoffice-bugs mailing list Libreoffice-bugs@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs
[Libreoffice-bugs] [Bug 83306] FILEOPEN: Artifacts when importing .docx files with Autoshape images (Windows only)
https://bugs.documentfoundation.org/show_bug.cgi?id=83306 Caolán McNamara changed: What|Removed |Added Status|NEW |RESOLVED Resolution|--- |FIXED --- Comment #16 from Commit Notification --- Michael Stahl committed a patch related to this issue. It has been pushed to "libreoffice-5-2": http://cgit.freedesktop.org/libreoffice/core/commit/?id=d47756ab1fce2aedd7658e49fa893c2e2b63286e&h=libreoffice-5-2 tdf#83306: sal: fix compare of rtl::OUString/OString containing '\0' It will be available in 5.2.3. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback. -- You are receiving this mail because: You are the assignee for the bug.___ Libreoffice-bugs mailing list Libreoffice-bugs@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs
[Libreoffice-bugs] [Bug 83306] FILEOPEN: Artifacts when importing .docx files with Autoshape images (Windows only)
https://bugs.documentfoundation.org/show_bug.cgi?id=83306 Commit Notification changed: What|Removed |Added Whiteboard|target:5.3.0 target:5.2.3 |target:5.3.0 target:5.2.3 ||target:5.1.6 -- You are receiving this mail because: You are the assignee for the bug.___ Libreoffice-bugs mailing list Libreoffice-bugs@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs
[Libreoffice-bugs] [Bug 83306] FILEOPEN: Artifacts when importing .docx files with Autoshape images (Windows only)
https://bugs.documentfoundation.org/show_bug.cgi?id=83306 --- Comment #17 from Commit Notification --- Michael Stahl committed a patch related to this issue. It has been pushed to "libreoffice-5-1": http://cgit.freedesktop.org/libreoffice/core/commit/?id=7bc09f32ce86c6ed0df00ad72451b41204e0162f&h=libreoffice-5-1 tdf#83306: sal: fix compare of rtl::OUString/OString containing '\0' It will be available in 5.1.6. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback. -- You are receiving this mail because: You are the assignee for the bug.___ Libreoffice-bugs mailing list Libreoffice-bugs@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs
[Libreoffice-bugs] [Bug 83306] FILEOPEN: Artifacts when importing .docx files with Autoshape images (Windows only)
https://bugs.documentfoundation.org/show_bug.cgi?id=83306 --- Comment #18 from Commit Notification --- Michael Stahl committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=a0687f7e656a137cd0c1661094afc72f466ba629 tdf#83306 add unit test for compareWithLength and '\0' It will be available in 5.3.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback. -- You are receiving this mail because: You are the assignee for the bug.___ Libreoffice-bugs mailing list Libreoffice-bugs@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs
[Libreoffice-bugs] [Bug 83306] FILEOPEN: Artifacts when importing .docx files with Autoshape images (Windows only)
https://bugs.documentfoundation.org/show_bug.cgi?id=83306 Luke changed: What|Removed |Added Summary|FILEOPEN: Regression|FILEOPEN: Artifacts when |importing .docx files with |importing .docx files with |Autoshape images, click -> |Autoshape images (Windows |Crash |only) --- Comment #7 from Luke --- It's no longer crashing. Changed the title back. -- 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
[Libreoffice-bugs] [Bug 83306] FILEOPEN: Artifacts when importing .docx files with Autoshape images (Windows only)
https://bugs.documentfoundation.org/show_bug.cgi?id=83306 Xisco Faulí changed: What|Removed |Added Keywords|notBibisectable |bibisectRequest --- Comment #9 from Xisco Faulí --- I guess this can be bibisected with this repository: http://dev-downloads.libreoffice.org/bibisect/win/bibisect_win_44.tar.xz. Adding keyword 'bibisectRequest' -- You are receiving this mail because: You are the assignee for the bug.___ Libreoffice-bugs mailing list Libreoffice-bugs@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs
[Libreoffice-bugs] [Bug 83306] FILEOPEN: Artifacts when importing .docx files with Autoshape images (Windows only)
https://bugs.documentfoundation.org/show_bug.cgi?id=83306 raal changed: What|Removed |Added Keywords|bibisectRequest |bibisected CC||mst...@redhat.com, ||r...@post.cz --- Comment #10 from raal --- git bisect log # bad: [489ffd1df414698b6cea9ab03bf6f663b8b5af7e] source-hash-3f94c9e9ddfd807b449f3bb9b232cf2041fa12d2 # good: [8aa9fc9a0c92172593d6cd97662116a965db229d] source-hash-dea4a3b9d7182700abeb4dc756a24a9e8dea8474 git bisect start 'latest' 'oldest' # bad: [897913acd244cb6a5d2f4c2da1d625d9b978edb6] source-hash-ac57362b23859591c088e36b7218f4a606dcf3bb git bisect bad 897913acd244cb6a5d2f4c2da1d625d9b978edb6 # bad: [dc97f44745f96315fb6c5480705bb5d595d39c6c] source-hash-01c8962f281887db59e581906b89d027a994b52a git bisect bad dc97f44745f96315fb6c5480705bb5d595d39c6c # good: [a5a39af2ec486b10b26d6373f58a9b59142106c4] source-hash-a595879302e26a616131aa0cab5de31bb287b37d git bisect good a5a39af2ec486b10b26d6373f58a9b59142106c4 # bad: [54a9a5df46b192bd5b9bd44702ef3bffd5730523] source-hash-e2f1fffc81896c0773a18e468635056710927d8f git bisect bad 54a9a5df46b192bd5b9bd44702ef3bffd5730523 # bad: [54a9a5df46b192bd5b9bd44702ef3bffd5730523] source-hash-e2f1fffc81896c0773a18e468635056710927d8f git bisect bad 54a9a5df46b192bd5b9bd44702ef3bffd5730523 # bad: [54a9a5df46b192bd5b9bd44702ef3bffd5730523] source-hash-e2f1fffc81896c0773a18e468635056710927d8f git bisect bad 54a9a5df46b192bd5b9bd44702ef3bffd5730523 # good: [f1cbe8b4bb7351ee7ca566e6c29229948fcac423] source-hash-9263b101c39172cbcf04189c515bca80cb15f3aa git bisect good f1cbe8b4bb7351ee7ca566e6c29229948fcac423 # good: [649737e10cac6fffed69f0b6f02d517ca5ff2a05] source-hash-c38be106ff1c1ff5e1b279e4ea2c710b524d23f1 git bisect good 649737e10cac6fffed69f0b6f02d517ca5ff2a05 # bad: [b9e32c086d6e3c96eb05ad80b0005afeb43305c8] source-hash-81df594b4fbb147d3e4b3cb31ae27ff7f66d83b4 git bisect bad b9e32c086d6e3c96eb05ad80b0005afeb43305c8 # bad: [30fbacc51969fa4e3f572b25142b58103956ed06] source-hash-da36ded02c67bb7481cd4378ce5f7d779c1a3533 git bisect bad 30fbacc51969fa4e3f572b25142b58103956ed06 # good: [1d9fe19e246fed2d4e059300c6c19232f64659cb] source-hash-1f90cae1debed4b45bb51ced21a03aacd7973cff git bisect good 1d9fe19e246fed2d4e059300c6c19232f64659cb # good: [1d9fe19e246fed2d4e059300c6c19232f64659cb] source-hash-1f90cae1debed4b45bb51ced21a03aacd7973cff git bisect good 1d9fe19e246fed2d4e059300c6c19232f64659cb # good: [e33eab517819eed0b577bb43de03305a9d2c3aa5] source-hash-6f92b58987f754de31c9ca756e813deb7462d98e git bisect good e33eab517819eed0b577bb43de03305a9d2c3aa5 # first bad commit: [30fbacc51969fa4e3f572b25142b58103956ed06] source-hash-da36ded02c67bb7481cd4378ce5f7d779c1a3533 One of these commits http://cgit.freedesktop.org/libreoffice/core/log/?qt=range&q=6f92b58987f754de31c9ca756e813deb7462d98e..da36ded02c67bb7481cd4378ce5f7d779c1a3533 Age Commit message (Expand) Author Files Lines 2014-07-04 fix autotextCaolán McNamara 1 -4/+4 2014-07-04 Related fdo#77603: update Spanish autocorrection patterns. Adolfo Jayme Barrientos 1 -5/+140 2014-07-04 Use standard library optimised routines for OUString/OString Noel Grandin1 -60/+178 2014-07-04 fix spelling in class name OSpecialHanldeXMLExportPropertyMapperNoel Grandin2 -11/+11 2014-07-04 Fix: EE_CHAR_COLOR to EE_CHAR_BKGCOLOR matteocam 1 -1/+1 2014-07-04 Avoid possible memory leaks in case of exceptions Takeshi Abe 6 -47/+33 2014-07-04 Drop unused #includes Takeshi Abe 2 -4/+0 2014-07-03 fdo#76803: writerfilter: fix image wrap polygon import again Michael Stahl 1 -2/+4 Michael, can it be your commit? author Michael Stahl2014-07-03 22:29:55 (GMT) committer Michael Stahl2014-07-03 22:39:36 (GMT) commit dcbac37efebb9877a72f7c9914b63d60f46a5656 (patch) fdo#76803: writerfilter: fix image wrap polygon import again -- You are receiving this mail because: You are the assignee for the bug.___ Libreoffice-bugs mailing list Libreoffice-bugs@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs
