summaryrefslogtreecommitdiffstats
path: root/sax
Commit message (Collapse)AuthorAgeFilesLines
* New loplugin:dynexcspec: Add @throws documentation, saxStephan Bergmann2017-01-193-0/+36
| | | | Change-Id: If43818ca79bf134a5025ed184c201f4129630500
* Also consider saved exceptions when mbEnableThreads is trueMike Kaganski2017-01-191-0/+4
| | | | | | | | | | | | Previously, saved exceptions are only checked in FastSaxParserImpl::parse(), which is used in case mbEnableThreads is false (when data in input stream is no more than 10000). This patch also enables the same check for the other case. Change-Id: Ie718556b7c01322e30698515ecd331b7ebad4105 Reviewed-on: https://gerrit.libreoffice.org/33249 Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com> Tested-by: Mike Kaganski <mike.kaganski@collabora.com>
* replace #ifdef SOLARIS with #ifdef __sunMichael Stahl2017-01-172-4/+4
| | | | | | | Check for a macro that is defined by the compiler, we don't really need one defined by the build system. Change-Id: Iccb8e3198396881395c97a6b81690ebe64b7e9d2
* new loplugin: useuniqueptr: jvmfwk..saxNoel Grandin2017-01-172-20/+7
| | | | | | | Change-Id: I732e2e22c6f953f0982fbc8833453e7c23cf9d49 Reviewed-on: https://gerrit.libreoffice.org/33166 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* New loplugin:conststringvar: saxStephan Bergmann2017-01-101-5/+5
| | | | Change-Id: I110325963655391bb6223f132fcaab2b95f3c9e3
* Adapt to no-longer explicit OUStringLiteral ctorStephan Bergmann2016-12-171-3/+3
| | | | | | | | | ...from previous commit Change-Id: I062b7cd212c17e7eb5274476e6859228d0477c7f Reviewed-on: https://gerrit.libreoffice.org/32098 Reviewed-by: Stephan Bergmann <sbergman@redhat.com> Tested-by: Stephan Bergmann <sbergman@redhat.com>
* loplugin:unnecessaryoverride (dtors) in saxStephan Bergmann2016-12-051-1/+0
| | | | Change-Id: I88ef1fc5b154f29f13ddab00626bcfcd67385bd1
* tdf#103544: DOCX exp.: Image loss when have a frame anchored to the same para.Tamás Zolnai2016-10-281-5/+1
| | | | | | | | | | | | | | | | | | Regression from: 83d51e5e52688c4c9bc0ad70a511458bb06a242d Partly revert the commit causes this regression. I checked the related bugs (tdf#78590,tdf#80748) intended to be fixed by this commit and reverting this part does not bring back the corruption. I guess something changed in frames' and text boxes' import in the meantime, because this MergeMarks::IGNORE is useless now. Change-Id: If17776e8628561961c7ce2a2994e3fc609f75639 Reviewed-on: https://gerrit.libreoffice.org/30351 Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com> Tested-by: Tamás Zolnai <tamas.zolnai@collabora.com>
* loplugin:unnecessaryoverrideNoel Grandin2016-10-132-12/+0
| | | | | | | Change-Id: I08c55a3023ec2e8990098eeb60e91cd18556e7ae Reviewed-on: https://gerrit.libreoffice.org/29656 Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk> Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
* tdf#103159 allow compiling on 12.04 systems stillJustin Luth2016-10-121-9/+15
| | | | | | | | | | | | | | | | | "An uncaught exception of type com.sun.star.xml.sax.SAXParseException [internal line 13]: Namespace prefix manifest on file-entry is not defined" regression from 3aa52d36824d11b8774de15708fdfcbb93cd9dc3 which states that "initialization parameter to FastParser will turn off the namespace declaration missing exception." However, although it doesn't RAISE the exception, it doesn't prevent it either. Change-Id: I26ea02444132e5370dcb4821a75181bc8c76e553 Reviewed-on: https://gerrit.libreoffice.org/29731 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Justin Luth <justin_luth@sil.org>
* loplugin:singlevalfields in l10ntools..saxNoel Grandin2016-09-152-22/+6
| | | | | | and ignore URE headers in the plugin Change-Id: I939df744c1173cf0f67d515f6c3fb4ac4cb25588
* loplugin:override: No more need for the "MSVC dtor override" workaroundStephan Bergmann2016-09-134-4/+4
| | | | | | | | | | | | | | | | | 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
* loplugin:stringconstant: adapt to improved OUStringLiteral1 (sax)Stephan Bergmann2016-08-301-1/+1
| | | | Change-Id: Ia39a4f4689ae0d6e2b7303a89251634bfe0eeb2f
* cid#1371329 Misused comma operatorNoel Grandin2016-08-251-2/+2
| | | | Change-Id: Iba29bce22c59b25a452b454e2b7cad3c4e95f3a3
* cid#1371333 Misused comma operatorNoel Grandin2016-08-251-2/+2
| | | | Change-Id: I5288cfb4e86bbcf07df1c74cdacdd5f0e7bf03fb
* loplugin:stringstaticStephan Bergmann2016-08-161-15/+15
| | | | Change-Id: I3812abe3b275bc1b89e340d0e8f52aa1d67905e8
* GSoC - Making legacyfastparser to use tokens:Mohammed Abdul Azeem2016-08-103-54/+206
| | | | | | | | | | | This tokenizes some elements, de-tokenize while consuming and emits elements through legacy interface. DummyTokenHandler is just to test the correctness. Change-Id: I1ea1e4d806ed4d426215f93b3f6b66a9776f6479 Reviewed-on: https://gerrit.libreoffice.org/27849 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* Related: tdf#99312, add specialized interaction handlerGiuseppe Castagno2016-08-081-0/+1
| | | | | | | | | | Needed to manage https certificates and authentication whenever needed. Change-Id: If20b85a9b349b203a8c46d453afa823629d114cb Reviewed-on: https://gerrit.libreoffice.org/27927 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Giuseppe Castagno <giuseppe.castagno@acca-esse.eu>
* loplugin:countusersofdefaultparams in oox..sdNoel Grandin2016-08-051-2/+2
| | | | | | | Change-Id: I4504939a3957606979c6ac36af6611e1fe072d01 Reviewed-on: https://gerrit.libreoffice.org/27902 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* GSoC - moving FastAttributeList::clear to consumer thread:Mohammed Abdul Azeem2016-07-291-8/+42
| | | | | | | | | | this shares the load of clearing the attributes list with the consumer when producer is busy. Change-Id: I6e89858703c7af9c30b2d99fd6825dc81290b488 Reviewed-on: https://gerrit.libreoffice.org/27649 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Michael Meeks <michael.meeks@collabora.com>
* GSOC - Avoid tokenizing strings for Unknown elements:Mohammed Abdul Azeem2016-07-272-58/+67
| | | | | | | | | | | Added a special case when token handler is not set, which avoids tokenizing strings. Change-Id: I749a7af22e45180cc8bfc55843832b7ccb529ac6 Reviewed-on: https://gerrit.libreoffice.org/27593 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Michael Meeks <michael.meeks@collabora.com> Tested-by: Michael Meeks <michael.meeks@collabora.com>
* GSOC - Handling namespace declaration missing case:Mohammed Abdul Azeem2016-07-195-5/+82
| | | | | | | | | | | | 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>
* loplugin:passstuffbyrefStephan Bergmann2016-07-171-1/+1
| | | | Change-Id: I7c8cd99210d1048b73e074831080e9c7121292fc
* GSoC - Sync namespace handler calls with elements:Mohammed Abdul Azeem2016-07-163-5/+31
| | | | | | | | | | | This syncs the namespace declaration calls with element's events. A test case is also added which enables threads of fastparser. Change-Id: Ic40a380c8523688ee6b2ec76bb3d36a0452923bd Reviewed-on: https://gerrit.libreoffice.org/27222 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Michael Meeks <michael.meeks@collabora.com> Tested-by: Michael Meeks <michael.meeks@collabora.com>
* new loplugin unnecessary overrideNoel Grandin2016-07-151-0/+1
| | | | | | | Change-Id: I88d3e33823d68745b98625050a8a274f9ef04bcb Reviewed-on: https://gerrit.libreoffice.org/27135 Reviewed-by: Stephan Bergmann <sbergman@redhat.com> Tested-by: Stephan Bergmann <sbergman@redhat.com>
* loplugin:unreffunStephan Bergmann2016-07-101-1/+0
| | | | Change-Id: I8649e6bbe3b5c1189ef3c07bcbc1c2155d265b59
* GSOC: Adapt XLegacyFastParser to function like XParser.Mohammed Abdul Azeem2016-07-093-175/+121
| | | | | | | | | | | | 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:staticmethodsStephan Bergmann2016-07-081-27/+22
| | | | Change-Id: Ia9a3b93df2c885a56705b35b88fe0a8f50a10b74
* GSOC: Added test case for mishandle of namespaces.Mohammed Abdul Azeem2016-07-081-0/+78
| | | | | | | | | | | Added a test case for misuse of namespace declaration and resolving an element name. Change-Id: Iadda81a82353de45418fff6315c2d9b94626126a Reviewed-on: https://gerrit.libreoffice.org/26661 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Reviewed-by: Michael Meeks <michael.meeks@collabora.com>
* loplugin:redundantcast: redundant static_casts in saxStephan Bergmann2016-07-081-1/+1
| | | | Change-Id: I8f9398a0d6e362c8f786e17e63ec48e4c4f77866
* loplugin:passstuffbyref also for {css::uno,rtl}::ReferenceStephan Bergmann2016-07-072-2/+2
| | | | Change-Id: If18f0a749c1e474c302fd797c2a2abc2b33a0730
* use SAL_DEPRECATED in uno::ReferenceNoel Grandin2016-07-041-1/+1
| | | | | | | | | and fix a couple of usages that have crept in Change-Id: Ia3e7fcc05dac6e0d205e69c0e0372c74653e7c5e Reviewed-on: https://gerrit.libreoffice.org/26851 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* loplugin:refcountingStephan Bergmann2016-06-171-6/+6
| | | | Change-Id: I39fbc89cffe25e85b8cd1f2e36f0a20c07e33497
* loplugin:passstuffbyrefStephan Bergmann2016-06-171-1/+1
| | | | Change-Id: I2fb13854a6da3f114dba386169e5aef7f741b922
* GSoC: Mapping legacy sax parser and XFastParserMohammed Abdul Azeem2016-06-174-8/+325
| | | | | | | | | | This implements legacy parser interface using XFastParser, and unit test is also added. Change-Id: Ia2eb7d517d80a3f7ec0cf26ffa2e5747ad22b186 Reviewed-on: https://gerrit.libreoffice.org/26229 Reviewed-by: Michael Meeks <michael.meeks@collabora.com> Tested-by: Michael Meeks <michael.meeks@collabora.com>
* loplugin:refcountingStephan Bergmann2016-06-161-4/+4
| | | | Change-Id: Iafc3987df4357098aab816fb7fd4619f9a6ee173
* GSoC: sax2/ unit tests:Mohammed Abdul Azeem2016-06-153-41/+190
| | | | | | | | Parsed the same test files using XFastParser and built string, we asssert that both the strings built from XParser and XFastParser are identical. Change-Id: Ie1e0dc6b676ca32dd10d75c3cb272b5c8b795ab7
* tdf#94698 cleanup Makefiles. Get rid of udkapi and offapiGleb Mishchenko2016-06-091-4/+1
| | | | | | | | | | | | | | | | | | | | This patch changes occurences in makefiles from $(eval $(call gb_CppunitTest_use_api,comphelper_test_config, \ udkapi \ offapi \ )) to $(eval $(call gb_CppunitTest_use_sdk_api,comphelper_test_config)) (corrected instead of abandon, jani) Change-Id: Ic96ec65d82d7452e288f05a8b6d576ef543b068e Reviewed-on: https://gerrit.libreoffice.org/23426 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: jan iversen <jani@documentfoundation.org> Tested-by: jan iversen <jani@documentfoundation.org>
* Let loplugin:passstuffbyref also look at fn defn not preceded by any declStephan Bergmann2016-06-082-2/+2
| | | | Change-Id: I752bc96d2d521d790e919283cabb14b6526626f4
* teach refcounting clang plugin about uno::ReferenceNoel Grandin2016-05-313-5/+6
| | | | | | | | | | | | | | uno::Reference is only allowed to used with classes that have a ::static_type member. So convert all those places to rtl::Reference. Maybe we need some LIBO_INTERNAL_ONLY constructors on rtl::Reference and uno::Reference to make this a little smoother? Change-Id: Icdcb35d71ca40a87b1dc474096776412adbfc7e3 Reviewed-on: https://gerrit.libreoffice.org/25516 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* loplugin:unreffunStephan Bergmann2016-05-291-1/+0
| | | | Change-Id: I31e62926ddfddc8bbdab681a55cdc381d3275794
* sax2/ unit tests.Mohammed Abdul Azeem2016-05-2810-33/+197
| | | | | | | | | | Test xml files are added, string is built for each file and namespaces are handled. Change-Id: I0ab799ca5c9de7311ccca2a6033a96e02598064f Reviewed-on: https://gerrit.libreoffice.org/25468 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Michael Meeks <michael.meeks@collabora.com>
* -Werror,-Wunused-private-fieldStephan Bergmann2016-05-221-1/+1
| | | | Change-Id: I286afa5f5f849217c282c8408822522d9ce5a10b
* loplugin:unreffun and loplugin:nullptrNoel Grandin2016-05-191-2/+2
| | | | | | | Change-Id: I0b8e094d5f5db4b9add46894c4a606256c2d6904 Reviewed-on: https://gerrit.libreoffice.org/25159 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
* First cut at adding sax2/ unit tests.Mohammed Abdul Azeem2016-05-195-0/+243
| | | | | | | | | | This is a first-step towards implementing the legacy parser using the XFastParser. Change-Id: I37fc0419dbef1d6f067e45e0b1f2fd6051fb158d Reviewed-on: https://gerrit.libreoffice.org/24883 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* Replace fallthrough comments with new SAL_FALLTHROUGH macroStephan Bergmann2016-05-101-2/+2
| | | | | | | | | | | | | | | | ...which (in LIBO_INTERNAL_ONLY) for Clang expands to [[clang::fallthrough]] in preparation of enabling -Wimplicit-fallthrough. (This is only relevant for C++11, as neither C nor old C++ has a way to annotate intended fallthroughs.) Could use BOOST_FALLTHROUGH instead of introducing our own SAL_FALLTHROUGH, but that would require adding back in dependencies on boost_headers to many libraries where we carefully removed any remaining Boost dependencies only recently. (At least make SAL_FALLTHROUGH strictly LIBO_INTERNAL_ONLY, so its future evolution will not have any impact on the stable URE interface.) C++17 will have a proper [[fallthroug]], eventually removing the need for a macro altogether. Change-Id: I342a7610a107db7d7a344ea9cbddfd9714d7e9ca
* clang-tidy modernize-loop-convert in oox to saxNoel Grandin2016-05-052-5/+5
| | | | | | | Change-Id: If0d87b6679765fc6d1f9300c6972845cf3742b9c Reviewed-on: https://gerrit.libreoffice.org/24674 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
* tdf#94260 Convert pixel to 1/100 mmRosemary Sebastian2016-05-041-9/+15
| | | | | | | | | | The scaling factor 0.28 is chosen as per https://www.w3.org/TR/2001/REC-xsl-20011015/slice5.html#pixels Change-Id: I8dc7846699fbb2aa6e2a181a041b66d3cc33b8e5 Reviewed-on: https://gerrit.libreoffice.org/24547 Reviewed-by: Oliver Specht <oliver.specht@cib.de> Tested-by: Oliver Specht <oliver.specht@cib.de>
* loplugin:salbool: sal_Bool[] -> bool[]Stephan Bergmann2016-04-211-6/+6
| | | | Change-Id: I3c5bf7a53c9ae173f8fce885ecf022f092aa43a9
* loplugin:salbool: Automatic rewrite of sal_False/TrueStephan Bergmann2016-04-202-3/+3
| | | | Change-Id: I923b1b92f07c2db6d88c7682f1733fd8fd765f90