summaryrefslogtreecommitdiff
path: root/i18npool/source/characterclassification
AgeCommit message (Collapse)Author
2017-04-19Convert TOKEN constants to o3tl::typed_flagsNoel Grandin
Change-Id: I0fa6cd6160d6c61a3d6f9eeb6ffadf3db68c6d05 Reviewed-on: https://gerrit.libreoffice.org/36506 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2017-02-21loplugin:subtlezeroinit: i18npoolStephan Bergmann
Change-Id: Ie94d8347a2a64e137221709da3e321f1eabb7d1b
2017-01-31teach lolugin:stringconstant about calling constructorsNoel Grandin
so we can remove unnecessary calls to the OUString(literal) constructor when calling constructors like this: Foo(OUString("xxx"), 1) Change-Id: I1de60ef561437c86b27dc9cb095a5deb2e103b36 Reviewed-on: https://gerrit.libreoffice.org/33698 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2017-01-26Remove dynamic exception specificationsStephan Bergmann
...(for now, from LIBO_INTERNAL_CODE only). See the mail thread starting at <https://lists.freedesktop.org/archives/libreoffice/2017-January/076665.html> "Dynamic Exception Specifications" for details. Most changes have been done automatically by the rewriting loplugin:dynexcspec (after enabling the rewriting mode, to be committed shortly). The way it only removes exception specs from declarations if it also sees a definition, it identified some dead declarations-w/o-definitions (that have been removed manually) and some cases where a definition appeared in multiple include files (which have also been cleaned up manually). There's also been cases of macro paramters (that were used to abstract over exception specs) that have become unused now (and been removed). Furthermore, some code needed to be cleaned up manually (avmedia/source/quicktime/ and connectivity/source/drivers/kab/), as I had no configurations available that would actually build that code. Missing @throws documentation has not been applied in such manual clean-up. Change-Id: I3408691256c9b0c12bc5332de976743626e13960 Reviewed-on: https://gerrit.libreoffice.org/33574 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2016-07-27improve passstuffbyref return analysisNoel Grandin
Change-Id: I4258bcc97273d8bb7a8c4879fac02a427f76e18c Reviewed-on: https://gerrit.libreoffice.org/27317 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-05-31teach refcounting clang plugin about uno::ReferenceNoel Grandin
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>
2016-05-11convert MappingType to scoped enumNoel Grandin
Change-Id: I1f00e1fbdb9213d0c2f30da116684b77842282f5 Reviewed-on: https://gerrit.libreoffice.org/24851 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-05-10Replace fallthrough comments with new SAL_FALLTHROUGH macroStephan Bergmann
...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
2016-04-28clang-tidy modernize-loop-convert in h-l/*Noel Grandin
Change-Id: I843528327b25d18476f8959cabba16371213a48a Reviewed-on: https://gerrit.libreoffice.org/24460 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-04-20loplugin:salbool: Automatic rewrite of sal_False/TrueStephan Bergmann
Change-Id: I3f7f71f05db09219f0de3ec14bdb56bd16fb0c8d
2016-04-12fix infinite loop in text search, tdf#99208Markus Mohrhard
Change-Id: If77055d6b292d5126165252d7198e8abf9d2c44d Reviewed-on: https://gerrit.libreoffice.org/24029 Reviewed-by: Eike Rathke <erack@redhat.com> Tested-by: Eike Rathke <erack@redhat.com>
2016-01-21i18npool: handle non-BMP Unicode in cclass_Unicode::parseText()Michael Stahl
The UTF-16 code unit limitation was mangling starmath import. Change-Id: I087e5c5b7954799fdb73e7ee1a8d3d02669f8831
2015-11-15use initialiser syntax for Sequence<OUString>Noel Grandin
replaced using the script: git grep -lP 'Sequence.*OUString.*\(1\)' | xargs perl -0777 -pi -e "s/Sequence< OUString > (\w+)\(1\); .*\[0\] = (\S+);/Sequence< OUString > \1 { \2 };/g" Change-Id: I23688a91562051a8eed11fc2a85599545c285c34 Reviewed-on: https://gerrit.libreoffice.org/19967 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2015-11-10loplugin:nullptr (automatic rewrite)Stephan Bergmann
Change-Id: I7af8d14bfe7337fea3f18dbbd41eb32e0f7ae331
2015-11-06loplugin:stringconstant: elide explicit ctor usage (automatic rewrite)Stephan Bergmann
Change-Id: I2c65eac8aa0b24be7f0c8a083b21904c842ca1ee
2015-10-20loplugin:defaultparamsStephan Bergmann
Change-Id: I7cb0b7137c71c3112d50e12245dcfbc59c84a47e
2015-10-19com::sun::star->css in i18npoolNoel Grandin
Change-Id: I13a290cba0f38c79eb2d93148b972d3d931c3862 Reviewed-on: https://gerrit.libreoffice.org/19445 Reviewed-by: Eike Rathke <erack@redhat.com> Tested-by: Eike Rathke <erack@redhat.com>
2015-08-03new loplugin: refcountingNoel Grandin
This was a feature requested by mmeeks, as a result of tdf#92611. It validates that things that extend XInterface are not directly heap/stack-allocated, but have their lifecycle managed via css::uno::Reference or rtl::Reference. Change-Id: I28e3b8b236f6a4a56d0a6d6f26ad54e44b36e692 Reviewed-on: https://gerrit.libreoffice.org/16924 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2015-07-08Fix typosAndrea Gelmini
Change-Id: I70b03c152f63e48341dc5629a99b0eeab7b497c0 Reviewed-on: https://gerrit.libreoffice.org/16834 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Julien Nabet <serval2412@yahoo.fr>
2015-05-08Also mention exception typeStephan Bergmann
Change-Id: I3ba60fd0ff8f4ed0a2683986cec7bdb25a837c18
2015-05-07Resolves: tdf#91057 transliterateChar2Char throws MultipleCharsOutputExceptionCaolán McNamara
for ß, but toTitle only allows RuntimeException, which is our usual awesomeness Change-Id: Ib5618a55a369fa5cd1d323f657f0798776828386
2015-04-15remove unnecessary use of void in function declarationsNoel Grandin
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
2014-12-18i18npool: Use appropriate OUString functions on string constantsStephan Bergmann
Change-Id: I6f2fabd9248b1b385439ff5c074342029fb660e0
2014-11-14fdo#86023 - O[U]String needs a 'clear' methodBrij Mohan Lal Srivastava
Added clear() method to OString and OUString class, Updated appropriate call-sites. Change-Id: I0ba97fa6dc7af3e31b605953089a4e8e9c3e61ac Signed-off-by: Stephan Bergmann <sbergman@redhat.com>
2014-11-12Fix common typos. No automatic tools. Handmade…Andrea Gelmini
Change-Id: I1ab4e23b0539f8d39974787f226e57a21f96e959 Reviewed-on: https://gerrit.libreoffice.org/12164 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2014-10-29remove unnecessary 'using namespace rtl' declarationsNoel Grandin
It turns out that almost none of them were necessary. Change-Id: I1311ed28409c682b57ea8d149bcbaf2c49133e83 Reviewed-on: https://gerrit.libreoffice.org/12133 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2014-04-23i18npool: sal_Bool->boolNoel Grandin
Change-Id: Id73ee64dbf8586743f0e436e004e663a65d4548f
2014-02-26cppuhelper: retrofit std::exception into overriding exception specsStephan Bergmann
Change-Id: I56e32131b7991ee9948ce46765632eb823d463b3
2014-02-26Remove visual noise from i18npoolAlexander Wilms
Change-Id: I1cd4254713ce9c1bc7c232414c24e926f1f8a653 Reviewed-on: https://gerrit.libreoffice.org/8269 Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2014-01-27i18npool: Ctor feature for css.i18n.CharacterClassification(_Unicode).Matúš Kukan
Change-Id: I5a0498a3871e2b90629af8ed6b725ff22f242fba
2014-01-27i18npool: Ctor feature for another services.Matúš Kukan
Change-Id: Id2346686ac2f6c537825d70cac1a537d09e351d5
2013-12-18fdo#54938: Convert i18npool to use cppu::supportsServiceMarcos Paulo de Souza
Change-Id: I9acc496cd95c8362972fa0c41d35b77fc0715aba Signed-off-by: Stephan Bergmann <sbergman@redhat.com>
2013-11-11Revert "Guard CharacterClassificationImpl with mutex."Kohei Yoshida
This reverts commit fc04b55f7f96a4f70f31c145dafd44c1d9276a41.
2013-11-11Guard CharacterClassificationImpl with mutex.Kohei Yoshida
They are accessed from multiple threads frequently. Change-Id: I3f9720ede076109efe0b7eaa4a05dd50f2e38102
2013-11-11convert OUString !compareToAscii to equalsAsciiNoel Grandin
Convert code like if( ! aStr.compareToAscii("XXX") ) to if( aStr.equalsAscii("XXX") ) which is both clearer and faster. Change-Id: I267511bccab52f5225b291acbfa4e388b5a5302b
2013-11-04Convert code that calls OUString::getStr()[] to use the [] operatorNoel Grandin
This also means that this code now gets bounds checked in debug builds. Change-Id: Id777f85eaee6a737bbcb84625e6e110abe0e0f27
2013-10-29lets be super sure in the face of falling between surrogatesCaolán McNamara
Change-Id: I6d8259df3d4f2e73f9236b7c0547f87c89801082
2013-10-28Related: fdo#69641 double iterateCodePoints doesn't make sense to meCaolán McNamara
This came in with 04212c3015cd4ab118a0aec2bb04bc153a64af41 but the bug number of #i86439# appears to be the wrong id. It doesn't make sense to me. Rework cclass_Unicode::getStringType to iterate safely over the codepoints of the requested range. Perhaps that was the reason for the original commit. Change-Id: Ice4287eb6f9fc6a9705845c0cf995263815de2e7
2013-10-22Bin comments that claim to say why some header is includedTor Lillqvist
They are practically always useless, often misleading or obsolete. Change-Id: I2d32182a31349c9fb3b982498fd22d93e84c0c0c
2013-09-12Resolves: fdo#61754 allow midletters in field name tokensCaolán McNamara
Change-Id: I77cea3c1d2210435109ca0b6f762b6aea478e91b
2013-09-05use sensible indentationEike Rathke
that always annoyed me Change-Id: I0c439ddb00d78590b465cd47db6e60d1939bd9f7
2013-09-05renamed LocaleData to LocaleDataImpl to prevent confusionEike Rathke
... between com::sun::star::i18n::LocaleData of com/sun/star/i18n/LocaleData.hpp and i18npool/inc/localedata.hxx Change-Id: Id23f67c1e3a654fd4f2886029ee45fcc1f466879
2013-09-05use get*LocaleServiceNameEike Rathke
Change-Id: I4cd6a9a502b4bad11265ac5e42dc338b56b90598
2013-04-07mass removal of rtl:: prefixes for O(U)String*Luboš Luňák
Modules sal, salhelper, cppu, cppuhelper, codemaker (selectively) and odk have kept them, in order not to break external API (the automatic using declaration is LO-internal). Change-Id: I588fc9e0c45b914f824f91c0376980621d730f09
2013-03-04fdo#46808, clean up some old queryInterface() callsNoel Grandin
clean up stuff like this xI->queryInterface( getCppuType((const Reference< XExtendedInputSequenceChecker>*)0) ) >>= xISC; to xISC.set(xI, UNO_QUERY); Change-Id: I3b16a38c2a363440f6079cfe6ae47f009d3940db
2013-01-15fdo#46808, Convert services in i18npool to XComponentContextNoel Grandin
Change-Id: I3aeaca995bc9a3d865b6955aa47cee4d9eb07750
2012-12-06mvoe rtl_(u)string_alloc to salLuboš Luňák
No point in hidding something useful like this in some helper lib. Change-Id: I7332d7f6bd428378cd19e7e95ad130771a541140
2012-11-27fdo#46808, convert i18n::BreakIterator impl to XComponentContextNoel Grandin
Change-Id: I2e6800d0f5a6346fc743d5a57bab34d1e36b4ee4
2012-11-27fdo#46808, use service constructor for i18n::NativeNumberSupplierNoel Grandin
Change-Id: I092ca8f912e26f0743909920c6e740d648b8677e
2012-09-19ComponentContext::getUnoContext -> getComponentContext simplificationStephan Bergmann
...and some further clean-up. Change-Id: If5dce53e382b56390c502d0d0d93fc06cbfe33ea