summaryrefslogtreecommitdiff
path: root/sax
AgeCommit message (Collapse)AuthorFilesLines
2014-07-01Use a new fast parser instance for each XML fragment.Kohei Yoshida1-0/+3
Otherwise a crash ensues when the threaded XML parsing kicks in. Change-Id: Ic41e5a29bbb860d7b63b70f2f0d8896264d9d53e (cherry picked from commit dc93074f71f91efd8a615ad8f1a5289deb210b75) Reviewed-on: https://gerrit.libreoffice.org/10002 Tested-by: Michael Stahl <mstahl@redhat.com> Reviewed-by: Michael Stahl <mstahl@redhat.com>
2014-07-01Don't go further and pop the stack if it's empty.Kohei Yoshida1-0/+6
Change-Id: I27bd30ca65cf0066cd022b4b060757913ea01fed (cherry picked from commit 3ede68e1cc879e6adbf6336eccaf88ced974b8b6) Reviewed-on: https://gerrit.libreoffice.org/9988 Tested-by: Michael Stahl <mstahl@redhat.com> Reviewed-by: Michael Stahl <mstahl@redhat.com>
2014-07-01Check for empty() before calling top().Kohei Yoshida1-0/+3
Else it might crash sometimes. Change-Id: I6a24fff83c3d36346debae5c0f2b8c0646a15c01 (cherry picked from commit 3f177756dbdb67d901453000c3f11694770d2761) Reviewed-on: https://gerrit.libreoffice.org/9990 Tested-by: Michael Stahl <mstahl@redhat.com> Reviewed-by: Michael Stahl <mstahl@redhat.com>
2014-05-15Resolves fdo#70681: fixincludeguards.pl: all that's leftThomas Arnhold2-4/+4
Change-Id: I3e51a62710bb46c8255fd228d41d9300c90a1fb5 Reviewed-on: https://gerrit.libreoffice.org/9360 Reviewed-by: Thomas Arnhold <thomas@arnhold.org> Tested-by: Thomas Arnhold <thomas@arnhold.org>
2014-05-14various loplugin:passsequencebyrefNoel Grandin1-2/+2
Change-Id: Id1045a7f66b4fa10b6491587ba07246a31ceba72
2014-05-14Find places where uno::Sequence is passed by value.Noel Grandin3-4/+4
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>
2014-05-10coverity#707979 Uninitialized pointer fieldCaolán McNamara1-0/+3
Change-Id: Id41ba6ab2c6f0a43f439c2b45d20d7d76a3cbe82
2014-04-24Avoid possible memory leaks in case of exceptionsTakeshi Abe2-20/+13
Change-Id: I047fd88a89900153089a55b6af123f11fb8bde55
2014-04-14typo: attribut -> attributeThomas Arnhold1-2/+2
2014-04-14typo: choosen -> chosenThomas Arnhold1-1/+1
2014-04-09Remove unused functionsStephan Bergmann3-7/+0
Change-Id: Iddf4e29005aaa510af00c7345487996b75f41c41
2014-04-07sax: sal_Bool->boolNoel Grandin5-113/+113
Change-Id: I8d9dd585c509fd4c1f09ee817d828f3e600e31d5
2014-04-01Explicitly mark overriding destructors as "virtual"Stephan Bergmann2-2/+2
It appears that the C++ standard allows overriding destructors to be marked "override," but at least some MSVC versions complain about it, so at least make sure such destructors are explicitly marked "virtual." Change-Id: I0e1cafa7584fd16ebdce61f569eae2373a71b0a1
2014-03-26First batch of adding SAL_OVERRRIDE to overriding function declarationsStephan Bergmann7-56/+56
...mostly done with a rewriting Clang plugin, with just some manual tweaking necessary to fix poor macro usage. Change-Id: I71fa20213e86be10de332ece0aa273239df7b61a
2014-03-18Find places where OUString and OString are passed by value.Noel Grandin2-3/+3
It's not very efficient, because we generally end up copying it twice - once into the parameter and again into the destination OUString. So I create a clang plugin that finds such places and generates a warning so that we can convert them to pass-by-reference. Change-Id: I5341a6ea9e3190f4b4c05c42c85595e3dcd83361
2014-03-05sax: FastAttributeList - add attribute in a specific namespaceTomaž Vajngerl1-0/+6
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
2014-03-02remove whitespacesMarkus Mohrhard1-31/+0
Change-Id: I624ca41c3e51ba785e359649c429feb61cc9a647
2014-02-26cppuhelper: retrofit std::exception into overriding exception specsStephan Bergmann6-100/+100
Change-Id: I56e32131b7991ee9948ce46765632eb823d463b3
2014-02-26Remove visual noise from saxAlexander Wilms6-14/+14
Change-Id: Ica31580f72b43456c33b2f6abbf2140a79efae1a Reviewed-on: https://gerrit.libreoffice.org/8299 Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2014-02-25remove unused code sax_fastparser::FastSerializerHelper::copyTopMarkPop()Noel Grandin3-14/+0
Change-Id: If1dd0e99d83f8211c58fa3b5d1ba56a74bc2157f
2014-02-25remove unused code sax_fastparser::FastSerializerHelper::copyTopMarkPush()Noel Grandin3-12/+0
Change-Id: I4569e53f0423783bc5c9ee95b07d0de51b75b466
2014-02-23Remove unneccessary commentsAlexander Wilms5-39/+39
Change-Id: I939160ae72fecbe3d4a60ce755730bd4c38497fb Reviewed-on: https://gerrit.libreoffice.org/8182 Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2014-02-21[API CHANGE] fix endCDATA to allow throwing css::xml::sax::SAXExceptionCaolán McNamara2-3/+3
coverity#1158397 Uncaught exception Change-Id: Iabf0b1f0500cb4218d8cb3aff372777471e8f287
2014-02-12Improve output in case of test failureStephan Bergmann1-2/+2
Change-Id: If094349998d41972f50e12363a5a223ee87fdcec
2014-02-11Related: #i97873# (coverity#705407) Improper use of negative valueCaolán McNamara1-1/+1
a pity the other one wasn't fixed when the first one was fixed back in 2002 with 0645e6a1a1917fef833358f328b5d56396380698 Change-Id: Ia7ac0f57a6805dc8ff09668198d0530b1cfc31db
2014-02-11sax: my suggestion was not as good as i thought...Michael Stahl1-0/+4
... as indicated by the failing unit test. Change-Id: I1a8a8b933b10673e71261726dc8887dfe1d3e2ee
2014-02-11fdo#72468 avoid out of bounds array accessArnaud Versini1-2/+2
Change-Id: I2bccdf1aaeb290a3cb8f76cd831a419e1776e92f Reviewed-on: https://gerrit.libreoffice.org/7775 Reviewed-by: Michael Stahl <mstahl@redhat.com> Tested-by: Michael Stahl <mstahl@redhat.com>
2014-02-08coverity#1130446 Uncaught exceptionCaolán McNamara1-1/+2
Change-Id: I9225b4ffd507fa3d666862a55dae349c9a76e91d
2014-02-07Use rtl::ReferenceStephan Bergmann1-10/+7
Change-Id: Ia14ad80f13bffe4c1d0078da0198a3ae6eae581e
2014-02-07Avoid css::uno using directives (Reference ambiguities)Stephan Bergmann1-50/+49
Change-Id: Ia702572bef47738531b526addeecd86e7b39aad2
2014-01-251158273 Uncaught exceptionCaolán McNamara1-2/+3
Change-Id: I3983da2d585762d81446902a710e9a97c6354669
2014-01-23coverity#1158443 Uninitialized pointer fieldCaolán McNamara1-3/+6
Change-Id: I1602678c24c1aeec60acee05801e7decc914560f
2014-01-23coverity#1158444 Uninitialized scalar fieldCaolán McNamara1-12/+13
Change-Id: I15f6c6288e2a951543702a15e777167f2240899e
2014-01-23coverity#1158445 Uninitialized pointer fieldCaolán McNamara1-0/+3
Change-Id: I14dc2197543bf98763190b5aea3a366e823312b4
2014-01-23coverity#1158493 Uninitialized scalar fieldCaolán McNamara1-39/+43
Change-Id: I26de8ab1fa4a20bc08e800f04716507469430612
2014-01-22Introduce static inline cppu::acquire(), and make use of that.Jan Holesovsky3-6/+6
This is much better approach compared to the callback function, as it allows passing arguments to the c++ constructor directly, while still allowing some additional initialization after having acquired the instance. Change-Id: I5a0f981915dd58f1522ee6054e53a3550b29d624
2014-01-21Change _get_implementation()'s not to do initialization directly.Jan Holesovsky3-3/+3
Many of the initalizations (in eg. framework) have to be done on an acquire()'d object, so instead of doing the initialization directly, return the initialization member function back to the createInstance() / createInstanceWithContext() / ... and perform the initialization there. As a sideeffect, I belive the calling initialize() from servicemanager is not that much a hack any more - whoever converts the implementation to be constructor-base has the choice to provide the callback, or still initialize through XInitialization, where the callback is preferred by servicemanager when it exists. Change-Id: I8a87b75c54c1441ca0f184967d31ff4902fc4081
2014-01-20Minimize the constructor functions to a bare minimum.Jan Holesovsky3-9/+3
Most of the constructors are supposed to be only a call of new TheInstance(arguments) or an equivalent; so let's just change the constructor caller accordingly, to accept unacquired new instance. If there are exceptions that need to do more heavy lifting, they do not have to use the constructor feature, or there can be a wrapper for the real implementation, doing the additional work in their (C++) constructor. Change-Id: I035c378778aeda60d15af4e56ca3761c586d5ded
2014-01-18Unify ctor functions for component implementations.Matúš Kukan3-15/+11
There is no need to use different styles for writing the same thing. It also makes it easier in future to use search & replace. But of course, there are also some more complicated functions. Change-Id: I773da20378af0e0d5a27689d3903df7063fb8ac0
2014-01-15Unify ctor functions to have _get_implementation suffix.Matúš Kukan4-6/+6
Change-Id: I07fe0671d0633ef9480a4f3431df6a64c7902db8
2014-01-15Use const& arguments parameter for ctor functions.Matúš Kukan3-6/+3
Change-Id: I19ce8bd1a23123ac9a62a7fc95cd54fea5315221
2014-01-14fastparser: avoid boost::optional where it is un-necessary.Michael Meeks1-5/+5
boost::optional appears to show up rather heavily on many profiles. We already use mnElementToken == DONTKNOW to flag / use these guys. Change-Id: Ibf2b0167f259cc601da2fb9703e880b78e60886e
2014-01-10Use boolStephan Bergmann1-2/+2
Change-Id: Iae455f53f8317eecb8edc38d111c9ef7398a36fb
2014-01-07remove unnecessary sal_Unicode casts in OUStringBuffer::append callsNoel Grandin1-7/+7
Convert code like: buf.append( static_cast<sal_Unicode>('!') ); to: buf.append( '!' ); Change-Id: Iacb03a61de65a895540940953b49620677b3d051
2014-01-03WaE: C4101: 'e' : unreferenced local variableTor Lillqvist1-1/+1
Change-Id: I06c0f9dc742736f632ed9210ae954e0dc19ca19d
2013-12-31don't call top on an empty stackCaolán McNamara1-1/+1
Change-Id: Ibe4b4d3785535816b40d46fd0baa60f01e1f9d33
2013-12-31fastparser: avoid std::stack::top() - cache it's results.Michael Meeks1-13/+9
amazingly std::stack::top() takes 146 pseudo-cycles to do not much, so instead cache the result in a single pointer in lieu of burning that code. Change-Id: Ie326be47da6cbad0850e5f1026a1632bb840b6b8
2013-12-20fastparser: move lclGetErrorMessage into the anonymous namespace.Michael Meeks1-41/+41
Change-Id: I70e1597f917c2a8dedb5b38807dfde7ec05a1a39
2013-12-20fastparser:: move Entity:: code into the anonymous namespace.Michael Meeks1-46/+46
Change-Id: I564e35aa63e4c01cc1a0fb45f674dc1a2a0e89ec
2013-12-20fastparser: fix load regressionMichael Meeks1-68/+77
Remove erroneous assert: maSavedException is indeed empty for XML parser reported exceptions. Clean cut/paste code, and comment. Change-Id: Ia538bcc87a7efcd079d3021e00ac4d2eb62f3e8d
generated by cgit v1.2.3 (git 2.39.1) at 2024-05-24 17:58:51 +0000