summaryrefslogtreecommitdiff
path: root/basegfx
AgeCommit message (Collapse)Author
2015-10-30basegfx tree cleanupMario J. Rugiero
- Eliminated an unnecessary boost/bind.hxx include. - Replaced simple old-style for loops that iterated over a container by ranged based for loops. - Replaced for_each by ranged based for loops wherever valid. Change-Id: Ib5c8291cf6d417047b350560f0428723efeccd1c Reviewed-on: https://gerrit.libreoffice.org/19679 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2015-10-27loplugin:unusedmethodsNoel Grandin
Change-Id: I73180266c0af98dbd8d29bd3b11850996b94def9 Reviewed-on: https://gerrit.libreoffice.org/19195 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Thorsten Behrens <Thorsten.Behrens@CIB.de>
2015-10-26WaE: loplugin:defaultparamsTor Lillqvist
Change-Id: Ibb93059d4adab6b672cb9122b2dc8fe8af5bbbd6
2015-10-26Replace boost::bind by lambdas in basegfx tree.Mario J. Rugiero
Change-Id: I8f72bec11b5dfd71cc60a18b980629c176d43f49 Reviewed-on: https://gerrit.libreoffice.org/19595 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2015-10-22coverity#1326120 try and silence Constant expression resultCaolán McNamara
Change-Id: Iafd034dd51135ad99f56279ceee1c770f91436b0
2015-10-20loplugin:defaultparamsStephan Bergmann
Change-Id: I44f3c4c2b6adf766f569f90367dcc294ec734616
2015-10-12Replace "SAL_OVERRIDE" with "override" in LIBO_INTERNAL_ONLY codeStephan Bergmann
Change-Id: I2ea407acd763ef2d7dae2d3b8f32525523ac8274
2015-10-12Replace "SAL_DELETED_FUNCTION" with "= delete" in LIBO_INTERNAL_ONLY codeStephan Bergmann
Change-Id: I328ac7a95ccc87732efae48b567a0556865928f3
2015-10-05clang-analyzer-deadcode.DeadStoresStephan Bergmann
Change-Id: Id2d7c2aa0ba1783710a141bd910af785ecf81f8a
2015-09-30Fix typosAndrea Gelmini
Change-Id: Ie0e941d48457d758525353d6c5cf858652f86fd3 Reviewed-on: https://gerrit.libreoffice.org/18938 Reviewed-by: Samuel Mehrbrodt <Samuel.Mehrbrodt@cib.de> Tested-by: Samuel Mehrbrodt <Samuel.Mehrbrodt@cib.de>
2015-09-29loplugin:removeunusedmethods, remove unused stuffNoel Grandin
Change-Id: I35456b2a3ad2a84a1d045f09cdfb29e4c19b8350
2015-09-18boost->stdCaolán McNamara
Change-Id: Ie490bf2c6921f393bdeed96b1a8815996b701bf0 Reviewed-on: https://gerrit.libreoffice.org/18670 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2015-08-11loplugin: defaultparamsNoel Grandin
Change-Id: I29ef505ee77965df33677f8051ec39398ef0c74e
2015-07-20com::sun::star->css in basegfxNoel Grandin
Change-Id: I27f28e6a09e8bc9e90f0b60a7ac8519b2ab41860 Reviewed-on: https://gerrit.libreoffice.org/17166 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2015-07-14Revert "Avoid loplugin:staticmethods in non-debug build"Stephan Bergmann
This reverts commit d0ba487bae3a0f6d1fcb9af13397c85e70ad8e1b, obsoleted by cbce40e965acef51822b31d73da5fbc271fbcad0 "Make content of OSL_ASSERT, DBG_ASSERT, etc. visiblie in non-debug builds."
2015-07-10do not include boost/utility.hppMichael Stahl
It just includes a bunch of other boost headers; mostly we need boost/noncopyable.hpp so include that directly. This eliminates 831 MB(!) of boost/preprocessor/seq/fold_left.hpp completely, which is the 2nd biggest header after ustring.hxx. Change-Id: I3df55770adcb46e56f389af828e8ba80da2dc1f2
2015-07-08Avoid loplugin:staticmethods in non-debug buildStephan Bergmann
Change-Id: Ie943f6a998f142014c0405892264c8257dccaf6f
2015-07-08s/Coordiante/CoordinateTor Lillqvist
Change-Id: Iac584b09844b5877586061e9a44b1f7473bffed4
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-07-07Fix some misspellingsTor Lillqvist
Change-Id: Iff2303650e1dcfc56064907a28df1c4c452eac56
2015-07-04Fix typosAndrea Gelmini
Change-Id: I75b4ad61785bf0ba1cb07735d938c0977356b8cc Reviewed-on: https://gerrit.libreoffice.org/16705 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Samuel Mehrbrodt <s.mehrbrodt@gmail.com>
2015-06-30Fix typosAndrea Gelmini
Change-Id: I1c5e4970d4e90bdea68a901294594104de5e189e Reviewed-on: https://gerrit.libreoffice.org/16582 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Andras Timar <andras.timar@collabora.com>
2015-06-28Fix typosAndrea Gelmini
Change-Id: Ifd126a535f18db79a18aaec61b76cdf26ab33dff Reviewed-on: https://gerrit.libreoffice.org/16522 Reviewed-by: Julien Nabet <serval2412@yahoo.fr> Tested-by: Julien Nabet <serval2412@yahoo.fr>
2015-06-22Fix typosAndrea Gelmini
Change-Id: I60ed5eb658d50cbc7dc572facb5463b7527b4d9b Reviewed-on: https://gerrit.libreoffice.org/16408 Reviewed-by: Michael Stahl <mstahl@redhat.com> Tested-by: Michael Stahl <mstahl@redhat.com>
2015-06-21Fix typosAndrea Gelmini
Change-Id: I2297bf392a405f02be2450f4624f1c9fc0b512ce Reviewed-on: https://gerrit.libreoffice.org/16373 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2015-06-16Fix typosAndrea Gelmini
Change-Id: I28cfc629dc3d6ef54128615452667ccce86c1072 Reviewed-on: https://gerrit.libreoffice.org/16297 Reviewed-by: Michael Stahl <mstahl@redhat.com> Tested-by: Michael Stahl <mstahl@redhat.com>
2015-06-14cppcheck: redundantCondition [part1]Julien Nabet
Change-Id: I21c5340e7b5ec09248b08aa76f43acf883c56cd8
2015-06-11Typo: vertival -> verticalAndras Timar
Change-Id: I3dce3d131ce78713333d6e4bc2f3e839af5e04ec
2015-06-04Removed getNormal() from polygontools, it just calls B3DPolygon::getNormal()Zsolt Bölöny
Change-Id: I5a59ff8e0db1aee52d0b683c9c64e4b18b64e66b Reviewed-on: https://gerrit.libreoffice.org/16077 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2015-06-04uno::Sequence provides now begin and endNoel Grandin
use begin() and end() when calling std::copy on an uno::Sequence Inspired by commit b34b648fc3262c5d9aa295f621e8fe9c97d4c6b2 "uno::Sequence provides now begin and end" Change-Id: I08e8c3fd6144e77b95a26f85bc0daf6a9edeeb0b Reviewed-on: https://gerrit.libreoffice.org/16057 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2015-06-04convert B2DLineJoin to scoped enumNoel Grandin
Change-Id: I4fb8f2c9e2080e3aa542d2ccf838e6784baa8429
2015-06-04convert B2VectorContinuity to scoped enumNoel Grandin
Change-Id: Ifa401dd0d2d1bef8c0ae2987d2aee86100b59816
2015-06-04convert B2VectorOrientation to scoped enumNoel Grandin
Change-Id: Id7e444bc896ae5d52e5a0bd1b471679c4945afa4
2015-06-02Replaced boost::tuple with std::tupleZsolt Bölöny
Change-Id: Ia001c8e1e00df85d2be93e330811a1081179785d Reviewed-on: https://gerrit.libreoffice.org/15816 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2015-06-01Remove B1IRange, it's not used anywhereZsolt Bölöny
Change-Id: I2eeded849b9b36611407e8ecc545d7801939e4c9 Reviewed-on: https://gerrit.libreoffice.org/15815 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2015-06-01Remove B1IBox, it's not used anywhereZsolt Bölöny
Change-Id: Ifff5fc008552f80fcbb705013c1f578f5f825b24 Reviewed-on: https://gerrit.libreoffice.org/15814 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2015-05-28new clang plugin: loopvartoosmallNoel Grandin
Idea from bubli - look for loops where the index variable is of such size that it cannot cover the range revealed by examining the length part of the condition. So far, I have only run the plugin up till the VCL module. Also the plugin deliberately excludes anything more complicated than a straightforward incrementing for loop. Change-Id: Ifced18b01c03ea537c64168465ce0b8287a42015
2015-05-27Remove B3IPoint, it's not used anywhereZsolt Bölöny
Change-Id: I7031c126179bbceebc40dcbdab5a0570db5bc29f Reviewed-on: https://gerrit.libreoffice.org/15813 Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2015-05-27B3IVector is not used anywhereZsolt Bölöny
Change-Id: Ic439b33c974500f1b351b41a0e391ffec71007b3 Reviewed-on: https://gerrit.libreoffice.org/15808 Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2015-05-27Remove B3IRange, it's not used anywhereZsolt Bölöny
Change-Id: I19a6835eaab2efba530d30f871f1b4509b683801 Reviewed-on: https://gerrit.libreoffice.org/15812 Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2015-05-26Remove B3I64Tuple, it's not used by anythingZsolt Bölöny
Change-Id: Ie1b05375a8e8d634e6ef6b2bf6edde65b32574bd Reviewed-on: https://gerrit.libreoffice.org/15811 Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2015-05-24comments: fix mis-use of eventual in EnglishThorsten Behrens
The adjective was used as the German meaning, which translates to maybe or possibly. Change-Id: Iaf5b38f628b23796829a1c4bd3919d4e13b874f0
2015-05-23Typo: Trapeziod->TrapezoidJulien Nabet
Change-Id: Iab575622819e245b1459cba72dc34a9c387d00b6
2015-05-21Use the better precision equality check in b2trapezoid.cxxZsolt Bölöny
Change-Id: Ie048543ad19c12dc47f6638af7dc029a38da964e Reviewed-on: https://gerrit.libreoffice.org/15805 Reviewed-by: Adolfo Jayme Barrientos <fitojb@ubuntu.com> Tested-by: Adolfo Jayme Barrientos <fitojb@ubuntu.com>
2015-05-19That was totally pointlessZsolt Bölöny
Change-Id: I6046461b2e26a8506249c4a2fdd783bbd2ec223f Reviewed-on: https://gerrit.libreoffice.org/15810 Reviewed-by: Adolfo Jayme Barrientos <fitojb@ubuntu.com> Tested-by: Adolfo Jayme Barrientos <fitojb@ubuntu.com>
2015-05-18cppunit: noExplicitConstructorCaolán McNamara
Change-Id: Iaf1387ea236bf3a5db9b2165d5b827dbdcf95207
2015-05-07simplify code by replacing std::find_if with std::any_of/std::none_ofTakeshi Abe
Change-Id: Ife6ab3544f73b1de60f5241c68466045161a4d88 Reviewed-on: https://gerrit.libreoffice.org/15615 Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2015-05-05Drop misused lcl_ prefixTor Lillqvist
If used at all, lcl_ is supposed to be used for static functions truly local to one source files. The functions here occur in several source files (well, two, the one where they are defined and one other) so they are not "local" in that sense. (But they could be local in the one file that uses them...) Also, they are already in a basegfx::internal namespace. While at it, drop the :: prefix eyesore from basegfx::internal, and align parameter lists consistently. Change-Id: I68b91075e0b1779ac0fa884d8f9e956f1ab7b308
2015-04-27More loplugin:simplifyboolStephan Bergmann
Change-Id: Ic0e8d409e33dcc494a6111eba9ba405424ecd53f
2015-04-21fix crash on export of ooo30155-1.sxw to odtCaolán McNamara
Change-Id: I1dd087ea6803f29756e1b78385a4a98f7378e616