summaryrefslogtreecommitdiffstats
path: root/canvas
Commit message (Collapse)AuthorAgeFilesLines
* Replaced boost::bind by std::bind in canvas/source/opengl/ogl_canvashelper.cxxMario J. Rugiero2015-11-021-12/+11
| | | | | | | | | | Chose not to use lambdas instead because the number of arguments in these calls would have made it way too verbose, hurting readability. Change-Id: I8662a937ec10ca0fac6cd7e0d78b6268e8023ada Reviewed-on: https://gerrit.libreoffice.org/19707 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: David Tardon <dtardon@redhat.com>
* Partial revert 'for_each -> range-based'Thorsten Behrens2015-11-021-3/+4
| | | | | | | | | There was a subtle change in semantics, the earlier loop was only iterating until aEnd, covering only unique elements of the vector. This partially reverts commit c04fd82433e9155ad61ebd92ca7b43a729a87890 Change-Id: I75c5f1b343c3fa8855f6d7f8f706b9259bc9f6df
* boost::bind -> lambdas in canvas/source/simplecanvas/simplecanvasimpl.cxxMario J. Rugiero2015-11-011-10/+5
| | | | | | | Change-Id: If768595d3c3ec294d5a59ad42f26aacfd6e0fda3 Reviewed-on: https://gerrit.libreoffice.org/19708 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
* boost::bind -> lambdas in canvas/source/vcl subtreeMario J. Rugiero2015-11-012-16/+7
| | | | | | | Change-Id: I2b2e746197b94222c73b78379d64086adfb5e693 Reviewed-on: https://gerrit.libreoffice.org/19712 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* std::for_each -> range based for in canvas/source/tools/verifyinput.cxxMario J. Rugiero2015-11-011-6/+5
| | | | | | | Change-Id: I71c714a459ea81a268f93f8ffc5286eb532794ca Reviewed-on: https://gerrit.libreoffice.org/19711 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* for_each -> range-based for in canvas/source/tools/spriteredrawmanager.cxxMario J. Rugiero2015-11-011-8/+7
| | | | | | | Change-Id: Ibe089e53488a9ba6baded637b17a1aeee0908f79 Reviewed-on: https://gerrit.libreoffice.org/19709 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* boost::bind -> lambdas in canvas/source/tools/surface.cxxMario J. Rugiero2015-11-011-7/+3
| | | | | | | Change-Id: I9a9c4832cf18b10e15d2a4de8944b404b082e990 Reviewed-on: https://gerrit.libreoffice.org/19710 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* Replace boost::bind by lambdas in canvas/source/directx tree.Mario J. Rugiero2015-10-312-9/+2
| | | | | | | Change-Id: I1622f563b921a03234b9cb02de7c8abd00444129 Reviewed-on: https://gerrit.libreoffice.org/19703 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
* tdf#93684 - Remove SalFrame::SyncRyan McCoskrie2015-10-301-1/+1
| | | | | | | | | Change-Id: Icae9cf0a17f9cdf9a671c7e7278019d6980602b1 Reviewed-on: https://gerrit.libreoffice.org/18453 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: jan iversen <jani@documentfoundation.org> Reviewed-by: Michael Meeks <michael.meeks@collabora.com> Tested-by: Michael Meeks <michael.meeks@collabora.com>
* tdf#92272: We need to twiddle the bytes from BGR to RGBTor Lillqvist2015-10-301-1/+26
| | | | | | | | | | | | As far as I see, the surfaces created in the DX canvas are of D3DFMT_A8R8G8B8 or D3DFMT_X8R8G8B8 format, which means that the bytes (in memory order, on little-endian Windows) are B,G,R,A/X. So if the desired destination wants R,G,B,A, we need to swap the blue and red bytes. Let's hope this doesn't break anything else... Change-Id: I1b90d2cf95418f6557cac633ec6fce27599e8a61
* UNO: no need to use OUString constructor when calling createInstanceNoel Grandin2015-10-301-2/+2
| | | | | | | Change-Id: I37da28539b94706574116d0fff5f008aabfb5526 Reviewed-on: https://gerrit.libreoffice.org/19682 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
* use uno::Reference::set method instead of assignmentNoel Grandin2015-10-303-8/+7
| | | | Change-Id: I58410209f32f988f258a588364e0b037c2790211
* loplugin:unusedmethodsNoel Grandin2015-10-301-39/+0
| | | | | | | Change-Id: Id3b5cd75d4357336ed592ef11a3f34d209f8e95f Reviewed-on: https://gerrit.libreoffice.org/19636 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
* Replace "SAL_OVERRIDE" with "override" in LIBO_INTERNAL_ONLY codeStephan Bergmann2015-10-1238-315/+315
| | | | Change-Id: I2ea407acd763ef2d7dae2d3b8f32525523ac8274
* Replace "SAL_DELETED_FUNCTION" with "= delete" in LIBO_INTERNAL_ONLY codeStephan Bergmann2015-10-121-1/+1
| | | | Change-Id: I328ac7a95ccc87732efae48b567a0556865928f3
* eliminate typedef only being used onceNoel Grandin2015-10-072-3/+1
| | | | Change-Id: I1e54968a67f3edce3b2297ea1932d5b07e2da643
* inline ISurfaceProxySharedPtr typedefNoel Grandin2015-10-072-2/+2
| | | | | | some places were not using the typedef anyhow Change-Id: Idf92ad63b05101832827113701a85eca48c084c6
* Fix typosAndrea Gelmini2015-10-024-5/+5
| | | | | | | Change-Id: I776c517ba87838c0c3391f79010a30fa7fe2abd3 Reviewed-on: https://gerrit.libreoffice.org/18939 Reviewed-by: Samuel Mehrbrodt <Samuel.Mehrbrodt@cib.de> Tested-by: Samuel Mehrbrodt <Samuel.Mehrbrodt@cib.de>
* boost->stdCaolán McNamara2015-09-181-1/+0
| | | | | | | | 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>
* boost->stdCaolán McNamara2015-09-1429-81/+55
| | | | | | | | 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>
* boost->stdCaolán McNamara2015-09-131-2/+0
| | | | | | | | Change-Id: I9b4f884c6313a53fea543ea6f93175205351ad14 Reviewed-on: https://gerrit.libreoffice.org/18517 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
* coverity#1323755 Using invalid iteratorCaolán McNamara2015-09-111-1/+1
| | | | | | | try and help coverity to know that aEnd and maFragments.end() are the same place Change-Id: I9ce4187d5ed15d0ae26b69bb360dbd47c2a6a185
* tdf#94006 - need an explicit dispose for GLContext's SystemChildWindow.Michael Meeks2015-09-081-1/+1
| | | | | | | | | | | | Previously we would get an explicit ~OpenGLContext - and potentially leave FMR's around for other OGC users, now we treat the other users properly - we need an explicit dispose() to get Window::dispose ordering right. Change-Id: I5edcbd73399b6db3dbcfb391570f364f9ab0c70d Reviewed-on: https://gerrit.libreoffice.org/18412 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
* 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