summaryrefslogtreecommitdiffstats
path: root/cppcanvas
Commit message (Collapse)AuthorAgeFilesLines
* aliasing(?) gives a ever so slightly different color on hidpiCaolán McNamara2017-05-161-1/+4
| | | | | | | | | | e.g. without hidpi this could probably be reproduced with export SAL_USE_VCLPLUGIN=gtk3 export GDK_BACKEND=x11 export GDK_SCALE=2 and CppunitTest_cppcanvas_emfplus fails with 0x00ff00 instead of 0x00fe00 Change-Id: I1ecd7dce7703ed50fe396e007424a3ec4252d063
* tdf#43157: convert cppcanvas module away from OSL_ASSERTChris Sherlock2017-05-143-3/+3
| | | | | | | | | | 2nd attempt, this time I've decided makes more sense to just produce a warning. Change-Id: Iaf373a2d38e8f47266b00ea869de38a584788d45 Reviewed-on: https://gerrit.libreoffice.org/37598 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Chris Sherlock <chris.sherlock79@gmail.com>
* EMF+ tdf#31814 Add support of reading EmfPlusBoundaryPointDataBartosz Kosiorek2017-05-123-68/+87
| | | | | | | | | | The EmfPlusBoundaryPointData object specifies a closed cardinal spline boundary for a gradient brush. This data is starting point for displaying correctly gradients. Change-Id: I91b01417c6dc00a04dabfc5a035afe9085999240 Reviewed-on: https://gerrit.libreoffice.org/37519 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Bartosz Kosiorek <gang65@poczta.onet.pl>
* EMF+ We already implemented most used EMF+ records.Bartosz Kosiorek2017-05-091-6/+5
| | | | | | | | | | Now it's time to enable warnings, to track not implemented cases, to have clear notification what is wrong with imported image. Change-Id: Ic880a5b7df03e4a98d4700bebb3d0f6d77980505 Reviewed-on: https://gerrit.libreoffice.org/37407 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Bartosz Kosiorek <gang65@poczta.onet.pl>
* tdf#31814 EMF+ Fix an issue when not all elements were displayedBartosz Kosiorek2017-05-092-8/+15
| | | | | | | | | | | On on EMF+ images generated by ChemDraw, some elements were not displayed. After investigation, occurs that position of points was not read properly in same cases. This commit fixes such cases. Change-Id: I1d01d8defc41f4e437a669ef1268b8e33823cfc1 Reviewed-on: https://gerrit.libreoffice.org/37406 Reviewed-by: Bartosz Kosiorek <gang65@poczta.onet.pl> Tested-by: Bartosz Kosiorek <gang65@poczta.onet.pl>
* tdf#31814 Introduce minimal value of line width to fix EMF+ import issuesBartosz Kosiorek2017-05-071-5/+9
| | | | | | | | | | | Based on observation of different EMF+ files (eg. exported by ChemDraw), there is minimal value of line width. This commit is fixing all such import issues. Change-Id: I5c26a1b511363eb1e429bf770ad0f482b124ecb5 Reviewed-on: https://gerrit.libreoffice.org/37319 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Bartosz Kosiorek <gang65@poczta.onet.pl>
* revert OSL_ASSERT changesChris Sherlock2017-05-073-3/+3
| | | | Change-Id: I365d140446bd2a62cf8256acbfdd53fe72987380
* tdf#43157: convert cppcanvas module away from OSL_ASSERT to assertChris Sherlock2017-05-073-3/+3
| | | | Change-Id: I1db04c35cc004a33b0d2586f9eb1d21bc545eb81
* EMF Replace magic numbers of pen and brush objectsBartosz Kosiorek2017-05-063-27/+58
| | | | | | | Change-Id: Ia3b6208b3d42b46fbdc9d61bbdea1c790aaf3ba0 Reviewed-on: https://gerrit.libreoffice.org/37268 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Bartosz Kosiorek <gang65@poczta.onet.pl>
* Fix typosAndrea Gelmini2017-05-052-2/+2
| | | | | | | | | | To complete commit 0ef94e2b559547bc4e906e7f24e57ff5d642e108 and f12096272e684ddcd8ffa4e34dcb0a680cc594c2 Change-Id: Ie86bbfbd58dd728a013bef221e4d5c8fbcaf8e03 Reviewed-on: https://gerrit.libreoffice.org/37199 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Julien Nabet <serval2412@yahoo.fr>
* Fix number of EmfPlusEndContainer recordBartosz Kosiorek2017-05-041-1/+1
| | | | | | | | | | EmfPlusEndContainer record closes a graphics state container that was previously opened by a begin container operation. Change-Id: I80d977ba53b98ef87b70e0dca2008c1980f901f9 Reviewed-on: https://gerrit.libreoffice.org/37217 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Bartosz Kosiorek <gang65@poczta.onet.pl>
* loplugin:checkunusedparams in cppcanvas and dbaccessNoel Grandin2017-05-023-3/+3
| | | | | | | Change-Id: Ia66e6326f3d1750b4e46a3f9f7127e7ed4aa99ca Reviewed-on: https://gerrit.libreoffice.org/37068 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* crashtesting: more crashing on exporting fdo31814-3.emf to odgCaolán McNamara2017-04-271-3/+6
| | | | Change-Id: I4cb359b426daed6c5d83c4ed4b85fe87de9d739a
* crashtesting: crash on exporting fdo31814-3.emf to odgCaolán McNamara2017-04-271-1/+1
| | | | | | | | | | | | | | | | | | | since commit f9af0abce2c266a53f536a207a62bc691213f6a0 Date: Sat Apr 15 01:44:46 2017 +0200 EMF+ Add initial support for EmfPlusStringFormat Object The EmfPlusStringFormat object specifies text layout, display manipulations, and language identification. With this commit the support for font language and country was added. Also Bold style is applied, for strings which needs that. elsewhere the pattern is to mask with 0xff to restrict to the hardcoded 256 size, so do that here too Change-Id: Iba5bcc3fabe836c963b3eef47a864e87425675c7
* tdf#47243 tdf#39327 tdf#103639 Proper scaling of SetPageTransformBartosz Kosiorek2017-04-271-13/+6
| | | | | | | | | | | | | The EmfPlusSetPageTransform record specifies scaling factors and units for converting page space coordinates to device space coordinates. In previous implementation I made scaling after translating page scaling. For some images it was working correctly (if there wasn't translating). WIth this commit I fixed this issue. Change-Id: I1e9e78ecdab663328f86a960d8547ad0a7fc1314 Reviewed-on: https://gerrit.libreoffice.org/37010 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Bartosz Kosiorek <gang65@poczta.onet.pl>
* errinf.hxx moved out of tools and into vcl moduleChris Sherlock2017-04-251-1/+1
| | | | | | | | | | ErrorInfo has a hard depency on VCL, yet is in the tools package. It is more appropriate to have it reside in the VCL module. Change-Id: Ica54a46c3a7f86cf0331ed7245234bea69c05650 Reviewed-on: https://gerrit.libreoffice.org/36839 Reviewed-by: Chris Sherlock <chris.sherlock79@gmail.com> Tested-by: Chris Sherlock <chris.sherlock79@gmail.com>
* tools: svstream.hxx needs only errcode.hxx & not errinf.hxxChris Sherlock2017-04-251-0/+1
| | | | | | | Change-Id: Ia28e35ae5af4f601e9a586a3deffbcd61702b0ca Reviewed-on: https://gerrit.libreoffice.org/36896 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Chris Sherlock <chris.sherlock79@gmail.com>
* tdf#47243 tdf#39327 Add support for SetPageTransformBartosz Kosiorek2017-04-241-4/+30
| | | | | | | Change-Id: Iea78cf1e9ca99c97814d9eaa4a65ad4b83a2bbe2 Reviewed-on: https://gerrit.libreoffice.org/36749 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Bartosz Kosiorek <gang65@poczta.onet.pl>
* EMF+ Add initial support for EmfPlusStringFormat ObjectBartosz Kosiorek2017-04-245-8/+155
| | | | | | | | | | | | The EmfPlusStringFormat object specifies text layout, display manipulations, and language identification. With this commit the support for font language and country was added. Also Bold style is applied, for strings which needs that. Change-Id: Id9735a9ddb959aebd20d6d4d1bb5da0fa9c1fb85 Reviewed-on: https://gerrit.libreoffice.org/36555 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Bartosz Kosiorek <gang65@poczta.onet.pl>
* gbuild: Remove MSVC 2013 legacy codeDavid Ostrovsky2017-04-213-3/+0
| | | | | | | | | Uwinapi is discontinued. Change-Id: I063b4d0d8fab2d60de168e960a63b8181158ac01 Reviewed-on: https://gerrit.libreoffice.org/23198 Reviewed-by: David Ostrovsky <david@ostrovsky.org> Tested-by: David Ostrovsky <david@ostrovsky.org>
* remove unnecessary explicit linefeeds from end of SAL and OSL log callsNoel Grandin2017-04-211-1/+1
| | | | | | | Change-Id: I3fa363c8e76e6cfb297f4ec346e3f031c09d6fbf Reviewed-on: https://gerrit.libreoffice.org/36727 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* loplugin:unusedmethodsNoel Grandin2017-04-201-1/+0
| | | | | | | Change-Id: Ib7a9b1b0802ca751da258065e89b412b090bb672 Reviewed-on: https://gerrit.libreoffice.org/36718 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* tdf#107159 EMF+ Add support for EmfPlusDrawArc recordBartosz Kosiorek2017-04-161-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>
* tdf#106084 EMF+ Add support for EmfPlusScaleWorldTransform recordBartosz Kosiorek2017-04-131-1/+25
| | | | | | | | | The EmfPlusScaleWorldTransform record performs a scaling on the current world space transform. Change-Id: I03717b28a88d574d597174c6ca3446f937c96a15 Reviewed-on: https://gerrit.libreoffice.org/36320 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Bartosz Kosiorek <gang65@poczta.onet.pl>
* tdf#39053 EMF+ Draw an extra line between the last point and the first point.Bartosz Kosiorek2017-04-133-4/+12
| | | | | | | | | | | | The EmfPlusDrawlLines record specifies drawing a series of connected lines. Bit 0x2000 indicates whether to draw an extra line between the last point and the first point, to close the shape. In this commit support of additional line which close shape was added. Change-Id: I47ae3d8003cbfdd5b8ff5ba78e1ebe10f97af04b Reviewed-on: https://gerrit.libreoffice.org/36317 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Bartosz Kosiorek <gang65@poczta.onet.pl>
* tdf#106084 EMF+ Add support for EmfPlusTranslateWorldTransform recordBartosz Kosiorek2017-04-132-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>
* tdf#107034 EMF+ Add support for import EmfPlusDrawPie recordBartosz Kosiorek2017-04-131-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>
* tdf#107019 EMF+ Add support for import EmfPlusRecordTypeDrawBeziers recordBartosz Kosiorek2017-04-131-1/+49
| | | | | | | | | EmfPlusDrawBeziers record defines the pen strokes for drawing a Bezier spline. Change-Id: I6ae08a861bcbadd373741781af0011528947243e Reviewed-on: https://gerrit.libreoffice.org/36280 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Bartosz Kosiorek <gang65@poczta.onet.pl>
* no need to use OUStringToOString in SAL_INFONoel Grandin2017-04-121-8/+2
| | | | | | | Change-Id: I707e0d72aba89b7e644def6f4c251e14f6599ad2 Reviewed-on: https://gerrit.libreoffice.org/36451 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* tdf#107016 EMF+ Add support of import EmfPlusRecordTypeDrawRects recordBartosz Kosiorek2017-04-111-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>
* Replace EmfPlusRecordType decimal valuse with hexBartosz Kosiorek2017-04-071-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>
* loplugin:redundantcast find cstyle double castsNoel Grandin2017-04-061-4/+4
| | | | | | | Change-Id: I5507be190dac781e5cdb545a60acf3d50056c9f8 Reviewed-on: https://gerrit.libreoffice.org/36187 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* tdf#82580 tools: rename Rectangle to tools::RectangleMiklos Vajna2017-03-315-15/+15
| | | | | | | | | | | | | | | Mostly generated using make check COMPILER_EXTERNAL_TOOL=1 CCACHE_PREFIX=clang-rename-wrapper RENAME_ARGS="-qualified-name=Rectangle -new-name=tools::Rectangle" Except some modules have their own foo::tools namespace, so there have to use ::tools::Rectangle. This commit just moves the class from the global namespace, it does not update pre/postwin.h yet. Change-Id: I42b2de3c6f769fcf28cfe086f98eb31e42a305f2 Reviewed-on: https://gerrit.libreoffice.org/35923 Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk> Tested-by: Jenkins <ci@libreoffice.org>
* make SfxGetpApp just get and move GetOrCreate to the peripheryCaolán McNamara2017-03-081-0/+2
| | | | | | | | Change-Id: If3b22635e46dbccf0fad101f51bb653cbbcd3a32 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>
* FontDescription.IsVertical is a tristateNoel Grandin2017-02-281-2/+1
| | | | | | | | | found by still in progress "UNO scoped enum" change Change-Id: I0a21b89e77be165bc96e914017a402cf8724d218 Reviewed-on: https://gerrit.libreoffice.org/34719 Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk> Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
* Better make the zero-initialization more explicitStephan Bergmann2017-02-231-2/+10
| | | | | | | ...if initializing these members is necessary at all; a bit hard to tell from the code. Change-Id: I3ddb25c35f0d1b44a092bdb2cb0192ce619f947c
* Drop :: prefix from std in c*/Tor Lillqvist2017-02-155-27/+27
| | | | | | | Change-Id: If078cda95fa6ccd37270a5e9d81cfa0b84e71155 Reviewed-on: https://gerrit.libreoffice.org/34324 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Tor Lillqvist <tml@collabora.com>
* new loplugin:unusedenumconstantsNoel Grandin2017-02-032-3/+3
| | | | | | | | | | 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:useuniqueptr extend to check local varsNoel Grandin2017-02-011-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 Change-Id: I3a7a094da543debdcd2374737c2ecff91d644625 Reviewed-on: https://gerrit.libreoffice.org/33749 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* tdf#101638 diameter and radius were mixed up, devide by twoPatrick J2017-01-311-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>
* Remove dynamic exception specificationsStephan Bergmann2017-01-262-7/+7
| | | | | | | | | | | | | | | | | | | | | | | | | ...(for now, from LIBO_INTERNAL_CODE only). See the mail thread starting at <https://lists.freedesktop.org/archives/libreoffice/2017-January/076665.html> "Dynamic Exception Specifications" for details. Most changes have been done automatically by the rewriting loplugin:dynexcspec (after enabling the rewriting mode, to be committed shortly). The way it only removes exception specs from declarations if it also sees a definition, it identified some dead declarations-w/o-definitions (that have been removed manually) and some cases where a definition appeared in multiple include files (which have also been cleaned up manually). There's also been cases of macro paramters (that were used to abstract over exception specs) that have become unused now (and been removed). Furthermore, some code needed to be cleaned up manually (avmedia/source/quicktime/ and connectivity/source/drivers/kab/), as I had no configurations available that would actually build that code. Missing @throws documentation has not been applied in such manual clean-up. Change-Id: I3408691256c9b0c12bc5332de976743626e13960 Reviewed-on: https://gerrit.libreoffice.org/33574 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
* loplugin: unnecessary destructor cppcanvas..filterNoel Grandin2017-01-241-1/+0
| | | | | | | Change-Id: I009cdd60869a67d210edf879ddb451663a7b8676 Reviewed-on: https://gerrit.libreoffice.org/33491 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* Split emfplus.cxx into multiple filesMark Page2017-01-2316-1033/+1664
| | | | | | | | | | | | This helps make emfplus.cxx more readable. No source code was changed, only moved except these renamed to avoid inheritance warnings:: EMFPBrush::transformation renamed to brush_transformation EMFPPen::transformation renamed to pen_transformation Change-Id: I6952d6300c9c459833c2dda2b715d851b9e80de7 Reviewed-on: https://gerrit.libreoffice.org/31165 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: jan iversen <jani@documentfoundation.org>
* Dispose Window when doneStephan Bergmann2016-12-221-1/+1
| | | | Change-Id: I6b67d12174c3caa0a4bf230d87421f5768a331f8
* convert PolyStyle to scoped enumNoel Grandin2016-12-061-3/+3
| | | | | | | Change-Id: Ia35fdbb5e40888db9558988ac069741b31721f0e Reviewed-on: https://gerrit.libreoffice.org/31633 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* loplugin:unnecessaryoverride (dtors) in cppcanvasStephan Bergmann2016-11-282-8/+0
| | | | Change-Id: I4f6f6e36a965b5c01ffd51b1710fa8cc57d497fa
* 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>
generated by cgit (git 2.34.1) at 2024-07-25 16:07:44 +0000