summaryrefslogtreecommitdiff
path: root/xmloff
AgeCommit message (Collapse)AuthorFilesLines
2016-02-18[API CHANGE] add Author Initials to XAnnotationCaolán McNamara2-0/+19
and use the users preferred initials for impress annotations i.e. the ones entered in the tools->options and save and load those to/from odf similarly to what we do for writer annotation initials Change-Id: Iadc0e994bfaf58632ce25b8f7cdc737580ee97bc Reviewed-on: https://gerrit.libreoffice.org/22143 Reviewed-by: Michael Stahl <mstahl@redhat.com> Tested-by: Michael Stahl <mstahl@redhat.com>
2016-02-11[API CHANGE] add operator==/!= to UNO structsNoel Grandin1-7/+0
this is useful now that we are storing UNO structs in std::vector Change-Id: Ic558bcd669bd2b3cdf9eb8393269eb906ac52369 Reviewed-on: https://gerrit.libreoffice.org/22257 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-02-09Remove excess newlinesChris Sherlock93-335/+0
A ridiculously fast way of doing this is: for i in $(pcregrep -l -M -r --include='.*[hc]xx$' \ --exclude-dir=workdir --exclude-dir=instdir '^ {3,}' .) do perl -0777 -i -pe 's/^ {3,}/ /gm' $i done Change-Id: Iebb93eccbee9e4fc5c4380474ba595858a27ac2c Reviewed-on: https://gerrit.libreoffice.org/22224 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Chris Sherlock <chris.sherlock79@gmail.com>
2016-02-09Formatting changes across all modulesChris Sherlock6-17/+17
+ Removed comment cruft + Tab formatting in number of files + Some commented out code removed + Tab characters replaced with spaces + Newline cleanup in quite a few files + Tweak header guard #endifs Change-Id: I3208ff2f047da890edcc49b73389aca22442f5fc Reviewed-on: https://gerrit.libreoffice.org/22221 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Chris Sherlock <chris.sherlock79@gmail.com>
2016-02-05Fix typosAndrea Gelmini1-1/+1
Change-Id: Ice72f8d9971e15dd6ef365e64cd567b8581a92d3 Reviewed-on: https://gerrit.libreoffice.org/21797 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Ashod Nakashian <ashnakash@gmail.com>
2016-02-04Replace push(std::make_tuple(...)) with emplace(...)Stephan Bergmann1-4/+4
...the former reportedly caused problems now with GCC 6 libstdc++ Change-Id: I4b13adc71d9e00b14df1c8e875dba9e90cbcdcc7
2016-02-03remove bogus nullprt checks for pContextJochen Nitschke1-11/+2
reported by Cppcheck Change-Id: I737c2bd7be197ef9d0b29b922acd8021fefb7af6 Signed-off-by: Jochen Nitschke <j.nitschke+logerrit@ok.de> Reviewed-on: https://gerrit.libreoffice.org/22059 Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com> Tested-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
2016-02-02xmloff: tdf#96147: ODF export: fix duplicate fo:background-colorMichael Stahl2-2/+12
... attributes that happen if both CharHighlight and CharBackColor properties are used, because the CharBackTransparent property wasn't taken into account, and combining the CharBackColor and CharBackTransparent properties happens *after* XMLTextExportPropertySetMapper::ContextFilter() runs. Also, it looks like a transparent highlight wouldn't export properly but apparently DomainMapper::getColorFromId() won't create such. (regression from f880962f5bf26bfaef06bd3f9e67e2d901a2e74c) Change-Id: Ib628ef8bb377482f74fadb97c81afb95fbbf7184
2016-01-31boost::tuple -> std::tupleCaolán McNamara4-24/+23
Change-Id: Ia08c184b792ff73d3ed49d6a5d54dec1a35943e9 Reviewed-on: https://gerrit.libreoffice.org/21955 Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2016-01-31unnecessary use of OUString constructorNoel Grandin3-5/+5
Change-Id: Idd31b0a53c8318af69bbcd32f6798721ec8eb8e1 Reviewed-on: https://gerrit.libreoffice.org/21945 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-01-21Move XMLMetaExportComponent_get_implementation to a more reasonable placeStephan Bergmann1-8/+8
Change-Id: Iaf3dc5351453b4aee8ffc1e7fb1bc670c45c507f
2016-01-21cppcheck: variableScopeCaolán McNamara1-2/+2
Change-Id: Id7cf5887c1ef0e0c0aad72ea5ac49f4d3db5065e
2016-01-19Resolves: tdf#96245 relax test for duplicate frame namesCaolán McNamara1-1/+2
to allow duplicated unnamed frames Change-Id: I871849b23d01209f027b6295777db705f7dc86d8
2016-01-18Convert NULL/0 to nullptrPeter Foley1-3/+3
Change-Id: Iee6bb0867afd8aebe36e62b507f1332e3a581838 Reviewed-on: https://gerrit.libreoffice.org/21549 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Michael Stahl <mstahl@redhat.com>
2016-01-18loplugin:unreffunStephan Bergmann1-19/+0
Change-Id: If023e3916c7d2fa7f22958304606a2eec418fcf7
2016-01-18tdf#74608 xmloff: Constructor feature for XMLMetaExportComponentkripton4-12/+11
Change-Id: I3edde1291e9dec21420238edb5a05f25849ebfff Reviewed-on: https://gerrit.libreoffice.org/19792 Reviewed-by: Matúš Kukan <matus@libreoffice.org> Tested-by: Matúš Kukan <matus@libreoffice.org>
2016-01-15-Werror=unused-but-set-variableNoel Grandin2-31/+7
Change-Id: I981e1b660dc94333935b3a451847198940e47e99
2016-01-15loplugin:unusedmethods unused return value in include/xmloffNoel Grandin10-65/+20
Change-Id: I6615a467152a50e9c65f04fd0d6c8cc128517bdb
2016-01-13-Werror=addressStephan Bergmann1-4/+1
the original looks odd, but the code was like that ever since it got introduced with 44cfc7cb6533d827fd2d6e586d92c61d7d7f7a70 "re-base on ALv2 code. Includes (at least) relevant parts of:" Change-Id: I327c5a53a2634aca1b36367ee09c068ac610d3f4
2016-01-13-Werror=unused-but-set-variableNoel Grandin1-4/+0
Change-Id: I61a07db63cc8a6bad159ca455f92d604c5e78edc
2016-01-13loplugin:unusedmethods unused return value in xmloff/Noel Grandin9-53/+13
Change-Id: Ia5a3716ca4f7eb987092837ac83a415e5c0730df
2016-01-12xmloff: SvXMLExport::mbExtended is dead nowMichael Stahl1-3/+0
Change-Id: Ie679259deccce87949d7bf8767af707827b4134c
2016-01-12xmloff: there is no reason why office:version should be omittedMichael Stahl1-15/+14
... if a document contains foreign elements or attributes. In fact since ODF 1.2 the office:version attribute is mandatory and any document that omits it is therefore invalid, while "extended conforming" documents are allowed to contain foreign elements and attributes. This mysterious check was there since CVS initial import, but without any justification. Change-Id: Ifeafad2b7af41f06356461adb7ae65dbf7bae11d
2016-01-12typosMichael Stahl1-1/+1
Change-Id: I970429c266cc1a8cf36cfba379b447899b5dbce9
2016-01-11tdf#40517 Export/Import in ODF decimal replacement with spaceLaurent Balland-Poirier2-8/+27
Number format such as "0.???" is saved in ODF with number:decimal-replacement=" " Change-Id: I0dd19b5ea126f380bcf17a3ccc5d8c355119e23b Reviewed-on: https://gerrit.libreoffice.org/21021 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: jan iversen <jani@documentfoundation.org>
2016-01-10Fix typosAndrea Gelmini19-22/+22
Change-Id: I9a5940027423ff0791fa7da0b79b617412ce6b86 Reviewed-on: https://gerrit.libreoffice.org/21209 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Ashod Nakashian <ashnakash@gmail.com>
2016-01-08tdf#96480: ODF import: eliminate duplicate cross reference heading bookmarksMichael Stahl5-12/+141
7c3c3006deaaaf1bb3f2f4eeeaf11da3bcebe53c is apparently worse than it appeared at first glance since there are numerous assumptions about bookmarks, such as that if they were inserted successfully they may be copied successfully, which isn't the case for duplicate cross reference bookmarks. So fix this differently, by eliminating the duplicates and mapping all reference fields to refer to the surviving bookmark. It was not possible to do this in SwXBookmark by checking the makeMark() return as that would raise interesting problems such as it's currently guaranteed to have 1:1 SwXBoomarks to core Marks so we can't just connect 2 SwXBookmarks to the same core Mark, and we also can't leave the SwXBookmark unconnected after attach. Another alternative would be to temporarily allow inserting the duplicate bookmarks and then eliminate them after the import, but what is implemented now is to check from xmloff for duplicates, which is reasonably simple. Change-Id: I7ee4854d1c9d8bf74201089cbb7287b1bd8ee3b9
2016-01-08indentation fixesMarkus Mohrhard1-91/+92
Change-Id: Ibc652519d7ccc9a1ec483f19d443c9b5303e0f92
2016-01-05tdf#94205: Use o3tl::make_unique insted of new+std::move.Sheikha AL-Hinai1-4/+2
Change-Id: I9ff14760479d2ac882546e2e5b74ab750ba2fa4b Reviewed-on: https://gerrit.libreoffice.org/20984 Reviewed-by: Stephan Bergmann <sbergman@redhat.com> Tested-by: Stephan Bergmann <sbergman@redhat.com>
2015-12-29cppcheck: noExplicitConstructorCaolán McNamara1-1/+1
Change-Id: Ib43e53d5b6c9c130adb765ac9b769f58060ac640
2015-12-26xmloff: indent cleanup in headerChris Sherlock1-1/+2
Change-Id: Ibcd03dcdc8936ce95feada20c72a358d5021a1d9 Reviewed-on: https://gerrit.libreoffice.org/20962 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Chris Sherlock <chris.sherlock79@gmail.com>
2015-12-26xmloff: change instances of maName to more descriptive namesChris Sherlock2-4/+4
Change-Id: I7ff771f78481677ae65f933608e484714cd43f4f Reviewed-on: https://gerrit.libreoffice.org/20961 Reviewed-by: Chris Sherlock <chris.sherlock79@gmail.com> Tested-by: Chris Sherlock <chris.sherlock79@gmail.com>
2015-12-22tdf#96181 SVG: Add support for <use> elementXisco Fauli1-0/+1
Change-Id: Ia95c7e35158f8a4c6a597662524c74e1909983bb Reviewed-on: https://gerrit.libreoffice.org/20483 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Armin Le Grand <Armin.Le.Grand@cib.de>
2015-12-22loplugin:unusedfields in include/vclNoel Grandin1-1/+0
Change-Id: I61fc04e00970ea0d398ff80f6b1f81ec2f62dd5e Reviewed-on: https://gerrit.libreoffice.org/20868 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2015-12-21loplugin:unusedfields in xmlhelp,xmloffNoel Grandin11-46/+4
Change-Id: I4d2f0ff95da5abaa3d0a0a5223616e09e0c476fc
2015-12-19xmloff: one more missing includeMiklos Vajna1-0/+1
Change-Id: I92080eefa485dec86c3a10f978831157b04a0c53
2015-12-19coverity#1343637 Double freeCaolán McNamara3-21/+14
and coverity#1343638 Double free coverity#1343639 Double free coverity#1343640 Double free coverity#1343641 Double free Change-Id: I8ce42f03d40fe514f73b3a7eabdb4f323b2239e6
2015-12-17Avoid memory leaksStephan Bergmann5-6/+9
...when exceptions get thrown while a context owns an rewind map Change-Id: I90d26caa57ef2ff508e6eae05ddbc62cdb3f01ea
2015-12-17Remove trivial class XMLTransformerContextVectorStephan Bergmann7-70/+34
Change-Id: I140614895d5727ffd97681ddd06bf7439fe7a443
2015-12-17Hold XMLTransformerContext by rtl::ReferenceStephan Bergmann35-158/+145
Change-Id: I60b8d422d53d47537996756aa1b7fb3c26f8383a
2015-12-17tdf#32248 SVG: first attempt to support tspan elementsXisco Fauli1-0/+1
it's not perfect but at least we can already display the text, which is better than displaying nothing Change-Id: I89627010ac8071c106b41c1de6b183a619f8158b Reviewed-on: https://gerrit.libreoffice.org/20642 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Armin Le Grand <Armin.Le.Grand@cib.de>
2015-12-11Resolves: tdf#93994 flush API objects at load timeArmin Le Grand1-0/+24
When importing SdrObjCustomShape for each an Outliner and a VirtualDevice as RefDevice are created and held. On 32bit systems this breaks the import with many such objects (in the bugdoc around 4300). On 64bit it works, but more memory as necessary is used. To avoid this, flush the UNO API implementations at these objects after one page is imported. Do not do this for each single object, that will break the connector import which does some processing later. Change-Id: I7296edd40f01c1a40258808d78bf39e6d4fe29bf Reviewed-on: https://gerrit.libreoffice.org/19990 Reviewed-by: Thorsten Behrens <Thorsten.Behrens@CIB.de> Tested-by: Thorsten Behrens <Thorsten.Behrens@CIB.de>
2015-12-10loplugin:nullptr: More NULL -> nullptr automatic rewriteStephan Bergmann1-1/+1
Change-Id: Ie83819e2bcdc5fa160b39296b005ca9a5ff74b1d
2015-11-26-Werror,-Wunused-private-fieldStephan Bergmann4-7/+2
Change-Id: I3f328023d99d3505ab2ce2a80eef82d84d81446e
2015-11-26mark UNO structs as SAL_WARN_UNUSED, where possibleNoel Grandin2-3/+0
Change-Id: Ie3de518f60c9f1313c68df54dbdc1fb2804f1f0d
2015-11-26loplugin:unusedfields variousNoel Grandin1-1/+0
Change-Id: I18f94269a1172cf195ee402384f7144610e1e82d
2015-11-26com::sun::star->css in xmloff/Noel Grandin353-2892/+2437
Change-Id: Iee6e21e093fb1383f19a0716dbc1c341f446b8c2 Reviewed-on: https://gerrit.libreoffice.org/20192 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2015-11-25loplugin:unusedfields in include/xmloffNoel Grandin2-11/+1
Change-Id: I979743671e9a2e868d815f917427588b7b0320ba
2015-11-25store custom icon set info into ODFMarkus Mohrhard1-0/+3
Change-Id: I1581b28237d2b9f9c22d3f3b0436dc281508aafa Reviewed-on: https://gerrit.libreoffice.org/20118 Reviewed-by: Katarina Behrens <Katarina.Behrens@cib.de> Tested-by: Katarina Behrens <Katarina.Behrens@cib.de>
2015-11-24crashtesting: ooo72999-1.ods NaNs found in light directionCaolán McNamara1-1/+10
and NaN != NaN so busted logic propogates, drop invalid directions on initial read Change-Id: Ic76c714666df14d37a4c68f43b817327675bd0e0