summaryrefslogtreecommitdiffstats
path: root/canvas
Commit message (Collapse)AuthorAgeFilesLines
* tdf#94006 - fix OpenGLContext mis-use in several places.Michael Meeks2015-09-082-17/+18
| | | | | | | gltf rendering, OpenGL canvas, GL transitions & GL capable (charts) Avoid GLX operations on un-initialized contexts. Change-Id: I7f523640f66ab656896181e5c865879234f6640e
* canvas: simplify PageManager::nakedFragment loopsDaniel Robertson2015-09-041-30/+16
| | | | | | | | | | Convert while loops and break statements in PageManager::nakedFragment into for loops Change-Id: I671f4eea140f26c2f451d54911d017325084bd08 Reviewed-on: https://gerrit.libreoffice.org/18138 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Thorsten Behrens <Thorsten.Behrens@CIB.de>
* tdf#93870 - GL accel. via VCL canvas for presentations when GL enabled.Michael Meeks2015-09-034-0/+14
| | | | | | | | | Change-Id: Ia31a88cb3d9e6baa987b22f77f407a3f592031f1 Reviewed-on: https://gerrit.libreoffice.org/18285 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Michael Meeks <michael.meeks@collabora.com> Tested-by: Michael Meeks <michael.meeks@collabora.com> Reviewed-by: Jan Holesovsky <kendy@collabora.com>
* canvas: replace while loops with range-based forDaniel Robertson2015-08-307-104/+54
| | | | | | | Change-Id: Ide16bee666cf4df41646f9336a585e22a7fe53bd Reviewed-on: https://gerrit.libreoffice.org/18131 Reviewed-by: Thorsten Behrens <Thorsten.Behrens@CIB.de> Tested-by: Thorsten Behrens <Thorsten.Behrens@CIB.de>
* o3tl/cow_wrapper: remove boost dependencyDaniel Robertson2015-08-292-0/+2
| | | | | | | | | | | Remove boost dependencies from ::o3tl::cow_wrapper, and add the necessary includes to files including checked_delete and noncopyable that do not already include the necessary files. Change-Id: Iedae4232002792724226829a5d5cf9d16ffd0686 Reviewed-on: https://gerrit.libreoffice.org/18125 Reviewed-by: Thorsten Behrens <Thorsten.Behrens@CIB.de> Tested-by: Thorsten Behrens <Thorsten.Behrens@CIB.de>
* o3tl: rename compat_functional headerThorsten Behrens2015-08-171-1/+1
| | | | | | | | | | After the cleanup, the stuff there really stands on its own (and the remaining usage of select1st/2nd is more descriptive than lambdas). Change-Id: I0aba131d5dc550189f8130d167dc94167e404540 Reviewed-on: https://gerrit.libreoffice.org/17806 Reviewed-by: Thorsten Behrens <Thorsten.Behrens@CIB.de> Tested-by: Thorsten Behrens <Thorsten.Behrens@CIB.de>
* Put Polygon from tools under tools:: namespaceNorbert Thiebaud2015-08-174-10/+8
| | | | | | | | | | 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>
* canvas: replace for_each with range-based for-loopDaniel Robertson2015-08-164-142/+83
| | | | | | | | | | | Replace ::std::for_each for a more readable range-based for loop in cases in which the function object to be applied by for_each is more readable as the body of a for loop. Change-Id: I5ea0f6a464855b8cc8af38f211bb784dd91eca0d Reviewed-on: https://gerrit.libreoffice.org/17775 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Thorsten Behrens <Thorsten.Behrens@CIB.de>
* canvas: whitespace/comments cleanupThorsten Behrens2015-08-158-122/+3
| | | | | | The more redundant cases of whitespace/comments had to go... Change-Id: Id0bfab5f60f123c0a4d3d9f8468791e0d8046225
* canvas: replace for_each with range-based for-loopDaniel Robertson2015-08-155-124/+45
| | | | | | | | | | | Replace ::std::for_each for a more readable range-based for loop in cases in which the function object to be applied by for_each is more readable as the body of a for loop. Change-Id: I6f96833d7f251d44e2308278bb92a7d49fd49bcd Reviewed-on: https://gerrit.libreoffice.org/17763 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Thorsten Behrens <Thorsten.Behrens@CIB.de>
* tdf#92459 Cleanup unclear lambdasDaniel Robertson2015-08-111-10/+10
| | | | | | | | | | | Replace lambdas used to select the first/second member of a pair with the new simplified select1st/2nd from o3tl/compat_functional. There should be no side effects due to this change. Change-Id: I17f37796e0c4defe96a10aa491d192adb9eebb89 Reviewed-on: https://gerrit.libreoffice.org/17656 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Thorsten Behrens <Thorsten.Behrens@CIB.de>
* remove imgdebugNorbert Thiebaud2015-08-115-60/+0
| | | | | | | | | | | | | | imgdbug.h ins included in canvs under some debugging flags but that header comes from http://billbaxter.com/projects/imdebug/ and is nowhere to be found in our source. the original project has been dead for a long while.. the last 'news' was that it was migrating to the now defunc 'berlios' hosting site. Change-Id: Idd030164f4ef0b28973530df69323cb952e99169 Reviewed-on: https://gerrit.libreoffice.org/17655 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Norbert Thiebaud <nthiebaud@gmail.com>
* tdf#92459 remove compat_functional from canvasDaniel Robertson2015-08-111-17/+6
| | | | | | | | | | | | | Replace all uses of deprecated features from the o3tl in compat_functional.hxx with lambda expressions in canvas. There should be no side effects due to this patch. Change-Id: Ia08ff1642a4f64035441dfdbac03c6fb09fa0443 Reviewed-on: https://gerrit.libreoffice.org/17586 Reviewed-by: Thorsten Behrens <Thorsten.Behrens@CIB.de> Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
* stop inclduing canvas/verbosetraces.hxx and canvas/debug.hxxNorbert Thiebaud2015-08-1183-769/+741
| | | | | | | | | also clean-up and 'standardize' the include sections of canvas' source code Change-Id: Ib437a41ea41ae2ea6528f755ae5b2828775aaf73 Reviewed-on: https://gerrit.libreoffice.org/17633 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Norbert Thiebaud <nthiebaud@gmail.com>
* remove VERBOSE use from canvaNorbert Thiebaud2015-08-1033-108/+40
| | | | | | | | | | | | | in preparation to kill --enable-verbose in configure.ac convert modules that use VERBOSE-YES/NO (uppercase) env variable to do things differently at compile time Convert canvas to use SAL_INFO/SAL_WARN mechanism instead of its home-grown 'verbose logging'. Change-Id: I2898d2309d421457839e3cd7078166c6da919882 Reviewed-on: https://gerrit.libreoffice.org/17619 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Norbert Thiebaud <nthiebaud@gmail.com>
* canvas: replace boost::function with std::functionMichael Stahl2015-07-312-6/+7
| | | | Change-Id: I1a4a3007a07a796d80d06ab7554953925bb8be3d
* canvas: replace boost::function with std::functionMichael Stahl2015-07-314-3/+5
| | | | Change-Id: Ia3644bc0d9db2159aa4e9f05ca2f36e60b63e6f4
* tdf#92459 Replace select1st for lambda expressionsDaniel Robertson2015-07-311-49/+35
| | | | | | | | | | Replace all instances of select1st with lambda functions in canvas/source/factory/cf_service.cxx. Change-Id: I935282817fdf6496bd03752b8adb89e827ff28c5 Reviewed-on: https://gerrit.libreoffice.org/17409 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: David Ostrovsky <david@ostrovsky.org>
* com::sun::star->css in canvasNoel Grandin2015-07-2284-1389/+1350
| | | | | | | Change-Id: I23c3f774834bb7e84e2f02fe05fa9d844adadc9d Reviewed-on: https://gerrit.libreoffice.org/17259 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* tdf#91574 - ensure we get paint events to render directx presenter console.Michael Meeks2015-07-211-1/+0
| | | | | | This is an odd-one; lots of details in the bug. Change-Id: I287945eba965cc637642e9226c7fe7f2a0754130
* do not include boost/utility.hppMichael Stahl2015-07-1022-22/+22
| | | | | | | | | | 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
* tdf#88206 replace cppu::WeakImplHelper* etc.Takeshi Abe2015-07-1031-75/+64
| | | | | | | | | with the variadic variants, in canvas. Change-Id: I8604e6b6e0f45539e3411c98a166518b837b6758 Reviewed-on: https://gerrit.libreoffice.org/16798 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* canvas: remove boost::bind from headersMichael Stahl2015-07-095-0/+7
| | | | Change-Id: Ieaa8a74d59ee9c74fafe823e01db4a349c49821e
* tdf#91574: SLIDESHOW: crash when second monitor connectedJulien Nabet2015-07-061-1/+1
| | | | | | | | | | | | | | | | | 0 0x00002aaab1f6a803 in VirtualDevice::InnerImplSetOutputSizePixel(Size const&, bool, boost::shared_array<unsigned char> const&, bool) (this=0x0, rNewSize=Size = {...}, bErase=true, pBuffer=..., bTopDown=false) at /home/julien/compile-libreoffice/libreoffice/vcl/source/gdi/virdev.cxx:293 1 0x00002aaab1f6ae54 in VirtualDevice::ImplSetOutputSizePixel(Size const&, bool, boost::shared_array<unsigned char> const&, bool) (this=0x0, rNewSize=Size = {...}, bErase=true, pBuffer=..., bTopDown=false) at /home/julien/compile-libreoffice/libreoffice/vcl/source/gdi/virdev.cxx:396 2 0x00002aaab1f6b1af in VirtualDevice::SetOutputSizePixel(Size const&, bool) (this=0x0, rNewSize=Size = {...}, bErase=true) at /home/julien/compile-libreoffice/libreoffice/vcl/source/gdi/virdev.cxx:444 3 0x00002aaae650e30f in vclcanvas::SpriteCanvasHelper::updateScreen(bool, bool&) (this=0x69daf50, bUpdateAll=true, io_bSurfaceDirty=@0x69dafe0: true) at /home/julien/compile-libreoffice/libreoffice/canvas/source/vcl/spritecanvashelper.cxx:316 So initialize maVDev Change-Id: I4a79ba6e8f13c4a2bed3127742a9a9c928e133cd Reviewed-on: https://gerrit.libreoffice.org/16770 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Michael Meeks <michael.meeks@collabora.com> Tested-by: Michael Meeks <michael.meeks@collabora.com>
* tdf#91574 fix presenter console not displayedChristian Lohmaier2015-07-031-1/+1
| | | | | | | | | | | | The bool simplification from 3ac5138cfc01c1795ae05963fa4e5a6c7d22f4de accidentally changed the logic of the return statement Change-Id: Id6075356184e340dc970d2e59d1fb1d1a173d0a7 Reviewed-on: https://gerrit.libreoffice.org/16724 Reviewed-by: Michael Stahl <mstahl@redhat.com> Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
* Fix typosAndrea Gelmini2015-06-281-1/+1
| | | | | | | Change-Id: I80995af6459af9f647e66d1c85da2eedad2647b9 Reviewed-on: https://gerrit.libreoffice.org/16523 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Julien Nabet <serval2412@yahoo.fr>
* Revert "Typo: iff->if"Julien Nabet2015-06-231-1/+1
| | | | | | This reverts commit cf92da3d6e1de14756efe3f1ee79f393a2f3787d. iff can mean "if and only if" so not a typo
* Typo: iff->ifJulien Nabet2015-06-231-1/+1
| | | | Change-Id: I3fc60856b5a56e71d70b55c89323be074bdec3b3
* Replace boost::scoped_array<T> with std::unique_ptr<T[]>Takeshi Abe2015-06-177-16/+12
| | | | | | | | | | | | | | | | | | | | | | This may reduce some degree of dependency on boost. Done by running a script like: git grep -l '#include *.boost/scoped_array.hpp.' \ | xargs sed -i -e 's@#include *.boost/scoped_array.hpp.@#include <memory>@' git grep -l '\(boost::\)\?scoped_array<\([^<>]*\)>' \ | xargs sed -i -e 's/\(boost::\)\?scoped_array<\([^<>]*\)>/std::unique_ptr<\2[]>/' ... and then killing duplicate or unnecessary includes, while changing manually m_xOutlineStylesCandidates in xmloff/source/text/txtimp.cxx, extensions/source/ole/unoconversionutilities.hxx, and extensions/source/ole/oleobjw.cxx. Change-Id: I3955ed3ad99b94499a7bd0e6e3a09078771f9bfd Reviewed-on: https://gerrit.libreoffice.org/16289 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
* inline COMPHELPER_SERVICEDECL_ macroNoel Grandin2015-06-156-8/+35
| | | | | | it was not adding any value Change-Id: I6a911aac81e883f9155ed0cf24bfb7904efb3f92
* Cleanup crossplatform cairoMox Soini2015-06-111-93/+4
| | | | | | | | | | | | Mac / Win cairo is not built / used, this cairo code can be removed to reduce code bloat. 4th Revised version with Android fixes Change-Id: I6e56850f535ca13b77839b7d67e227c5f39d388e Reviewed-on: https://gerrit.libreoffice.org/16218 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk>
* convert expressions like 'size() == 0' to 'empty()'Noel Grandin2015-06-111-1/+1
| | | | Change-Id: Ia5c8c0f38a347f398d587970a22e03f29ffd37af
* loplugin:cstylecast: deal with remaining pointer castsStephan Bergmann2015-06-084-7/+7
| | | | Change-Id: I8e3adffc4eca232889bbffd8efca48bad78a8b91
* cppcheck: noExplicitConstructorCaolán McNamara2015-06-089-10/+10
| | | | Change-Id: I8b03c27188b02c3c4a9a9fbbe0df1fb1ae1c5caf
* Revert "Cleanup crossplatform cairo"Jan Holesovsky2015-06-051-4/+93
| | | | | | Unfortunately this breaks Android and iOS. This reverts commit e4d55e8045c99af65a2cae57b26ec076825c9673.
* Cleanup crossplatform cairoMox Soini2015-06-041-93/+4
| | | | | | | | | | Mac / Win cairo is not built / used, this cairo code can be removed to reduce code bloat. Change-Id: I758d674291e25e0722f5c66a0096295e903786b1 Reviewed-on: https://gerrit.libreoffice.org/16049 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Thorsten Behrens <Thorsten.Behrens@CIB.de>
* convert B2DLineJoin to scoped enumNoel Grandin2015-06-041-5/+5
| | | | Change-Id: I4fb8f2c9e2080e3aa542d2ccf838e6784baa8429
* loplugin:loopvartoosmallNoel Grandin2015-05-291-5/+5
| | | | Change-Id: Icbe68b31d4ab04ca3cd9f572e3598413946a75c7
* fix Windows buildNoel Grandin2015-05-281-4/+1
| | | | Change-Id: I5f0489e2fe8fc08461b3409de2690988826ed334
* convert PARENTCLIPMODE constants to scoped enumNoel Grandin2015-05-281-1/+1
| | | | Change-Id: I36a53112e75fca9208b2f91e224a7c671695509b
* tdf#91426 - fix DX canvas - missed by VclPtr / Clang plugin.Michael Meeks2015-05-261-7/+6
| | | | Change-Id: I7969746b159051d65ea974b89ba458d0d14f466a
* callcatcher: update unused codeCaolán McNamara2015-05-256-35/+0
| | | | Change-Id: I735264eb32ebdd165d23a4717f0329f666b8b140
* convert ANTIALIASING constants to scoped enumNoel Grandin2015-05-208-31/+31
| | | | Change-Id: I175b8ea4e8bc01c3cdd3dd90506eba01b35e0085
* convert DRAWMODE constants to scoped enumNoel Grandin2015-05-202-5/+5
| | | | Change-Id: I36cbe8057d09226f8b302963bdd94dc5600b686f
* std::ptr_fun became deprecated since C++11Takeshi Abe2015-05-081-4/+4
| | | | | | | Change-Id: If93609d032c9864e0251230bf9ed76e6b6741a62 Reviewed-on: https://gerrit.libreoffice.org/15672 Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
* simplify code by replacing std::find_if with std::any_of/std::none_ofTakeshi Abe2015-05-072-8/+8
| | | | | | | 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>
* convert BMP_MIRROR constants to scoped enumNoel Grandin2015-05-061-2/+2
| | | | | | | and fix bug in svx/source/xoutdev/_xoutbmp.cxx,XOutBitmap::MirrorGraphic where it was not correctly translating between different flags types. Change-Id: I78b4965544da2aa54b67ec18307fa54c8ffc3aee
* convert BMP_SCALE constant to scoped enumNoel Grandin2015-05-062-2/+2
| | | | Change-Id: Ibc9f88d2588c028cd71aa86c26d970a73025ef22
* convert CONFIG_MODE constants to scoped enumNoel Grandin2015-05-061-1/+1
| | | | Change-Id: I1b0fb976e7bf8d7de06da0c30012e8bba1320498
* Get rid of the initial :: for the vcl namespaceTor Lillqvist2015-04-3013-46/+46
| | | | | | | | | | | We used it all over the place without leading :: already anyway, even in many files in include. So let's be consistent. In the majority of cases, prefer the easier-on-the-eyes choice, not the "safe" one. In those files in include where *all* existing uses of ::vcl:: indeed used the :: prefix, there let's keep it for consistency. Change-Id: If99cb41d3bf290d38c601d91125c3c8d935e61d0