summaryrefslogtreecommitdiffstats
path: root/sw/qa
Commit message (Collapse)AuthorAgeFilesLines
* tdf#107104 DOCX drawingML import: fix invisible arrow shapeMiklos Vajna2017-04-254-3/+12
| | | | | | | | | | | | | | | | | | | This is the drawingML equivalent of commit 3d9ebded1358395ed81db7a63629b046aec2aeac (Misc improvements for docx VML import, 2010-10-06), which made sure that shapes are never invisible just because they have zero height or width. For this particular bugdoc the Word-produced WW8 equivalent width is 20 twips, but let's be consistent with the VML import and just round up to 1 mm100. Also fix two existing tests that wanted to test something else, but implicitly asserted that some shapes indeed have zero width/height. Change-Id: I9600424520d0a3deecc711b44622eccc041a59da Reviewed-on: https://gerrit.libreoffice.org/36927 Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk> Tested-by: Jenkins <ci@libreoffice.org>
* tools: svstream.hxx needs only errcode.hxx & not errinf.hxxChris Sherlock2017-04-257-0/+7
| | | | | | | Change-Id: Ia28e35ae5af4f601e9a586a3deffbcd61702b0ca Reviewed-on: https://gerrit.libreoffice.org/36896 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Chris Sherlock <chris.sherlock79@gmail.com>
* tdf107359: imporve line pitch and char pitch importing of writerfilter.Mark Hung2017-04-242-0/+21
| | | | | | | | | | | 1. Use converted style name to assure the default style is found. 2. Switch off squared-page mode before setting the base text width and height. 3. Ruby text height is not effective per ODF spec. Change-Id: I0f2901a453a9f7b344cac6989780688cc2d6c7b4 Reviewed-on: https://gerrit.libreoffice.org/36828 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Mark Hung <marklh9@gmail.com>
* Fix typosAndrea Gelmini2017-04-241-1/+1
| | | | | | | Change-Id: I3e130b68c18ebe24c072d317d59b4854608a222f Reviewed-on: https://gerrit.libreoffice.org/36870 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Michael Stahl <mstahl@redhat.com>
* Indentation fixesMiklos Vajna2017-04-241-5/+5
| | | | | | No functional changes intended. Change-Id: Ibc23de9cb33428765b8b0c85a221a2014ad4d7bd
* sw: skip part of uiwriter/testTdf107025 if not testable.Mark Hung2017-04-221-3/+6
| | | | | | | | | | | | The unit test is about textgrid and uses DFKAI-SB that shipped with Windows. Required font might not installed in some build environment. Not having width in the first implies that the case is not testable. Change-Id: I98f9b9f6291eea95a1b36f1890b1be903ccb26dd Reviewed-on: https://gerrit.libreoffice.org/36802 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Mark Hung <marklh9@gmail.com>
* Silence loplugin:unreffun on "FIXME breaks on Windows"Stephan Bergmann2017-04-211-1/+2
| | | | Change-Id: I9b03f56d8d1b45648f9d71fe1e8632fe58079c4f
* Disable this test temporarily, breaks on Windows from some reason.Jan Holesovsky2017-04-211-1/+1
| | | | Change-Id: I47998d5844901b8090cd47f55a49c872550e2e38
* related tdf#68604: Unit test for writing the plaintext annotations in DOCX.Jan Holesovsky2017-04-211-0/+28
| | | | Change-Id: I8c747e72ca96ffd097c92326210c39740102ec79
* add CVE-2017-0199 testcaseCaolán McNamara2017-04-211-0/+0
| | | | Change-Id: I07cac2ffac85093761dbdd7c50c52483bfbb5754
* tdf#101627 add unit testPatrick Jaap2017-04-202-0/+8
| | | | | | | | | unit test for the patch 80b9b6761e8cb974e0cdc0c7be0eb95f8745d86f Change-Id: I68b7eb686fa52f1851455160e1196f0748935c4c Reviewed-on: https://gerrit.libreoffice.org/36747 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Michael Stahl <mstahl@redhat.com>
* Related: tdf#106690 DOCX import: don't reduce auto-space accross cellsMiklos Vajna2017-04-202-0/+12
| | | | | | | | | | As suggested at <https://gerrit.libreoffice.org/#/c/36142/>, and it indeed matches the Word behavior. Change-Id: I1ba5b70fc5a7acab52fa4baf816e9f6cd2f913ba Reviewed-on: https://gerrit.libreoffice.org/36719 Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk> Tested-by: Jenkins <ci@libreoffice.org>
* loplugin:unreffun (clang-cl)Stephan Bergmann2017-04-191-0/+2
| | | | Change-Id: I406d2418413729479fd96f9b4918c3e0a2b03e9f
* tdf#107116 RTF import: fix missing upper and lower borders around textMiklos Vajna2017-04-192-0/+16
| | | | | | | | | | | | | | | | See commit 1be0a3fa9ebb22b607c54b47739d4467acfed259 (n#825305: writerfilter RTF import: override style properties like Word, 2014-06-17) for the context. Here the problem was that various details of the top border were removed during the style deduplication, but not the top border sprm itself. That was interpreted (correctly) by dmapper as "no border", rather than "inherit from style". Change-Id: I3dec8df789fc7b75fccfff91ce66f457fecd2f6e Reviewed-on: https://gerrit.libreoffice.org/36661 Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk> Tested-by: Jenkins <ci@libreoffice.org>
* Make the testTdf106974_int32Crop pass on my Mac, tooStephan Bergmann2017-04-171-1/+3
| | | | | | | | cf. 1391f3702a3daaefc67a8ee4b62ac38959db283f "Make the testTdf106974_int32Crop pass on my Mac". For some reason, on my Mac the Right value is 40474 resp. 40471 the two times the function gets called. Change-Id: I731ff9c5cf76be9d4817ad14f296807017d10dbd
* tdf#106742: OOXML import/export: treat "tblInd" properly.Vitaliy Anderson2017-04-175-35/+76
| | | | | | | | | | | Since MS Word 2013+ if you change cell margin at the table, the border won't be shifted. The decision is to do the same ( see https://bugs.documentfoundation.org/show_bug.cgi?id=106742 ). Change-Id: Ia58693c44f63ed21dca2cd99591002ba68927b65 Reviewed-on: https://gerrit.libreoffice.org/36084 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
* tdf#107025 advance only if grid is wider than font.Mark Hung2017-04-172-0/+24
| | | | | | | | | | | | | EvalGridWidthAdd decide distance to advance by subtracting the grid width and the height of the CJK font of default style. Text cluttered if the value is negative on Windows or break into multiple lines if it overflow on Linux. Change-Id: I085d420ef168238cde2eac3fb129020d5e5608da Reviewed-on: https://gerrit.libreoffice.org/36568 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Mark Hung <marklh9@gmail.com>
* tdf#106736 do not insert kern portion before tab portion.Mark Hung2017-04-142-0/+12
| | | | | | | | | | | Postion of SwTabPortion is set in constructor and is not expected to change later. If it is changed later, overflow will make the line full and push the content to the next line. Change-Id: I75fa9842c2c5bc0c2c16f9c5c17c43cdf88ea6ff Reviewed-on: https://gerrit.libreoffice.org/36061 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk>
* update cppunit to 1.14.0Markus Mohrhard2017-04-1416-618/+0
| | | | | | | Change-Id: I95fa42f4ef0580734b605df859c1660b29adb8b2 Reviewed-on: https://gerrit.libreoffice.org/36499 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
* Make the testTdf106974_int32Crop pass on my MacTor Lillqvist2017-04-131-1/+2
| | | | | | | | | | | For some reason, on my Mac the Right value is 41226, both times the function gets called. (On my Linux box, it is 46387 and 46394.) So compare to 41000 instead of 46000. Or would it be better to just bypass the test on macOS, as we already do for other tests in this file...? Change-Id: I71e292d42a2a956fbd28eb47fc5bfe3c7d849efe
* enum spelling: throught -> throughJustin Luth2017-04-123-7/+9
| | | | | | | | | | | | | git grep -l "[ _\.]THROUGHT" | xargs sed -i 's/THROUGHT/THROUGH/g' git grep -l -i "[ _\.]THROUGHT" | xargs sed -i 's/throught/through/g' In ENUMs: THROUGHT = THROUGH (preserved as valid alternate spelling) In ooxmlexport8 - unit test confirms THROUGH = THROUGHT Change-Id: Iae0fef9a8adcb96761989f38903a24ffb1b91e77 Reviewed-on: https://gerrit.libreoffice.org/35998 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
* add SAL_WARN_UNUSED to sw::Ring and friendsNoel Grandin2017-04-121-1/+0
| | | | | | | Change-Id: Ic90e4357089c1f487dd5738f9c14862ce95d611d Reviewed-on: https://gerrit.libreoffice.org/36408 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* sw: let getProperty fail more helpfullyMichael Stahl2017-04-111-2/+6
| | | | Change-Id: Ibca942235060c5b3a6215325e262bacbc464f2a4
* OFFICE-2102: ODF export: write <text:s> after non-mark elements tooMichael Stahl2017-04-112-0/+323
| | | | | | | | | | | | | | | | | | | | | | | ... and fix the wrong export for <text:meta-field>, which erroneously did not use the same bIsPrevCharSpace flag as everything else (which was obviously a bug, as there should be one flag per paragraph). Interop testing demonstrates that at least Word and Calligra Words differ in their interpretation of spaces following <draw:frame> and other shape elements, and <text:note>: if there is a U+0020 space before the element, LO will not collapse a U+0020 space behind the element but Word and Calligra Words do collapse it. The distinction between "mark elements" and "non-mark elements" in the whitespace collapsing implementation in OOo since the beginning was never explicitly spelled out in ODF, although listing the <text:bookmark> etc. elements in ODF 1.1, 5.1.1 was a strong hint. Fortunately all 3 applications agree that a <text:s> following a <draw:frame> is consumed as a space, and it is valid to write a <text:s> in this situation, so just do that to improve interoperability. Change-Id: I42260c0528db9fe7e87e8dbae5105aeadb83780d
* tdf#106970 DOCX import: don't collapse para auto space for different numsMiklos Vajna2017-04-112-0/+9
| | | | | | | | | | | | | | | | Commit 1bf7f6a1a50ee9f24a3687240fe6ae390b905a6b (tdf#106690 DOCX import: fix automatic spacing before/after numbered para block, 2017-04-04) made sure that autospacing is only collapsed in case the adjacent text nodes both have a numbering rule. It turns out there is an additional condition: even if both text nodes have a numbering rule, do the collapsing only in case they have the same numbering rule. Change-Id: Idb7a2b24d7eaa9094cc36f86b8a483045a33d028 Reviewed-on: https://gerrit.libreoffice.org/36400 Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk> Tested-by: Jenkins <ci@libreoffice.org>
* loplugin:mergeclasses merge SwAnnotationWin and SwSidebarWinNoel Grandin2017-04-111-1/+1
| | | | | | | Change-Id: Idaf7d7e8e4da37e0ba423dca3e22dc6711ba806a Reviewed-on: https://gerrit.libreoffice.org/36380 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* restore missing ooxmlimport unit test: n757890.docxJustin Luth2017-04-111-0/+20
| | | | | | | | | | | | | This test was lost while round-trippable tests were moved into ooxmlexport8. commit 086550313260d9fa45b91dc705b21bb9b51ce0b8 This test doesn't round-trip because it checks for a frame, and frames are turned into something else during export. Change-Id: If0e9776a83b0ee98c56039b4cd43b674198779e7 Reviewed-on: https://gerrit.libreoffice.org/36388 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Justin Luth <justin_luth@sil.org>
* tdf#106692 writerfilter: RTF import: fix \'0d in \leveltextMichael Stahl2017-04-071-0/+6
| | | | | | | | It's not a newline but yet another one of those bizarre RTF-encodings. (regression from 10e733908038407791f9c14af2a86417cc4a653c) Change-Id: I568050b031b95ac0b6ebfa1a0c39107e62f68bed
* tdf#106974 docx export: Crop is "long", not sal_Int16Justin Luth2017-04-062-0/+10
| | | | | | | | | | | | I got size sal_Int16 from the return value type of the border spacing, but somehow failed to lookup the return value of GraphicCrop. It now matches .doc export's sal_Int32. Bad mistake: regression 8eff1decd91cbfb10094c25d4cf1d2b434a4da72 Change-Id: Ie149630b9da9a067de319149f23ca21f78a186cf Reviewed-on: https://gerrit.libreoffice.org/36200 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Michael Stahl <mstahl@redhat.com>
* Improved loplugin:redundantcast, static_cast on arithmetic types: swStephan Bergmann2017-04-062-4/+4
| | | | Change-Id: I1d52df4fe3f276ebf017aa7cfb7b313cefaf0844
* loplugin:unreffun (clang-cl)Stephan Bergmann2017-04-063-3/+25
| | | | Change-Id: Ifd5522ce1be31f9c9bd432ccf40a0c971063c8da
* tdf#106690 DOCX import: fix automatic spacing before/after numbered para blockMiklos Vajna2017-04-042-0/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | The context is text nodes with automatic before/after spacing and numbering rules set, like: A * B * C * D E The correct behavior seems to be (though I haven't found this explicitly written in the OOXML spec) to drop spacing between B and C and C and D, but not before B and not after D. Originally no spacing was dropped, then commit c486e875de7c8e845594f5043a37ee8800865782 (tdf#95031 DOCX import: auto spacing inside numbering means no spacing, 2016-10-18) removed spacing around all B/C/D. Fix the problem by checking the numbering rules and automatic after spacing of the previous paragraph, so spacing before B and after D is not removed. Change-Id: Icbdb36e31057ab0e8ac033888cf5cc7c52dad5d0 Reviewed-on: https://gerrit.libreoffice.org/36062 Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk> Tested-by: Jenkins <ci@libreoffice.org>
* sw: indentation fixesMiklos Vajna2017-04-042-18/+108
| | | | | | | Nested namespaces were always declared in new lines in these files, let's keep the consistency. Change-Id: I1dcfdd2b1f54ec56b3554f3c0095513cec33e49c
* make UNO enums scoped for internal LO codeNoel Grandin2017-04-042-108/+18
| | | | | | | | | | | | | | | | | | | | | | | | this modifies codemaker so that, for an UNO enum, we generate code that effectively looks like: #ifdef LIBO_INTERNAL_ONLY && HAVE_CX11_CONSTEXPR enum class XXX { ONE = 1 }; constexpr auto ONE = XXX_ONE; #else ...the old normal way.. #endif which means that for LO internal code, the enums are scoped. The "constexpr auto" trick acts like an alias so we don't have to use scoped naming everywhere. Change-Id: I3054ecb230e8666ce98b4a9cb87b384df5f64fb4 Reviewed-on: https://gerrit.libreoffice.org/34546 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* sw: indentation fixesMiklos Vajna2017-04-032-18/+108
| | | | Change-Id: I81076414ee335f34bb687f93e89d65343d2f3c57
* use actual UNO enums in swNoel Grandin2017-04-0315-0/+498
| | | | | | | Change-Id: If0f4a6532cc255f632d88d97e6b1a9e57462f5fc Reviewed-on: https://gerrit.libreoffice.org/35969 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* tdf#106849: don't reduce quality of picture in bullet list.Vitaliy Anderson2017-03-311-1/+2
| | | | | | | | | | | | | | | An image saved in cache when NS_ooxml::LN_CT_NumPicBullet_pict property is processed should not to be resized. It reduce quality the image. Just set the property "GraphicSize" of current level of the NumRule instead of resize the image when processing NS_ooxml::LN_CT_Lvl_lvlPicBulletId property. Change-Id: I8ac80643decb7794de7a295cc7c2895a5bd24e2d Reviewed-on: https://gerrit.libreoffice.org/35592 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk>
* tdf#82580 tools: rename Rectangle to tools::RectangleMiklos Vajna2017-03-312-9/+9
| | | | | | | | | | | | | | | Mostly generated using make check COMPILER_EXTERNAL_TOOL=1 CCACHE_PREFIX=clang-rename-wrapper RENAME_ARGS="-qualified-name=Rectangle -new-name=tools::Rectangle" Except some modules have their own foo::tools namespace, so there have to use ::tools::Rectangle. This commit just moves the class from the global namespace, it does not update pre/postwin.h yet. Change-Id: I42b2de3c6f769fcf28cfe086f98eb31e42a305f2 Reviewed-on: https://gerrit.libreoffice.org/35923 Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk> Tested-by: Jenkins <ci@libreoffice.org>
* tdf#106701 tabOverMargin: only affect specified tabstops, not autotabJustin Luth2017-03-312-0/+15
| | | | | | | Change-Id: I15bf3a8555a152cab90380524b4a968f9f95fc37 Reviewed-on: https://gerrit.libreoffice.org/35924 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Justin Luth <justin_luth@sil.org>
* tdf#105975 Add Formula field parsing (docx) in SWriterJean-Sebastien Bevilacqua2017-03-302-0/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Introduction ------------ In MSWord, you can create a formula field (starting with =). When you save your file as `docx`, this `FORMULA` field is registered in you file (a field starting with `=`). In its current state, LibreOffice can't parse the `FORMULA` field in `docx` file. Context of this fix ------------------- This fix is entirely located in the `DomainMapper_Impl.cxx` file because it's where the parsing is done. How this fix works ------------------ First, we add `FORMULA` support by adding it to the `aFields[]` variable. Next, to handle the `FORMULA` constant, we add a condition (swith case) in `DomainMapper_Impl::CloseFieldCommand()` to call `handleFieldFormula`. Note ---- In function `lcl_ExtractToken`, if command starts with `=`, it's a `FORMULA` field. Change-Id: If7d25de5413aa3133b22523d8a3f34ab6961adfc Reviewed-on: https://gerrit.libreoffice.org/34334 Reviewed-by: Michael Stahl <mstahl@redhat.com> Tested-by: Jenkins <ci@libreoffice.org>
* Related: tdf#105454 DOCX import: fix unwanted enabled-by-default kerningMiklos Vajna2017-03-302-0/+7
| | | | | | | | | Similar to DOC, DOCX doesn't enable kerning by default. Change-Id: I070ff5f0d43c27107593d629a1ad681d29d2038c Reviewed-on: https://gerrit.libreoffice.org/35890 Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk> Tested-by: Jenkins <ci@libreoffice.org>
* tdf#106606: Don't dispose NumPicBullet in sprm handler.Vitaliy Anderson2017-03-302-0/+25
| | | | | | | | | | | | | | | | | | | An image saved in cache once when NS_ooxml::LN_CT_NumPicBullet_pict is processed, may then be used multiple times (for each NumRule that requires it) when NS_ooxml::LN_CT_Lvl_lvlPicBulletId is processed for each of them. If the image was released here for first processing, subsequent rules couldn't find the image in cache and failed to create NumberingType::BITMAP style for the rule. The image is ultimately released in ListsManager::~ListsManager() after it is no more needed. Change-Id: Ib4c351437ba94d5a9d3e2927ccf459ec01f1b15f Reviewed-on: https://gerrit.libreoffice.org/35591 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk>
* tdf#106694 RTF import: fix missing paragraph tab positionMiklos Vajna2017-03-282-0/+18
| | | | | | | | | | | | | | | The problem here was that while in general paragraph style / direct formatting deduplication is supposed to happen in the tokenizer, paragraph tab positions is an exception, and dmapper expects to see the duplicated tokens. Fix the problem by introducing a blacklist that contains tokens not to deduplicate. Change-Id: I1cca53e99cfdb082df389ff295f3447cc8f9d3b8 Reviewed-on: https://gerrit.libreoffice.org/35790 Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk> Tested-by: Jenkins <ci@libreoffice.org>
* tdf#105975 Add Set field parsing (docx) in LibreOffice WriterJean-Sebastien Bevilacqua2017-03-283-1/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Introduction ------------ In MSWord, you can create a variable with `SET` field and then reference it later in a formula. When you save your file as `docx`, this `SET` field is registered in you file. In its current state, LibreOffice can't parse the `SET` field in `docx` file. Context of this fix ------------------- This fix is entirely located in the `DomainMapper_Impl.cxx` file because it's where the parsing is done. How this fix works ------------------ First, we add `SET` support by adding it to the `aFields[]` variable. Next, to handle the `SET` constant, we add a condition (swith case) in `DomainMapper_Impl::CloseFieldCommand()` to call `handleFieldSet`. Finally, `handleFieldSet` works like `handleFieldAsk` with small differences. Note ---- I have renamed `lcl_ExctractAskVariableAndHint` to `lcl_ExctractVariableAndHint` because this function is used for both `ASK` and `SET` fields. Change-Id: I2bf948e26e8506ac151d1d0bc8556721bbe0392b Reviewed-on: https://gerrit.libreoffice.org/34333 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk>
* tdf#106799 WW8: Accept TTP Mark without Cell MarkMike Kaganski2017-03-272-0/+15
| | | | | | | | | | | | | Paragraph mark of inner table (0x0D) sometimes has sprmPFInnerTtp, but no sprmPFInnerTableCell. This still counts as cell end (at least, MS Word treats it that way). Unit test included. Change-Id: I5589cdd486c03ca4567d61882826cc7c245a40c9 Reviewed-on: https://gerrit.libreoffice.org/35763 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
* Indentation fixesMiklos Vajna2017-03-271-1/+2
| | | | Change-Id: Idbd31b84f252abdab1787945c4964173ad5765ba
* Fix typosAndrea Gelmini2017-03-256-6/+6
| | | | | | | Change-Id: Ia87318cb323d403cdff947da0b70e0d2aabaacd4 Reviewed-on: https://gerrit.libreoffice.org/35657 Reviewed-by: Julien Nabet <serval2412@yahoo.fr> Tested-by: Julien Nabet <serval2412@yahoo.fr>
* fix tb71 buildNoel Grandin2017-03-241-1/+1
| | | | | | | | | | after commit 7916487cf4d9603cdbe4c7ffbe9bb3f28b51ce4e "convert ViewShellId to o3tl::strong_int" Change-Id: Ie7b8ac51a94c3b9e6d39e397aa3dd5cb42acc6d0 Reviewed-on: https://gerrit.libreoffice.org/35623 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* convert ViewShellId to o3tl::strong_intNoel Grandin2017-03-241-2/+7
| | | | | | | Change-Id: I45553d11d56aa8c4432aec126ca51f24bd3ead09 Reviewed-on: https://gerrit.libreoffice.org/35421 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* tdf#106492 DOCX export: fix duplicated section break at doc endMiklos Vajna2017-03-212-0/+7
| | | | | | | | | | | | There is always a section break at the end of a DOCX document, so in the rare case when there is no text node between the end of a Writer section and the end of the document make sure no duplicated section break is written. Change-Id: I3fcd852c1a63e2d707822ad08603e2d37386e439 Reviewed-on: https://gerrit.libreoffice.org/35499 Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk> Tested-by: Jenkins <ci@libreoffice.org>