summaryrefslogtreecommitdiffstats
path: root/drawinglayer
Commit message (Collapse)AuthorAgeFilesLines
* convert MapUnit to scoped enumNoel Grandin2016-10-056-21/+21
| | | | | | | | | | | | I left a prefix on the names "Map" so that I would not have to re-arrange each name too much, since I can't start identifiers with digits like "100thMM" And remove RSC_EXTRAMAPUNIT, which doesn't seem to be doing anything anymore. Change-Id: I5187824aa87e30caf5357b51b5384b5ab919d224 Reviewed-on: https://gerrit.libreoffice.org/29096 Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk> Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
* Fix typosAndrea Gelmini2016-10-012-2/+2
| | | | | | | Change-Id: I8374d6d08f4eb4ae2821e213371c615b92d7e9ab Reviewed-on: https://gerrit.libreoffice.org/29432 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* Resolves: tdf#101433 reset RasterOpMode on cached virtual device before reuseCaolán McNamara2016-09-291-0/+1
| | | | Change-Id: I178aa499b080b0b684d41fdd4115a6398f944c43
* loplogin:singlevalfields in include/Noel Grandin2016-09-162-6/+2
| | | | | | | Change-Id: I27842162fcf82120ecb811ee8e89e187430931fc Reviewed-on: https://gerrit.libreoffice.org/28931 Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk> Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
* loplugin:override: No more need for the "MSVC dtor override" workaroundStephan Bergmann2016-09-136-6/+6
| | | | | | | | | | | | | | | | | The issue of 362d4f0cd4e50111edfae9d30c90602c37ed65a2 "Explicitly mark overriding destructors as 'virtual'" appears to no longer be a problem with MSVC 2013. (The little change in the rewriting code of compilerplugins/clang/override.cxx was necessary to prevent an endless loop when adding "override" to OOO_DLLPUBLIC_CHARTTOOLS virtual ~CloseableLifeTimeManager(); in chart2/source/inc/LifeTime.hxx, getting stuck in the leading OOO_DLLPUBLIC_CHARTTOOLS macro. Can't remember what that isAtEndOfImmediateMacroExpansion thing was originally necessary for, anyway.) Change-Id: I534c634504d7216b9bb632c2775c04eaf27e927e
* Remove nonsense comments: // bitfieldTor Lillqvist2016-09-1312-13/+0
| | | | | | | Surely the actual bitfield syntax is enough to tell the code reader that it is a bitfield. Change-Id: Ic9552e01b19c8b34b2a17db56b9ff63e7c7de926
* loplugin:constantparam in dbaccess..editengNoel Grandin2016-09-132-4/+2
| | | | | | | Change-Id: I1d0dc3d89933d86cd229e503b350fccc1523dedc Reviewed-on: https://gerrit.libreoffice.org/28833 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* convert GradientStyle to scoped enumNoel Grandin2016-09-052-19/+19
| | | | Change-Id: Ib740da708612df7a5f4b8c82262b9b1bd436604d
* convert HatchStyle to scoped enumNoel Grandin2016-09-053-9/+9
| | | | Change-Id: Ibef51ae5ae135ae584791959ec3f7bf78c50e2a8
* convert LineStyle to scoped enumNoel Grandin2016-09-052-3/+3
| | | | Change-Id: I30cfa5a0649b806604c443f55683d1f2a430983d
* convert RasterOp to scoped enumNoel Grandin2016-09-053-10/+10
| | | | Change-Id: I136423c105316c9b5b18e64d04a248fd7ac5590b
* convert RefDevMode to scoped enumNoel Grandin2016-09-011-1/+1
| | | | | | and fix the bForceZeroExtleadBug TODO Change-Id: Iac9295c6ce31112d69a870e3a229823eb1e9a4f2
* Missing operator definitionStephan Bergmann2016-08-251-0/+6
| | | | Change-Id: I39d3b68ca317a4a259b77532efc0897fc89a6fe9
* cid#1371276 Missing move assignment operatorNoel Grandin2016-08-258-0/+88
| | | | | | | | also cid#1371234, cid#1371249, cid#1371208, cid#1371250, cid#1371258, cid#1371266, cid#1371257, cid#1371221 Change-Id: I18d952887b684e1999d003cec0f668789c851b52
* cid#1371277 Missing move assignment operatorNoel Grandin2016-08-251-0/+5
| | | | Change-Id: Ic737733f5951d5b21660a12a7a01202bd9bc303c
* cid#1371284 Missing move assignment operatorNoel Grandin2016-08-251-0/+11
| | | | Change-Id: I84d4cfe58f9659b468a1fd62b7974fcdbf0bb1f3
* cid#1371288 Missing move assignment operatorNoel Grandin2016-08-251-0/+11
| | | | Change-Id: I5181c76f558e1806de01f21422c715cc93f3cb3d
* fix windows build, doesn't like default move methodsNoel Grandin2016-08-252-4/+18
| | | | Change-Id: I6e756a11aa07722ff18ed13219b780f3db3bc4a0
* fix move operator= and add move constructorsNoel Grandin2016-08-252-14/+5
| | | | | | | | | followup to: 19828cc0 "cid#1371315 Missing move assignment operator" and 0e748707 "cid#1371320 Missing move assignment operator" Change-Id: Id2479fef41cf8d98eef9246f3a86e6c9289c3d58
* cid#1371309 Missing move assignment operatorNoel Grandin2016-08-251-0/+11
| | | | Change-Id: Ic00f39793f0341820ccce912f4350644a90104c5
* cid#1371315 Missing move assignment operatorNoel Grandin2016-08-251-0/+6
| | | | Change-Id: I9da0ffe856a0a40094c1c553bb2c5e05d8d3ef5d
* remove dead code in drawinglayerJochen Nitschke2016-08-031-102/+29
| | | | | | | | | | | | | | | | static bools were probably used for debugging proposes and should not be in master introduced in commits: 9f6018ec1472d7e4f2f26b300d8c00b09fda1fe8 ddcf9b9ff2caaffcc59d250b2d7f50ca3ab20330 d45ddb6d03846b0c576eeee062342962aa131bc0 7a652a2b2ab5e0d37e32185c8c5fac3af482bb76 and 70e3eb2c1762fb1ca097cf671e3c7ce3d0dfd1b7 Change-Id: Ided2bf923696cd9fc537f1cb4fedd1a7d4b7c5cd Reviewed-on: https://gerrit.libreoffice.org/26880 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Jochen Nitschke <j.nitschke+logerrit@ok.de>
* Resolves: tdf#101067 put SolarMutexGuard back in againCaolán McNamara2016-07-221-0/+1
| | | | | | | | | | | | | | | | | | | | | | | commit 4f27ff917237be96eec897d4af90a3379be904c6 Author: Tor Lillqvist <tml@iki.fi> Date: Mon Jul 29 18:24:23 2013 +0300 Avoid SolarMutex assertion in a dbgutil build when exiting Impress put in a SolarMutex in that ImpTimedRefDev dtor for what sounds like just this problem commit 9f0766917a4fb1bc8fe1786c3b46132dd63c1c66 Author: Armin Le Grand <Armin.Le.Grand@cib.de> Date: Fri Jul 1 14:40:00 2016 +0200 tdf#50613 add support to load charts asynchronously took it out again, to presumably move it into TextLayouterDevice but we destroy this thing asyncronously outside of TextLayouterDevice so lets put it back in again Change-Id: If801a701701a3d87fce2f76bc22bb3184b46743a
* add tagging to ThreadTasks so we don't need more one poolNoel Grandin2016-07-181-3/+6
| | | | | | | | | | | | | | | If more than one place in the code submits tasks to the shared pool, then waitTillDone() becomes unreliable. Add a tagging mechanism, so different callsites can wait on different sets of tasks. Also try to protect our worker threads against exceptions from the thread tasks code. Change-Id: Idde664ab50008d31a2dd73910bb22f50e62ae22f Reviewed-on: https://gerrit.libreoffice.org/27042 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* new loplugin unnecessary overrideNoel Grandin2016-07-151-0/+1
| | | | | | | Change-Id: I88d3e33823d68745b98625050a8a274f9ef04bcb Reviewed-on: https://gerrit.libreoffice.org/27135 Reviewed-by: Stephan Bergmann <sbergman@redhat.com> Tested-by: Stephan Bergmann <sbergman@redhat.com>
* tdf#82214 optimize PatternFillPrimitive and SVGArmin Le Grand2016-07-071-36/+37
| | | | | | | | | | Use buffering in the drawinglayer, and don't do slow stuff in the windows gdi renderer. Conflicts: svgio/source/svgreader/svgstyleattributes.cxx Change-Id: Id955ee6a3b03e568c2678f02d77af35d2e5ba1d4
* tdf#82214 optimize performance for primitivesArmin Le Grand2016-07-074-35/+253
| | | | | | | | | | | | See svg bug doc, which is processed quite slowly. Beyond needing faster renderers, there is also demand to improve the handling of primitives created by SVG import. Conflicts: drawinglayer/source/primitive2d/patternfillprimitive2d.cxx vcl/win/gdi/gdiimpl.cxx Change-Id: I10992a5746b8b2d6b50e3ee3fe415a035685c9ba
* tdf#50613 add support to load charts asynchronouslyArmin Le Grand2016-07-071-2/+2
| | | | | | | | | | | Generating primitives for chart visualisation can be moved to a paralell executed task that loads the chart, thus speeding up initial visualization. This is not possible for e.g. PDF or print targets, only for edit visualization. On fallback, the replacement images of the charts are used which are metafiles and have less quality as primitives, but load quicker. Change-Id: I68caa9e1bec50832bce535b5f54633d53cdef037
* comphelper::OBaseMutex -> cppu::BaseMutexNoel Grandin2016-07-041-2/+2
| | | | | | | | | convert usage of deprecated class, and remove the old class Change-Id: I19fb9271090d19e5531622b58492e1a848d306e2 Reviewed-on: https://gerrit.libreoffice.org/26843 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* Fix some spelling errors in comments and stringsOtto Kekäläinen2016-07-042-3/+3
| | | | | | | Change-Id: Iecd6b5e13d6be14651f77d8e37f01117ba15a11e Reviewed-on: https://gerrit.libreoffice.org/26883 Reviewed-by: jan iversen <jani@documentfoundation.org> Tested-by: jan iversen <jani@documentfoundation.org>
* loplugin:singlevalfields in drawinglayerNoel Grandin2016-06-273-62/+29
| | | | | | | Change-Id: I19f50b0afa624391bd0d8ada7afca822298e10cc Reviewed-on: https://gerrit.libreoffice.org/26655 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* Give unique, comprehensible names to timers tdf#97087emahaldar/em2016-06-131-1/+1
| | | | | | | Change-Id: I9f2be193b995d43d4e3440e55a025629fe5bcaed Reviewed-on: https://gerrit.libreoffice.org/26222 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* tdf#99519 Added more intelligent handling of animated GIFsArmin Le Grand2016-06-102-139/+386
| | | | | | | | | | | | | | | | Isolated to a single Primitive2D class based on the AnimatedSwitch- Primitive2D which does the specializing in one place. Buffers small GIFs completely, handles 1st frame always buffered, huge GIFs get animated by just playing he next frame. To reach more with the current approach we would have to re-implement AnimatedGIF import, replay it internally on a sys-specific Surface and blit the current content (with alpha) to our display Change-Id: I46c3325fa7936df73bea9a9284a0421f1475a34b Reviewed-on: https://gerrit.libreoffice.org/26103 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
* Convert ComplexTextLayoutMode to scoped enumNoel Grandin2016-06-032-9/+9
| | | | | | | Change-Id: I257f8e15fec92f0701235a6fe4b6a2272498c4f1 Reviewed-on: https://gerrit.libreoffice.org/25667 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* loplugin:constantparamsNoel Grandin2016-06-032-9/+6
| | | | | | | Change-Id: Ib162ba9297b9d900ea42c7e5216e152d3e58a361 Reviewed-on: https://gerrit.libreoffice.org/25769 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* tdf#95581: Assume tiny shears are rounding artefacts etc and can be ignoredTor Lillqvist2016-06-011-1/+4
| | | | | | See bug report for more discussion. Change-Id: I50ee82abac4ddfbdca0fb03d17c0518860466a52
* teach refcounting clang plugin about uno::ReferenceNoel Grandin2016-05-311-7/+7
| | | | | | | | | | | | | | 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>
* Convert FontRelief to scoped enumNoel Grandin2016-05-302-8/+8
| | | | | | | Change-Id: I545c83d742db27f1a004b66e5eb1ef768f62e011 Reviewed-on: https://gerrit.libreoffice.org/25626 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* Convert TextEffectStyle2D to scoped enumNoel Grandin2016-05-303-16/+16
| | | | | | | Change-Id: I1ef784473408a220788f0ce7550bf16809fa2cd2 Reviewed-on: https://gerrit.libreoffice.org/25625 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* convert TransparentType to scoped enumNoel Grandin2016-05-301-3/+3
| | | | | | | Change-Id: I25ec59fd41b5aa8741a9e864b5204481a84c3ba6 Reviewed-on: https://gerrit.libreoffice.org/25601 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* Convert GRAPHIC to scoped enumNoel Grandin2016-05-276-11/+11
| | | | | | | Change-Id: I1fd09a729cbda00f99841532e0dd3fa66bce7bea Reviewed-on: https://gerrit.libreoffice.org/25534 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* Convert Disposal to scoped enumNoel Grandin2016-05-251-8/+3
| | | | | | | | | and drop unused FULL value Change-Id: I3b9c26cb164785ef86f1a8d57cce962b015c85d6 Reviewed-on: https://gerrit.libreoffice.org/25432 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* Fix typosAndrea Gelmini2016-05-252-2/+2
| | | | | | | Change-Id: I860cc4aa04b4e5cd1437b2f24176ee2ccdec3266 Reviewed-on: https://gerrit.libreoffice.org/25420 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
* Convert GradientStyle to scoped enumNoel Grandin2016-05-236-30/+30
| | | | | | | Change-Id: I1da151dd25648e363ba88bf88caa3e820f0ee424 Reviewed-on: https://gerrit.libreoffice.org/25270 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* Convert HatchStyle to scoped enumNoel Grandin2016-05-237-21/+21
| | | | | | | Change-Id: I27878a67457d32e6922dfb6da73f896ff6ab1d01 Reviewed-on: https://gerrit.libreoffice.org/25271 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* Convert BreakupUnit to scoped enumNoel Grandin2016-05-222-5/+5
| | | | | | | Change-Id: Icda4584c79a8af2c2e5fc5a29c498f7869641aa5 Reviewed-on: https://gerrit.libreoffice.org/25299 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* Convert SpreadMethod to scoped enumNoel Grandin2016-05-221-9/+9
| | | | | | | Change-Id: I2a04247bd58ed6f2af8da7e109b31ff83d6d5e49 Reviewed-on: https://gerrit.libreoffice.org/25298 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* Convert HelplineStyle2D to scoped enumNoel Grandin2016-05-221-2/+2
| | | | | | | Change-Id: I32b3c61ed40fe041a395544c1ab32d8e1b5f89fa Reviewed-on: https://gerrit.libreoffice.org/25297 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* update unusedmethods plugin to deal with constructorsNoel Grandin2016-05-185-57/+0
| | | | | | | | | | and fix the operator< implementations in some of the other plugins too. Change-Id: Ie5631e0cdc8d2a994ad2af2533cdb558a6cfc035 Reviewed-on: https://gerrit.libreoffice.org/25057 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* Massage code to avoid warningsStephan Bergmann2016-05-121-6/+3
| | | | Change-Id: I847e1a109bd20e64161a7735ee30b2fe1ca507b0