summaryrefslogtreecommitdiffstats
path: root/include/sax
Commit message (Collapse)AuthorAgeFilesLines
* loplugin:override: No more need for the "MSVC dtor override" workaroundStephan Bergmann2016-09-132-2/+2
| | | | | | | | | | | | | | | | | The issue of 362d4f0cd4e50111edfae9d30c90602c37ed65a2 "Explicitly mark overriding destructors as 'virtual'" appears to no longer be a problem with MSVC 2013. (The little change in the rewriting code of compilerplugins/clang/override.cxx was necessary to prevent an endless loop when adding "override" to OOO_DLLPUBLIC_CHARTTOOLS virtual ~CloseableLifeTimeManager(); in chart2/source/inc/LifeTime.hxx, getting stuck in the leading OOO_DLLPUBLIC_CHARTTOOLS macro. Can't remember what that isAtEndOfImmediateMacroExpansion thing was originally necessary for, anyway.) Change-Id: I534c634504d7216b9bb632c2775c04eaf27e927e
* GSOC - Handling namespace declaration missing case:Mohammed Abdul Azeem2016-07-191-2/+8
| | | | | | | | | | | | initialization parameter to FastParser will turn off the namespace declaration missing exception. Test cases have also been given to verify the same. Change-Id: I4c3e02c7ad92d50e279f895ced53c78fc8f49b91 Reviewed-on: https://gerrit.libreoffice.org/27278 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Michael Meeks <michael.meeks@collabora.com> Tested-by: Michael Meeks <michael.meeks@collabora.com>
* GSOC: Adapt XLegacyFastParser to function like XParser.Mohammed Abdul Azeem2016-07-091-0/+1
| | | | | | | | | | | | Made XFastParser to pass namespace prefix instead of URI for Unknown attributes and elements, Namespace handler is provided to resolve those. Test for XFastParser unknown elements is removed, since testing XLegacyFastParser indirectly tests that also. Change-Id: Ia41ff5d3d4c07cef0ca23ba858bfb2a94b91b1f5 Reviewed-on: https://gerrit.libreoffice.org/26982 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Michael Meeks <michael.meeks@collabora.com>
* loplugin:passstuffbyref also for {css::uno,rtl}::ReferenceStephan Bergmann2016-07-071-2/+2
| | | | Change-Id: If18f0a749c1e474c302fd797c2a2abc2b33a0730
* update unusedmethods plugin to deal with constructorsNoel Grandin2016-05-181-6/+0
| | | | | | | | | | and fix the operator< implementations in some of the other plugins too. Change-Id: Ie5631e0cdc8d2a994ad2af2533cdb558a6cfc035 Reviewed-on: https://gerrit.libreoffice.org/25057 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* loplugin:passstuffbyref in saxNoel Grandin2016-04-131-2/+2
| | | | Change-Id: I95f0a17b9b8bab592a60f6b5223b4668c5355275
* clang-tidy performance-unnecessary-value-param in saxNoel Grandin2016-04-111-2/+2
| | | | Change-Id: I46aad7b0a96210e5f3f278c3f6f3a945825b7cae
* Remove excess newlinesChris Sherlock2016-02-091-1/+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>
* loplugin:unusedmethodsNoel Grandin2016-01-201-2/+0
| | | | | | | Change-Id: Ib2dc804f55d1a96ae01f2cf31b7d55956040a72e Reviewed-on: https://gerrit.libreoffice.org/21603 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* use unique_ptr for pImpl in sax/Noel Grandin2015-11-181-1/+2
| | | | Change-Id: I0bef03451437cbdc5b0fed6b67690ac1d547291d
* loplugin:unnecessaryvirtualNoel Grandin2015-11-171-4/+4
| | | | | | | | | | update the plugin with lessons learned from the mergeclasses plugin and re-run it Change-Id: I9d622eb3d05fceaf8fa764c533c8fa5dfb4c7711 Reviewed-on: https://gerrit.libreoffice.org/20015 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* loplugin:nullptr (automatic rewrite)Stephan Bergmann2015-11-102-3/+3
| | | | Change-Id: I54bf272b404d2302cafbde73ec5061ea2cd966ab
* com::sun::star->css in include/linguistic to include/sfx2Noel Grandin2015-10-235-90/+90
| | | | | | | Change-Id: Id69e293fda98ee6cf2cc3d3296a0cd2e06bd847e Reviewed-on: https://gerrit.libreoffice.org/19527 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* Replace "SAL_OVERRIDE" with "override" in LIBO_INTERNAL_ONLY codeStephan Bergmann2015-10-123-39/+39
| | | | Change-Id: I2ea407acd763ef2d7dae2d3b8f32525523ac8274
* xmloff: fix ODF import of gradient draw:angle attribute a bitMichael Stahl2015-10-091-0/+6
| | | | | | | | | | | | | | | | ODF 1.2 part 3, 18.3.1 angle, says "An angle, as defined in §4.1 of [SVG]" and "If no unit identifier is specified, the value is assumed to be in degrees." Unfortunately OOo could only read and write 10th of degree here. See also https://issues.oasis-open.org/browse/OFFICE-3774 As the first step towards fixing that, implement the import for draw:angle values with an angle unit identifier, but leave the import as-is if the angle identifier is missing. Change-Id: Ib88d417c03998ebcfc569b01492f0e1f851bbc85
* sax: convert MergeMarksEnum to enum classMichael Stahl2015-07-181-2/+2
| | | | | | This should detect if a MergeMarks value is erroneously passed as tag. Change-Id: I7b855a661f182136824cf25f2174a9bcce8ff3d1
* sax, sw: try to make that maMarkStack easier to understandMichael Stahl2015-07-181-2/+4
| | | | | | | | | In DocxAttributeOutput it's not at all obvious which mark() is supposed to be ended by which mergeTopMarks(), so add an extra parameter to the FastSaxSerializer functions and verify with an assertion that a LIFO order is maintained. Change-Id: I5a421e2fb11f15343147417fe0b9b23642c70721
* Fix typosAndrea Gelmini2015-06-161-1/+1
| | | | | | | Change-Id: I549635318f46718042e50c8f89ce3c620cade990 Reviewed-on: https://gerrit.libreoffice.org/16281 Reviewed-by: Michael Stahl <mstahl@redhat.com> Tested-by: Michael Stahl <mstahl@redhat.com>
* remove unnecessary use of void in function declarationsNoel Grandin2015-04-151-7/+7
| | | | | | | | | | | | | | | | ie. void f(void); becomes void f(); I used the following command to make the changes: git grep -lP '\(\s*void\s*\)' -- *.cxx \ | xargs perl -pi -w -e 's/(\w+)\s*\(\s*void\s*\)/$1\(\)/g;' and ran it for both .cxx and .hxx files. Change-Id: I314a1b56e9c14d10726e32841736b0ad5eef8ddd
* loplugin:staticfunctionNoel Grandin2015-03-271-1/+1
| | | | Change-Id: I10c30ef28e7321882a720d1e7869a75a89febfc5
* V801: Decreased performanceCaolán McNamara2015-03-091-1/+1
| | | | Change-Id: I9e9a00acf9503980f0c6c7cd0a20378e5ca7390d
* boost::foo_ptr->std::foo_ptrCaolán McNamara2015-02-161-3/+3
| | | | Change-Id: I9219619b538b6530a89f5932ac51eb3b62eb396a
* make FastAttributeList's unknown attribute value be passed as OStringLuboš Luňák2014-11-121-5/+4
| | | | | | | Since that's how it's internally stored anyway, and I have a use case where it's useful to limit the length by passing it to OString ctor. Change-Id: I5903ea4f1b2cdb48c1bbceac6b8e21eb5882d377
* allow inserting attributes that have zero lengthLuboš Luňák2014-11-121-1/+2
| | | | | | | Otherwise the strlen() might give an incorrect length if the attribute value is just a part of a longer string. Change-Id: I67eb7baecfa928fdee26c5ea9003bd7fc9b96d59
* typo: represant->representAndras Timar2014-11-121-1/+1
| | | | Change-Id: If783e9338644a97206d99864ed19807e5fb3882d
* Remove few pointless OUString::number() and one methodMatúš Kukan2014-10-231-1/+0
| | | | Change-Id: I3e9a302a7513eebfeff07402f71fc3dde22e4cc2
* FastSerializer: Use faster TokenValue struct when possibleMatúš Kukan2014-10-231-1/+0
| | | | | | Saves another ~100m pcycles for 650k calls in startElementInternal() Change-Id: I190326edc7feffb900e91fa7e5c3530b5b267f59
* FastSerializer: avoid some more OStringsMatúš Kukan2014-10-231-3/+1
| | | | Change-Id: I2d5dbe9adccdd231cc16a1f83a90a4adeb965c64
* FastSerializer: Use FastAttributeList directly to write faster.Matúš Kukan2014-10-232-4/+3
| | | | Change-Id: I28085d4e060bcf052e6aa97a0822a4d653d7c066
* fastparser: avoid allocating un-used FastTokenLookup class.Michael Meeks2014-10-231-18/+18
| | | | | | | | This contained an rtl_Sequence complete with horror internal allocator, caught red-handed serializing threaded loading to no good purpose. Change-Id: I837b2c17e4f70fd6a49bed33ad74a7d79f98f35c
* FastSerializer: Have OString version of write() too.Matúš Kukan2014-10-231-0/+1
| | | | | | And remove characters() member function. Change-Id: Ifcedbb6d969b7b057ff378d2fbce09c2dde5ac18
* fdo#80748: Fix for file corruption due to framePrRohit Deshmukh2014-07-031-1/+1
| | | | | | | | | | -For framePr Unit test is there in ooxmlexport.cxx -http://opengrok.libreoffice.org/xref/core/sw/qa/extras/ooxmlexport/ooxmlexport.cxx#3578 Change-Id: Ic1fd4c7a863d8ab57c6495e20bff6863eb941688 Reviewed-on: https://gerrit.libreoffice.org/10045 Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk> Tested-by: Miklos Vajna <vmiklos@collabora.co.uk>
* css already means ::com::sun::starThomas Arnhold2014-05-271-2/+2
| | | | Change-Id: I48d7b746d7cc49246743c62480b0d225d2519342
* Find places where uno::Sequence is passed by value.Noel Grandin2014-05-141-1/+1
| | | | | | | | | | Implement a clang plugin to find them, and clean up existing code to pass them by reference. Change-Id: If642d87407c73346d9c0164b9fc77c5c3c4354b8 Reviewed-on: https://gerrit.libreoffice.org/9351 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
* remove unnecessary scope qualifier from sal_Bool usesNoel Grandin2014-04-031-1/+1
| | | | | | i.e. convert "::sal_Bool" to "sal_Bool" Change-Id: Ie5943aee4fee617bf2670655558927ed25b7e067
* First batch of adding SAL_OVERRRIDE to overriding function declarationsStephan Bergmann2014-03-263-39/+39
| | | | | | | ...mostly done with a rewriting Clang plugin, with just some manual tweaking necessary to fix poor macro usage. Change-Id: I71fa20213e86be10de332ece0aa273239df7b61a
* sax: FastAttributeList - add attribute in a specific namespaceTomaž Vajngerl2014-03-051-0/+1
| | | | | | | | | | | This change is inspired by startElementNS & singleElementNS which have namespace as first parameter to define the namespace of an element. Some attributes of a element can be in different namespace but until now FastAttributeList "add" method did not have a namespace parameter. This commit adds "addNS" which accepts namespace as a first parameter. Change-Id: Iebf8b5e890c17f00a56923efc3506580eed070a9
* Remove visual noise from includeAlexander Wilms2014-03-011-1/+1
| | | | | | | | | | | | | | | Conflicts: include/framework/preventduplicateinteraction.hxx include/sfx2/sfxbasecontroller.hxx include/sfx2/sfxbasemodel.hxx include/toolkit/awt/vclxtabpagemodel.hxx include/vcl/field.hxx include/vcl/settings.hxx Change-Id: Ibccf9f88c68267a3d7e656012b51eaf644c418c2 Reviewed-on: https://gerrit.libreoffice.org/8272 Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
* cppuhelper: retrofit std::exception into overriding exception specsStephan Bergmann2014-02-263-38/+38
| | | | Change-Id: I56e32131b7991ee9948ce46765632eb823d463b3
* remove unused code sax_fastparser::FastSerializerHelper::copyTopMarkPop()Noel Grandin2014-02-251-1/+0
| | | | Change-Id: If1dd0e99d83f8211c58fa3b5d1ba56a74bc2157f
* remove unused code sax_fastparser::FastSerializerHelper::copyTopMarkPush()Noel Grandin2014-02-251-1/+0
| | | | Change-Id: I4569e53f0423783bc5c9ee95b07d0de51b75b466
* coverity#1130446 Uncaught exceptionCaolán McNamara2014-02-081-1/+1
| | | | Change-Id: I9225b4ffd507fa3d666862a55dae349c9a76e91d
* sax: various clean up. Move _getFactory next to the implementation.Matúš Kukan2013-12-181-3/+0
| | | | | | | Use more anonymous namespaces, de-duplicate code, bin some comments. Makes the library smaller. Change-Id: Id0cefdcaa72a74741303fc27e36038488ef8b059
* sax, xmloff: fix ODF import/export of text:time/text:time-valueMichael Stahl2013-12-101-0/+10
| | | | | | | | | | | | | | | | | | | | | | | The value written for an Impress time field is something like text:time-value="0000-00-00T23:28:07" (in LO 3.5+) or text:time-value="0-00-00T23:28:07" (in OOo 3.3) which contains an invalid all-zero date. Such values are actually rejected by the ODF import since commit ae3e2f170045a1525f67e9f3e9b7e03d94f2b56b. Actually there was no real support to read the RelaxNG type timeOrDateTime before. So fix that by: - adding convertTimeOrDateTime/parseTimeOrDateTime functions to sax::Converter - recognizing and ignoring the 2 invalid all-zero values written by LO 3.5 and historic OOo respectively - writing a bare "time" in text:time-value if the DateTime struct contains zero Date members (Older OOo versions and AOO cannot actually read that, but everything they _can_ read is invalid ODF...) Change-Id: I754076caee74a5163ed3f972af0f23796aa14f9f
* sax: stop using SAX_DLLIMPLEMENTATION for both sax and fastsaxMichael Stahl2013-12-032-2/+38
| | | | Change-Id: I243ec20015beec6b98ee0af55eb7c387008f32f1
* Hide the implementation.Kohei Yoshida2013-12-021-112/+9
| | | | Change-Id: Ibfd9bf626a40c3ec4eb18d09944e8943163595c8
* Add a means to check if a namespace exists.Kohei Yoshida2013-12-021-0/+3
| | | | | | | Useful when we just need to check if the stream has a certain namespace defined. Calling getNamespaceURL() may throw SAXException in such case. Change-Id: Ib2b7b202492390158270d87bab95d1793c9d8a70
* Move this header out into a public place.Kohei Yoshida2013-12-021-0/+177
| | | | Change-Id: I356b26947d1018276d8a9ff6012fdad3ca2c0fd8
* fastparser: Outline virtual destructor to please MSVC++.Michael Meeks2013-11-271-1/+1
| | | | Change-Id: I8368698e80e5ebe339b822a9e0e767e8ba867e07
* fastparser: Avoid copying all tokens into a sequence.Michael Meeks2013-11-261-1/+13
|