summaryrefslogtreecommitdiffstats
path: root/cppcanvas
Commit message (Collapse)AuthorAgeFilesLines
* loplugin:useuniqueptr extend to check local varsNoel Grandin2017-12-051-5/+2
| | | | | | | | | | | | | | just the simple and obvious case for now, of a local var being allocated and deleted inside a single local block, and the delete happening at the end of the block Reviewed-on: https://gerrit.libreoffice.org/33749 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk> (cherry picked from commit 2489000d3fd66319a8355fd4e37cfdfda47296d0) Change-Id: I3a7a094da543debdcd2374737c2ecff91d644625 (cherry picked from commit e540ccfb5eb43bd4a3d6920074dce8436720ba8e)
* EMF+: Restrict cropping to bitmap typesAron Budea2017-11-041-1/+3
| | | | | | | | Some EMF+ metafiles were incorrectly cropped. Regression from f115ec6ce1ff4bf6e68ceb441f716a4c70ecfdf6. Change-Id: I1bba0263cb999a0ed11feb25aaefc7b0416460be
* make SfxGetpApp just get and move GetOrCreate to the peripheryCaolán McNamara2017-05-301-0/+2
| | | | | | | | | | | | | | | | | | Reviewed-on: https://gerrit.libreoffice.org/34957 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com> (cherry-picked from commit 3d213b3dc5130bdbacbd64be00867eecad6373e8) Conflicts: svx/source/gengal/gengal.cxx Change-Id: If3b22635e46dbccf0fad101f51bb653cbbcd3a32 Reviewed-on: https://gerrit.libreoffice.org/38166 Reviewed-by: László Németh <nemeth@numbertext.org> Tested-by: László Németh <nemeth@numbertext.org> Reviewed-by: Andras Timar <andras.timar@collabora.com> Tested-by: Andras Timar <andras.timar@collabora.com>
* tdf#106084 EMF+ Add support for EmfPlusTranslateWorldTransform recordBartosz Kosiorek2017-05-042-11/+49
| | | | | | | | | | | | | | The EmfPlusTranslateWorldTransform record performs a translation on the current world space transform. Change-Id: I3e5744060c8a6d758bcc2804c6798e0208d2191f Reviewed-on: https://gerrit.libreoffice.org/36287 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Bartosz Kosiorek <gang65@poczta.onet.pl> (cherry picked from commit fcb32f1cbc335a953cea62f66b9f50170263fb56) Reviewed-on: https://gerrit.libreoffice.org/37090 Reviewed-by: Thorsten Behrens <Thorsten.Behrens@CIB.de> (cherry picked from commit cb2f3e524919444d84974fe3216edd8fafc28e10)
* tdf#107159 EMF+ Add support for EmfPlusDrawArc recordBartosz Kosiorek2017-05-041-4/+13
| | | | | | | | | | | | | EmfPlusDrawArc record specifies drawing the arc of an ellipse. Change-Id: I87788a9b14f518a383cbc5b804af635427e4c395 Reviewed-on: https://gerrit.libreoffice.org/36537 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Bartosz Kosiorek <gang65@poczta.onet.pl> (cherry picked from commit 715df6e9f8926e511dbdee52367ec08676f71952) Reviewed-on: https://gerrit.libreoffice.org/37092 Reviewed-by: Thorsten Behrens <Thorsten.Behrens@CIB.de> (cherry picked from commit 0adc3c002ddd7ccea6a184d9274e4695b1a857b4)
* tdf#107016 EMF+ Add support of import EmfPlusRecordTypeDrawRects recordBartosz Kosiorek2017-05-041-31/+32
| | | | | | | | | | | | | EmfPlusDrawRectss record defines the pen strokes for drawing a series of rectangles. Change-Id: Iaa061f8502fb51de44d2e65a1c92daf297b79d2b Reviewed-on: https://gerrit.libreoffice.org/36261 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Bartosz Kosiorek <gang65@poczta.onet.pl> (cherry picked from commit a45feba83e5fb35eee1884887e8dbe795ebacde5) Reviewed-on: https://gerrit.libreoffice.org/36528 Reviewed-by: Thorsten Behrens <Thorsten.Behrens@CIB.de> (cherry picked from commit 08fe3b0f1c1e72f15db1cd70d85effa3d63db276)
* tdf#107034 EMF+ Add support for import EmfPlusDrawPie recordBartosz Kosiorek2017-05-041-8/+25
| | | | | | | | | | | | | The EmfPlusDrawPie record specifies drawing a section of the interior of an ellipse. Change-Id: Icd9d64117cc612d348c531932931f31e4d04e567 Reviewed-on: https://gerrit.libreoffice.org/36286 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Bartosz Kosiorek <gang65@poczta.onet.pl> (cherry picked from commit 2e7c94f5054dec4ab19c44209136c886793f0acb) Reviewed-on: https://gerrit.libreoffice.org/36530 Reviewed-by: Thorsten Behrens <Thorsten.Behrens@CIB.de> (cherry picked from commit bafb6b59cf590f939753fc0009b8328c280837a9)
* Replace EmfPlusRecordType decimal valuse with hexBartosz Kosiorek2017-04-181-33/+55
| | | | | | | | | | Change-Id: Ia5f248282e663e86b76a8c90fb5dd55d112f6912 Reviewed-on: https://gerrit.libreoffice.org/36260 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Bartosz Kosiorek <gang65@poczta.onet.pl> (cherry picked from commit 723ee43cf65f85d7b586ae8c98529a22621054a3) Reviewed-on: https://gerrit.libreoffice.org/36522 (cherry picked from commit 42bf09876a8cb7faef0c2490ff3e7531de4fec8f)
* tdf#101638 diameter and radius were mixed up, devide by twoPatrick J2017-02-071-2/+2
| | | | | | | | | | | Change-Id: Iaece1ca3439ab3e1029156a5847768728b53b96e Reviewed-on: https://gerrit.libreoffice.org/33717 Reviewed-by: Katarina Behrens <Katarina.Behrens@cib.de> Tested-by: Katarina Behrens <Katarina.Behrens@cib.de> (cherry picked from commit e3a47c09500828d27c096c5bfb461c61f6e71378) Reviewed-on: https://gerrit.libreoffice.org/33894 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk>
* cppcanvas, oox: add missing #include <osl/endian.h>Michael Stahl2016-11-211-0/+2
| | | | | | | | | | | These files use the macro OSL_BIGENDIAN but did not include the header that defines it. Found via: git grep -l OSL_.*ENDIAN | grep -E '\.h(xx)?$' | while read f; do grep -L endian\\.h $f ; done Change-Id: Iffb36336026e49a9c3d8fcf125a54ec857c7552e
* loplugin:unusedmethods unused return typesNoel Grandin2016-11-032-8/+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>
* Revert "tdf#96505 Get rid of cargo cult "long" integer literals"jan iversen2016-10-251-4/+7
| | | | | | | | | | | This partly reverts commit 8ae33b1652cb1e654c426350169d3bb9fa031a4f. Added a comment why 1L is needed in this file. Change-Id: I92349b42decb964a45d4dc9c50b89cd22766acb9 Reviewed-on: https://gerrit.libreoffice.org/30259 Reviewed-by: jan iversen <jani@documentfoundation.org> Tested-by: jan iversen <jani@documentfoundation.org>
* tdf#96505 Get rid of cargo cult "long" integer literalsRosen2016-10-251-4/+4
| | | | | | | | | fixed all long integer literals in file Change-Id: Ib557d88d4e4f17d5c334c92d611d002fe163f2f6 Reviewed-on: https://gerrit.libreoffice.org/29235 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: jan iversen <jani@documentfoundation.org>
* convert MapUnit to scoped enumNoel Grandin2016-10-051-1/+1
| | | | | | | | | | | | 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>
* loplugin:singlevalfields in basic..idlNoel Grandin2016-09-161-6/+0
| | | | | | | | | | | Also fix obvious bug in the initialisation of the connectivity::odbc::OConnection::m_bClosed field. Probably closes some kind of connection leak there. Change-Id: I04579cf91bcd6d6c51c697d83971da4142743a82 Reviewed-on: https://gerrit.libreoffice.org/28932 Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk> Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
* replace sal_Size with std::size_t (or sal_uInt64 for SvStream pos)Michael Stahl2016-09-151-4/+4
| | | | | | | | ... except in include/rtl, include/sal, include/uno, where sal_Size is retained for compatibility, and where callers of rtl functions pass in pointers that are incompatible on MSVC. Change-Id: I8344453780689f5120ba0870e44965b6d292450c
* loplugin:override: No more need for the "MSVC dtor override" workaroundStephan Bergmann2016-09-1313-17/+17
| | | | | | | | | | | | | | | | | 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
* convert GradientStyle to scoped enumNoel Grandin2016-09-051-7/+7
| | | | Change-Id: Ib740da708612df7a5f4b8c82262b9b1bd436604d
* convert LineStyle to scoped enumNoel Grandin2016-09-051-3/+3
| | | | Change-Id: I30cfa5a0649b806604c443f55683d1f2a430983d
* loplugin:stringconstant: adapt to improved OUStringLiteral1 (cppcanvas)Stephan Bergmann2016-08-301-1/+1
| | | | Change-Id: I98a9e1404c2a9a39c8f930609c199c75f0a572f1
* Revert "temp tinderbox fix"Caolán McNamara2016-07-281-0/+1
| | | | | | This reverts commit d4cb9c3a8851cb874fd8b6c77d4d7521da7579b7. Change-Id: Ifd11427ea6b4137d0e0c007e7f140fbc6caa3f31
* 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>
* Clean up uses of Any::getValue() in cppcanvasStephan Bergmann2016-06-101-1/+2
| | | | Change-Id: I9a69b35ee7dc2da3912a72361e0e742bf480e6be
* tdf#94698 cleanup Makefiles. Get rid of udkapi and offapiGleb Mishchenko2016-06-092-8/+2
| | | | | | | | | | | | | | | | | | | | This patch changes occurences in makefiles from $(eval $(call gb_CppunitTest_use_api,comphelper_test_config, \ udkapi \ offapi \ )) to $(eval $(call gb_CppunitTest_use_sdk_api,comphelper_test_config)) (corrected instead of abandon, jani) Change-Id: Ic96ec65d82d7452e288f05a8b6d576ef543b068e Reviewed-on: https://gerrit.libreoffice.org/23426 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: jan iversen <jani@documentfoundation.org> Tested-by: jan iversen <jani@documentfoundation.org>
* tools: rename SvStream::Read/Write to ReadBytes/WriteBytesMichael Stahl2016-06-061-1/+1
| | | | | | | Change-Id: Ib788a30d413436aa03f813aa2fddcbc4d6cd2f9a Reviewed-on: https://gerrit.libreoffice.org/25972 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Michael Stahl <mstahl@redhat.com>
* Convert ComplexTextLayoutMode to scoped enumNoel Grandin2016-06-031-8/+8
| | | | | | | Change-Id: I257f8e15fec92f0701235a6fe4b6a2272498c4f1 Reviewed-on: https://gerrit.libreoffice.org/25667 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* Convert FontRelief to scoped enumNoel Grandin2016-05-302-5/+5
| | | | | | | Change-Id: I545c83d742db27f1a004b66e5eb1ef768f62e011 Reviewed-on: https://gerrit.libreoffice.org/25626 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* convert FONT_EMPHASIS_MARK to scoped enumNoel Grandin2016-05-092-3/+3
| | | | | | | Change-Id: I137c78b337e57d3442db08334128e79d186b278f Reviewed-on: https://gerrit.libreoffice.org/24753 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
* clang-tidy modernize-loop-convert in c*Noel Grandin2016-04-211-2/+2
| | | | Change-Id: I77d2548f8be97792660761e6156cd24734a95aaf
* clang-tidy performance-unnecessary-copy-initializationNoel Grandin2016-04-181-2/+1
| | | | | | | | | | | probably not much performance benefit, but it sure is good at identifying leftover intermediate variables from previous refactorings. Change-Id: I3ce16fe496ac2733c1cb0a35f74c0fc9193cc657 Reviewed-on: https://gerrit.libreoffice.org/24026 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
* clang-tidy performance-unnecessary-value-param in variousNoel Grandin2016-04-141-1/+1
| | | | | | | Change-Id: I7168d44dab8e6a8e37bb7920d744ff32f5e52907 Reviewed-on: https://gerrit.libreoffice.org/24019 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* loplugin:passstuffbyref in variousNoel Grandin2016-04-143-4/+4
| | | | Change-Id: I80070c83204e531c2f599f8a56193d6ffe0e5022
* tdf#94306 remove unused boost dependenciesJochen Nitschke2016-04-131-1/+0
| | | | | | | | | | remove <boost/noncopyable.hpp> in pch and remove boost from makefile if it was the only boost entry. Change-Id: Icb945ae59c137571f4f63807601738eea5c3e831 Reviewed-on: https://gerrit.libreoffice.org/24061 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Michael Stahl <mstahl@redhat.com>
* cleanup: remove unused com/sun/star/uno includesJochen Nitschke2016-04-121-2/+0
| | | | | | | | | | | | Sequence.h(xx), Any.h(xx) and Type.h(xx) and remove unused using-declarations from these files. Add a few missing includes provided by them. Change-Id: I6b91b6d1fdf9d0496dd546c0aab9bdcc6831a5d4 Reviewed-on: https://gerrit.libreoffice.org/23805 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* loplugin:unreffun (missing "= delete")Stephan Bergmann2016-04-081-2/+2
| | | | Change-Id: Ie9252e29b06242e8ab0b839f52de8b81897d269e
* tdf#94306 replace boost::noncopyable in c...Jochen Nitschke2016-04-087-24/+36
| | | | | | | | | | | | | | comphelper, connectivity and cppcanvas. Replace with C++11 delete copy-constructur and copy-assignment. Removed unused boost/noncopyable.hpp includes from some source files in cppcanvas. Change-Id: I90780820e21fbfd291ac10c266e7d16616e3a81b Reviewed-on: https://gerrit.libreoffice.org/23905 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Michael Stahl <mstahl@redhat.com>
* tdf#97525 vcl: make DeletionListener internalJochen Nitschke2016-03-311-1/+0
| | | | | | | | | | | | move include/vcl/impdel.hxx to vcl/inc/ include impdel.hxx in salframe.hxx remove vcl/impdel.hxx includes in pch add missing <list> includes Change-Id: Id146363b2e20ce0238542929c26a83efb1e8c4bd Reviewed-on: https://gerrit.libreoffice.org/23664 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* loplugin:constantparams in vcl/Noel Grandin2016-03-161-1/+1
| | | | | | | | | also some improvements to the plugin Change-Id: I0e3a519d70756e577fcb1bd47dd66864b5b4c871 Reviewed-on: https://gerrit.libreoffice.org/23289 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
* tdf#95709 adapt clip polygon for transparence groupsArmin Le Grand2016-02-281-0/+10
| | | | | | | | | | | | | | For presentation transparence groups are rendered by the canvas using VirtualDevices and Bitmaps. To adapt for then painting the Bitnmap to the canvas the RenderState needs to be adapted to e.g. reflect the already applied scaling. Missing was to adapt a set clip polyPolygon to be synchronized with the new RenderState transformation. Change-Id: If760a8076e016a6dcf306db71971f1fd2fef7017 Reviewed-on: https://gerrit.libreoffice.org/22720 Reviewed-by: Thorsten Behrens <Thorsten.Behrens@CIB.de> Tested-by: Thorsten Behrens <Thorsten.Behrens@CIB.de>
* loplugin:write only fieldsNoel Grandin2016-02-225-17/+0
| | | | Change-Id: I44f249a17d0a510ec63a488b656d57a1a392f821
* Remove excess newlinesChris Sherlock2016-02-096-26/+0
| | | | | | | | | | | | | | | | | | | A ridiculously fast way of doing this is: for i in $(pcregrep -l -M -r --include='.*[hc]xx$' \ --exclude-dir=workdir --exclude-dir=instdir '^ {3,}' .) do perl -0777 -i -pe 's/^ {3,}/ /gm' $i done Change-Id: Iebb93eccbee9e4fc5c4380474ba595858a27ac2c Reviewed-on: https://gerrit.libreoffice.org/22224 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Chris Sherlock <chris.sherlock79@gmail.com>
* vcl: cmdevt.[hc]xx -> commandevent.[hc]xxChris Sherlock2016-02-081-1/+1
| | | | | | | Change-Id: I77b30f28ae5a6fad360d7cada9acfaa9c324408b Reviewed-on: https://gerrit.libreoffice.org/22216 Reviewed-by: Chris Sherlock <chris.sherlock79@gmail.com> Tested-by: Chris Sherlock <chris.sherlock79@gmail.com>
* vcl: bmpacc.hxx -> bitmapaccess.hxxChris Sherlock2016-02-061-1/+1
| | | | Change-Id: I4bb19d6103c4a6a902d86b62a857e3478493924c
* vcl: split exception types from input typesChris Sherlock2016-02-061-6/+3
| | | | | | | Having them in the file apptypes.hxx isn't necessary helpful, IMO so I've split the types into inputtypes.hxx and exceptiontypes.hxx Change-Id: I89a1ff168c3ae276b2f5486669d4ec2dda062d57
* loplugin:fpcomparison in cppcanvas/Noel Grandin2016-02-041-2/+2
| | | | | | | Change-Id: I23d7e217f2616b488579be0617dbc14cd407540f Reviewed-on: https://gerrit.libreoffice.org/21868 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* vcl: take into account the font width is the average font widthChris Sherlock2016-02-042-5/+5
| | | | | | | | | | | | | | | | | | | | | | | | | I'm changing the Font class function names: - SetSize -> SetFontSize - GetSize -> GetFontSize - SetHeight -> SetFontHeight - GetHeight -> GetFontHeight - SetWidth -> SetAverageFontWidth - GetWidth -> GetAverageFontWidth That's because it really makes no sense to say that there is a single constant font width because obviously proportional fonts don't have one - the best we can do is an average font width, which is what folks like Microsoft sort of do already. On a fixed font, the average is still accurate, for obvious reasons :-) I'm also not a fan of GetSize/SetSize as I find it a might too generic. Change-Id: Ib80a604ba62d6883fd6cbc7994da763976be5c70 Reviewed-on: https://gerrit.libreoffice.org/22069 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Chris Sherlock <chris.sherlock79@gmail.com>
* remove B2DLineJoin::MiddleCaolán McNamara2016-02-021-2/+2
| | | | | | | and consistently map css::drawing::LineJoint_MIDDLE to the same thing that css::drawing::LineJoint_MITRE points to everywhere else Change-Id: I77b7586ea13f3fe84c0529172758256666488d36
* boost::hash->std::hashNoel Grandin2016-02-021-1/+0
| | | | | | | Change-Id: If4d1e4071995f07212fad958b0226d5824d168f8 Reviewed-on: https://gerrit.libreoffice.org/21989 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
* OSL_THIS_FUNC is a rip-off of BOOST_CURRENT_FUNCTION so we can use thatCaolán McNamara2016-01-311-1/+0
| | | | | | | | | | as a direct drop in I guess Change-Id: I3add63f1459f4e659019bd6db54da2f5431958ce Reviewed-on: https://gerrit.libreoffice.org/21941 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
* tools: rename FontUnderline to FontLineStyleChris Sherlock2016-01-313-41/+41
| | | | | | | Change-Id: I4750ad8569a1003b2f8c29052f3e25003ee433ca Reviewed-on: https://gerrit.libreoffice.org/21892 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Chris Sherlock <chris.sherlock79@gmail.com>