summaryrefslogtreecommitdiffstats
path: root/include/cppcanvas
Commit message (Collapse)AuthorAgeFilesLines
* tdf#42949 Fix IWYU warnings in include/cppcanvas/*Gabor Kelemen2019-03-0511-44/+3
| | | | | | | | | | Found with bin/find-unneeded-includes Only removal proposals are dealt with here. Change-Id: I8174b6e684b5b46a8abd3ca825b7de75f9255df3 Reviewed-on: https://gerrit.libreoffice.org/68703 Tested-by: Jenkins Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
* remove some unused typedefsNoel Grandin2019-02-282-4/+0
| | | | | | | Change-Id: I7775a7d86e9641cd514e58f03d3727d2ad846120 Reviewed-on: https://gerrit.libreoffice.org/68485 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* Silence bogus GCC 8 -Werror=virtual-move-assignStephan Bergmann2018-07-281-0/+2
| | | | | | | | | | | | | | | | | | | | ...seen with at least gcc-c++-8.1.1-1.fc28.x86_64 and gcc-c++-8.1.1-5.fc28.x86_64 after 5065bcf513147c1b92edc9d393768b01bc693c3a "cppcanvas: avoid -Werror=deprecated-copy (GCC trunk towards GCC 9)" and 6844ef8a7ed9b253b6cca4c8d8aec54ef05d013e "cppcanvas: avoid -Werror=deprecated- copy (GCC trunk towards GCC 9)": > [CXX] cppcanvas/source/wrapper/implbitmapcanvas.cxx > In file included from cppcanvas/source/wrapper/implbitmapcanvas.hxx:26, > from cppcanvas/source/wrapper/implbitmapcanvas.cxx:28: > include/cppcanvas/bitmapcanvas.hxx:41:11: error: defaulted move assignment for ‘cppcanvas::BitmapCanvas’ calls a non-trivial move assignment operator for virtual base ‘cppcanvas::Canvas’ [-Werror=virtual-move-assign] > class BitmapCanvas : public virtual Canvas > ^~~~~~~~~~~~ Change-Id: I5348eb110c147957347554504a7e7c6d7fb32b70 Reviewed-on: https://gerrit.libreoffice.org/58185 Tested-by: Jenkins Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
* cppcanvas: avoid -Werror=deprecated-copy (GCC trunk towards GCC 9)Stephan Bergmann2018-07-261-0/+6
| | | | | | | | | | | | | ...by explicitly defaulting the copy/move functions (and, where needed in turn, also a default ctor) for classes that have a user-declared dtor that does nothing other than an implicitly-defined one would do, but needs to be user- declared because it is virtual and potentially serves as a key function to emit the vtable, or is non-public, etc. Change-Id: Ifa6e6f71f73b9d0a577de1a90cbf3a8c969ac062 Reviewed-on: https://gerrit.libreoffice.org/58073 Tested-by: Jenkins Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
* loplugin:unused-returns in basegfx..cppcanvasNoel Grandin2018-01-251-3/+1
| | | | | | | Change-Id: I32dc8c92871c8349651d2f4204a332d387e6e1b2 Reviewed-on: https://gerrit.libreoffice.org/48428 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* remove cppcanvas Color classNoel Grandin2017-10-094-29/+17
| | | | | | | | | | | | | which actually does nothing useful. Looks like it was originally intended to be used to implement color profiles, but since nothing has happened on that front since it was created, safe to say it never will. Probably not the right place in the graphics stack to do it anyhow. Change-Id: I36990db4036e3b4b2b75261fc430028562a6dbd9 Reviewed-on: https://gerrit.libreoffice.org/43240 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* loplugin:unusedfields in cppcanvas..cuiNoel Grandin2017-07-221-3/+0
| | | | | | | Change-Id: I1cf1f7919e03d863c6f7756e1caf3b9313777479 Reviewed-on: https://gerrit.libreoffice.org/40294 Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk> Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
* teach unnecessaryparen loplugin about identifiersNoel Grandin2017-07-101-2/+2
| | | | | | | Change-Id: I5710b51e53779c222cec0bf08cd34bda330fec4b Reviewed-on: https://gerrit.libreoffice.org/39737 Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk> Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
* new loplugin:unusedenumconstantsNoel Grandin2017-02-031-63/+2
| | | | | | | | | | These are the simple removals, where it is obviously safe, the more complex ones will come in separate commits Change-Id: I7211945a6a5576354b60d9c709940ce9b674f308 Reviewed-on: https://gerrit.libreoffice.org/33828 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* loplugin:unusedmethods unused return typesNoel Grandin2016-11-031-3/+0
| | | | | | | Change-Id: I88204bca60dd1e299b040c52bc87e500cbfaa930 Reviewed-on: https://gerrit.libreoffice.org/30519 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* cleanup: remove unused Reference.h(xx) includesJochen Nitschke2016-04-021-1/+0
| | | | | | | | | and unused using-declarations from Reference.h Change-Id: I297a7ae6044fa329d245ecf08fd5c4cb930f5b19 Reviewed-on: https://gerrit.libreoffice.org/23735 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
* remove boost::noncopyable from /includeNoel Grandin2015-10-291-2/+4
| | | | Change-Id: I9fa22b06fabf79043ebc68be7afebc6e810f4db1
* loplugin:unusedmethodsNoel Grandin2015-10-271-5/+0
| | | | | | | Change-Id: I161cd52606c11b6008f5d8b1d8ee391692f91861 Reviewed-on: https://gerrit.libreoffice.org/19231 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* com::sun::star->css in include/cppcanvasNoel Grandin2015-10-217-18/+14
| | | | | | | Change-Id: Iffaf661fe915c9b0e3500ae3923fd8300583c512 Reviewed-on: https://gerrit.libreoffice.org/19484 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
* Replace "SAL_DELETED_FUNCTION" with "= delete" in LIBO_INTERNAL_ONLY codeStephan Bergmann2015-10-122-6/+6
| | | | Change-Id: I328ac7a95ccc87732efae48b567a0556865928f3
* boost->stdCaolán McNamara2015-09-1411-42/+25
| | | | | | | | Change-Id: I3fd9e1599c5ad812879a58cf1dabbcd393105e1c Reviewed-on: https://gerrit.libreoffice.org/18564 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
* Put Polygon from tools under tools:: namespaceNorbert Thiebaud2015-08-171-2/+4
| | | | | | | | | | Polygon is one of these names that Clash with some system objects A similar work has been done earlier with PolyPolygon. Change-Id: Icf2217cb2906292b7275760f1a16be0e150312f5 Reviewed-on: https://gerrit.libreoffice.org/17789 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Norbert Thiebaud <nthiebaud@gmail.com>
* loplugin:unusedmethodsNoel Grandin2015-08-057-42/+0
| | | | | | | Change-Id: I6801618efb5a66d24156fa429e026acb6ca03aba Reviewed-on: https://gerrit.libreoffice.org/17506 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* loplugin:staticmethodsNoel Grandin2015-04-132-25/+13
| | | | Change-Id: I33a8ca28b0c3bf1c31758d93238e74927bebde9c
* loplugin:deletedspecialStephan Bergmann2015-02-072-6/+4
| | | | Change-Id: I988d7cdfa048a3b093ebc34f9016ac42440daa4b
* followup code removal after changing virtual methods to non-virtualNoel Grandin2015-01-263-47/+0
| | | | | | | | | | This cleanups up indentation and removes dead classes. This is a followup patch to commit 272b1dd55797aacf511fb4342b0054e3697243f6 "new loplugin: change virtual methods to non-virtual" Change-Id: I1c2139589cf8cb23bb9808defe22c51039d38de1
* new loplugin: change virtual methods to non-virtualNoel Grandin2015-01-261-2/+2
| | | | | | | | | | Where we can prove that the virtual method is never overriden. In the case of pure-virtual methods, we remove the method entirely. Sometimes this leads to entire methods and fields being eliminated. Change-Id: I138ef81c95f115dbd8c023a83cfc7e9d5d6d14ae
* fdo#82577: Handle PolyPolygonNoel Grandin2014-09-304-5/+5
| | | | | | | Put the TOOLS PolyPolygon class in the tools namespace. Avoids clash with the Windows PolyPolygon typedef. Change-Id: I811ecbb3d55ba4ce66e4555a8586f60fcd57fb66
* fdo#82577: Handle WindowNoel Grandin2014-09-231-2/+2
| | | | | | | Put the VCL Window class in the vcl namespace. Avoids clash with the X11 Window typedef. Change-Id: Ib1beb7ab4ad75562a42aeb252732a073d25eff1a
* Further XCanvas != XBitmapCanvas cleanup.Thorsten Behrens2013-11-291-1/+4
| | | | | | | | It seems I missed a few places in 21ec9beae29b19b8ec6f0a16fd0e708e4f210208, getting that straight now hopefully. Change-Id: I34fc8df2d1ee65bef0639b3de9487ff311bed89a
* Revert "Resolves: fdo#71527 make presenter console not crash/useless"Thorsten Behrens2013-11-291-2/+2
| | | | | | This reverts commit 639aa8e72639fd01e9004977f1cfaafc13b1e45f. Change-Id: I0ac1a7ada530791e2d34d2f0802213c869ea48a1
* Resolves: fdo#71527 make presenter console not crash/uselessCaolán McNamara2013-11-281-2/+2
| | | | | | | | | | | Basically reverts the pieces of 21ec9beae29b19b8ec6f0a16fd0e708e4f210208 to make XSpriteCanvas a XBitmapCanvas again Otherwise PresenterHelper::loadBitmap is not an XBitmapCanvas and so VclFactory::createBitmap cannot succeed Change-Id: I197adf98e915102f383ee050a8ea16d1e41cecf2
* fixincludeguards.sh: include/c*Thomas Arnhold2013-10-2314-42/+42
| | | | Change-Id: Icba422d99836518a1b662b15144bc9334bed1473
* fdo#68849 add some header guardsThomas Arnhold2013-10-221-2/+3
| | | | Change-Id: I1cd598fba94e337c2ccad68a4a888743aeb292f0
* XCanvas does no longer imply XBitmapCanvasThorsten Behrens2013-10-071-2/+2
| | | | | | | | | | | Remove the implication that XCanvas IS-A XBitmapCanvas, i.e. one can access underlying pixels. That's a preparation for display-list based canvas implementations, and was a rather silly assumption in the first place. Also fixes up all client sites. Change-Id: I7a3d6f68ef46fe94b23cce7759cc0b8b6ca4dae9
* execute move of global headersBjoern Michaelsen2013-04-2315-0/+1271
see https://gerrit.libreoffice.org/#/c/3367/ and Change-Id: I00c96fa77d04b33a6f8c8cd3490dfcd9bdc9e84a for details Change-Id: I199a75bc4042af20817265d5ef85b1134a96ff5a