path: root/cppcanvas/source/mtfrenderer
AgeCommit message (Collapse)AuthorFilesLines
2018-08-28Bin superfluous whitespace on empty lineTor Lillqvist1-1/+1
Change-Id: Ida0ab3cd78514b2c10f8e6655becefe794217f8c
2018-08-04Use more basegfx deg<->rad functions, instead of direct formulasMike Kaganski1-3/+3
Also make the functions constexpr. Due to slight changes in floating-point arithmetics (90.0 instead of 180.0, M_PI2 instead of M_PI resp.), results might differ in last digits (usually 17th decimal digit). This has lead to need to tweak char2dump's PieChartTest unit test. Change-Id: I20323dd7dab27e4deb408ea4181e390cc05e7cd3 Reviewed-on: Tested-by: Jenkins Reviewed-by: Tamás Zolnai <> Reviewed-by: Mike Kaganski <>
2018-07-28loplugin:stringloop cui,dbaccessNoel Grandin1-4/+4
Change-Id: I32b63d2435d36b869823ea022ebd7f8347a46ea0 Reviewed-on: Tested-by: Jenkins Reviewed-by: Noel Grandin <>
2018-07-16Add missing sal/log.hxx headersGabor Kelemen17-0/+17
rtl/string.hxx and rtl/ustring.hxx both unnecessarily #include <sal/log.hxx> (and don't make use of it themselves), but many other files happen to depend on it. This is a continuation of commit 6ff2d84ade299cb3d14d4110e4cf1a4b8070c030 to be able to remove those unneeded includes. This commit adds missing headers to every file found by: grep -FwL sal/log.hxx $(git grep -Elw 'SAL_INFO|SAL_INFO_IF|SAL_WARN|SAL_WARN_IF|SAL_DETAIL_LOG_STREAM|SAL_WHERE|SAL_STREAM|SAL_DEBUG') to directories from connectivity to cui Change-Id: I9903c10d0a04bbeb93d0f776d1d252b152459499 Reviewed-on: Tested-by: Jenkins Reviewed-by: Miklos Vajna <>
2018-07-09loplugin:oncevarNoel Grandin1-6/+3
Change-Id: Id295dc0db174a448ef73db9de34f2de07d47f09a Reviewed-on: Tested-by: Jenkins Reviewed-by: Noel Grandin <>
2018-06-11rhbz#1589029 tdf#93789 impress not showing text highlight in presentation modeCaolán McNamara4-21/+119
the text hightlighting feature was implemented backed on to the vcl TextFillColor feature. TextFillColor fills the background of the bounds of the text with that color Likely either the same problem or similar as tdf#93789 Change-Id: Iace62cedc49e5f5844ac35d3caa23249b6cb4bc1 Reviewed-on: Reviewed-by: Caolán McNamara <> Tested-by: Caolán McNamara <>
2018-06-05tdf#42949 remove unused compheler includes ..Jochen Nitschke1-1/+0
and fix the fallout Change-Id: I15bc5d626f4d157cbc69a87392078b41e621d14e Reviewed-on: Tested-by: Jenkins <> Reviewed-by: Miklos Vajna <>
2018-05-31drop Graphic::GetBitmapNoel Grandin1-1/+1
so that we flush out various code using Bitmap, in favour of using BitmapEx. This is part of the process of making Bitmap largely an internal detail of vcl Change-Id: Iaf2ead5e3d9960838723fb55b812b97108093d74 Reviewed-on: Tested-by: Jenkins <> Reviewed-by: Noel Grandin <>
2018-05-29loplugin:unusedfields-in-constructor in variousNoel Grandin1-5/+4
Change-Id: Ie0fb647938e3cf730976fb2e435b92bfd67ef645 Reviewed-on: Tested-by: Jenkins <> Reviewed-by: Noel Grandin <>
2018-05-12make Bitmap/BitmapEx constructors explicitNoel Grandin1-3/+3
and add BitmapEx::operator=(Bitmap const &) Image::Image(Bitmap const &) to lessen the fallout Change-Id: Iff5fab88d167a7be739c370c9933d36c297bc61c Reviewed-on: Tested-by: Jenkins <> Reviewed-by: Noel Grandin <>
2018-04-09use more DBG_UNHANDLED_EXCEPTIONNoel Grandin1-2/+2
so we get nice logs of the exception dynamic type for UNO exceptions. Change-Id: Ic0b10dc14d354a2c9a0591b3a51d2f1640d54bdb Reviewed-on: Tested-by: Jenkins <> Reviewed-by: Noel Grandin <>
2018-04-07Revert "long->sal_Int32 in tools/gen.hxx"Stephan Bergmann1-4/+4
This reverts commit 8bc951daf79decbd8a599a409c6d33c5456710e0. As discussed at <> "long->sal_Int32 in tools/gen.hxx", that commit caused lots of problems with signed integer overflow, and the original plan was to redo it to consistently use sal_Int64 instead of sal_Int32. <> "sal_Int32->sal_Int64 in tools/gen.hxx" tried that. However, it failed miserably on Windows, causing odd failures like not writing out Pictures/*.svm streams out into .odp during CppunitTest_sd_export_ooxml2. So the next best approach is to just revert the original commit, at least for now. Includes revert of follow-up 8c50aff2175e85c54957d98ce32af40a3a87e168 "Fix Library_vclplug_qt5". Change-Id: Ia8bf34272d1ed38aac00e5d07a9d13fb03f439ae Reviewed-on: Tested-by: Jenkins <> Reviewed-by: Stephan Bergmann <>
2018-04-03long->sal_Int32 in tools/gen.hxxNoel Grandin1-4/+4
which triggered a lot of changes in sw/ Change-Id: Ia2aa22ea3f76463a85ea077a411246fcfed00bf6 Reviewed-on: Tested-by: Jenkins <> Reviewed-by: Noel Grandin <>
2018-03-07use more Color in cppcanvas..drawinglayerNoel Grandin2-5/+5
Change-Id: Ib0f15e8d4c01c1d2059cf32cc882042766d2a682 Reviewed-on: Tested-by: Jenkins <> Reviewed-by: Noel Grandin <>
2018-02-28convert COL_ constants to be of type ColorNoel Grandin1-3/+3
Change-Id: I0e25c8950ac26b851ff42f71e1471fcbe4770d48 Reviewed-on: Reviewed-by: Noel Grandin <> Tested-by: Noel Grandin <>
2018-02-27replace Color(COL_*) with COL_*Noel Grandin2-6/+6
using git grep -lwP "Color\s*\(\s*(COL_\w+)\s*\)" | xargs perl -pi -e "s/Color\s*\(\s*(COL_\w+)\s*\)//g" and then some manual fixup where the resulting expression no longer compiled Change-Id: I0e268d78611c3be40bba9f60ecfdc087a36c0df4 Reviewed-on: Tested-by: Jenkins <> Reviewed-by: Noel Grandin <>
2018-01-19loplugin:unusedmethods also check for functions returning boolNoel Grandin1-3/+1
we were previously excluding them Change-Id: I48a68799b0de60b4995fae541eb363e043d4dd11 Reviewed-on: Tested-by: Jenkins <> Reviewed-by: Noel Grandin <>
2018-01-19loplugin:useuniqueptr in EMFPPenNoel Grandin2-6/+4
Change-Id: Icc50b2ff1d6c7490b67d39aed7c3621b74c03bac
2018-01-19loplugin:useuniqueptr in EMFPRegionNoel Grandin2-6/+2
Change-Id: I19107173318180ec08864e523f4065181eac00cc
2018-01-19loplugin:useuniqueptr in EMFPBrushNoel Grandin2-36/+17
Change-Id: I17fc33cfd7c15734fd961fa76c676424900a4373
2018-01-12More loplugin:cstylecast: cppcanvasStephan Bergmann3-23/+23
auto-rewrite with <> "Enable loplugin:cstylecast for some more cases" plus solenv/clang-format/reformat-formatted-files Change-Id: Ib58b16600c97f8dab1b32d7e049c946ca82e660b
2017-11-23Make loplugin:unnecessaryparen look through implicitStephan Bergmann1-10/+10
...similar to how <> "Make not warning about !! in loplugin:simplifybool consistent" does for loplugin:simplifybool Change-Id: I23eef400af71c582d380c9bae6546ce06e8a1e18 Reviewed-on: Tested-by: Stephan Bergmann <> Reviewed-by: Stephan Bergmann <>
2017-11-06loplugin:constparams in cppcanvas,drawinglayerNoel Grandin8-12/+12
Change-Id: I15cac1411c985afe8d7cb9439214a8092702bdb8 Reviewed-on: Tested-by: Jenkins <> Reviewed-by: Noel Grandin <>
2017-11-03loplugin:constmethod in drawinglayerNoel Grandin1-3/+3
Change-Id: I6a33765f6589fc2941162eb2dcaa4e0a2d9e46e1 Reviewed-on: Tested-by: Jenkins <> Reviewed-by: Noel Grandin <>
2017-10-26vcl: make MapMode constructor explicitMichael Stahl1-1/+1
Insert constructor everywhere, except a couple places that apparently want to compare GetMapUnit(). Change-Id: I1910deb60562e5e949203435e827057f70a3f988
2017-10-23loplugin:includeform: cppucanvasStephan Bergmann19-42/+42
Change-Id: I83da2a535f89dfaa675629ca3dc2a6cefc334443
2017-10-23chmod 0755 -> 0644 for some source filesStephan Bergmann14-0/+0
Change-Id: Ibe5b5e03374419c2c23cd6559ab213d2dc2fcc66
2017-10-16tdf#31814 Resolve TODO from EMF+ DrawImage and DrawImagePointsBartosz Kosiorek1-33/+29
Change-Id: I1b128aa467286b5ae5e0d3cc298ee59f3e6ec12d Reviewed-on: Tested-by: Jenkins <> Reviewed-by: Patrick Jaap <> Reviewed-by: Bartosz Kosiorek <>
2017-10-09remove cppcanvas Color classNoel Grandin1-6/+4
which actually does nothing useful. Looks like it was originally intended to be used to implement color profiles, but since nothing has happened on that front since it was created, safe to say it never will. Probably not the right place in the graphics stack to do it anyhow. Change-Id: I36990db4036e3b4b2b75261fc430028562a6dbd9 Reviewed-on: Tested-by: Jenkins <> Reviewed-by: Noel Grandin <>
2017-09-26Rename the basegfx::tools namespace to basegfx::utilsTor Lillqvist16-79/+79
Reduce potential confusion with the global tools namespace. Will hopefully make it possible to remove the annoying initial :: when referring to the global tools namespace. Unless we have even more tools subnamespaces somewhere. Thorsten said it was OK. Change-Id: Id088dfe8f4244cb79df9aa988995b31a1758c996 Reviewed-on: Tested-by: Jenkins <> Reviewed-by: Tor Lillqvist <>
2017-09-11clang-tidy modernize-use-emplace in c*Noel Grandin2-83/+57
Change-Id: I419d1f67ba301050d05981db2a3d6178878684a9 Reviewed-on: Tested-by: Jenkins <> Reviewed-by: Noel Grandin <>
2017-08-24loplugin:useuniqueptr in cppcanvasNoel Grandin2-31/+26
Change-Id: I8e995128a7a1c5911efc82a762dbe7be015b2050 Reviewed-on: Tested-by: Jenkins <> Reviewed-by: Noel Grandin <>
2017-08-12tdf#96099 Remove ActionSharedPtr typedefIvan Safonov14-78/+78
Replace ActionSharedPtr by its definition. Change-Id: Iedd14c85169230d961f0707671885451875529d7 Reviewed-on: Reviewed-by: Julien Nabet <> Tested-by: Julien Nabet <>
2017-08-01loplugin:checkunusedparamsNoel Grandin1-5/+0
the "check for taking address of function" part was generating false+ Change-Id: Iad6203850901229b7b1b2f8938c68ec703cd343f Reviewed-on: Tested-by: Jenkins <> Reviewed-by: Noel Grandin <>
2017-07-28loplugin:constparams handle constructorsNoel Grandin1-12/+12
had to change the structure of the plugin considerably, was too messy to structure it to do the calculations on a per-function basis Change-Id: I4edee7735f726101105c607368124a08dba21086 Reviewed-on: Tested-by: Jenkins <> Reviewed-by: Noel Grandin <>
2017-07-28loplugin:checkunusedparams more part1Noel Grandin2-8/+3
seems I got one of the checks wrong, and was missing a bunch of stuff Change-Id: I2c662fc4e735f8d6cbe56c6f82906a60a580331b Reviewed-on: Tested-by: Jenkins <> Reviewed-by: Noel Grandin <>
2017-07-25use more comphelper::InitAnyPropertySequenceNoel Grandin2-24/+14
Found with: git grep -n -A10 'Sequence.*Any' -- *.cxx | grep -B5 -w PropertyValueProvider and: git grep -n 'Sequence.*Any.*( *&' Change-Id: Icb18c98bdd3f8352817e443ff78de5df042859ad Reviewed-on: Tested-by: Jenkins <> Reviewed-by: Noel Grandin <>
2017-07-22loplugin:unusedfields in cppcanvas..cuiNoel Grandin1-4/+1
Change-Id: I1cf1f7919e03d863c6f7756e1caf3b9313777479 Reviewed-on: Reviewed-by: Noel Grandin <> Tested-by: Noel Grandin <>
2017-07-20loplugin:constparams in cppcanvasNoel Grandin4-8/+8
Change-Id: Ic440819b5a1c71d2e4503b9a3b1e961c4ce123af Reviewed-on: Tested-by: Jenkins <> Reviewed-by: Noel Grandin <>
2017-07-06use more begin()/end() for SequenceNoel Grandin1-3/+1
Change-Id: I399be6b6ef7a6ce01e883569a177c0969bc29c69
2017-07-05loplugin:useuniqueptr in basic..cppcanvasNoel Grandin4-26/+16
Change-Id: Ib40241eb794607154ae52f8aa68fbf5ea5e944af Reviewed-on: Tested-by: Jenkins <> Reviewed-by: Noel Grandin <>
2017-06-23loplugin:oncevar in cppcanvas..drawinglayerNoel Grandin1-3/+1
Change-Id: I5456aad61fb0dfe6830eae62b91d1a6399d6343f Reviewed-on: Tested-by: Jenkins <> Reviewed-by: Noel Grandin <>
2017-06-18remove unused osl/mutex.hxx includesJochen Nitschke1-1/+0
Change-Id: I3b50e45fdb99e9cd8bfda07356ee3ddb4dd0f8bb Reviewed-on: Tested-by: Jenkins <> Reviewed-by: Jochen Nitschke <>
2017-05-31clang-tidy readability-redundant-control-flowNoel Grandin1-1/+0
Change-Id: I832f7ef0f1bd55e365db7e49823fe8bc30390c04 Reviewed-on: Tested-by: Jenkins <> Reviewed-by: Noel Grandin <>
2017-05-30teach redundantcast plugin about functional castsNoel Grandin1-5/+4
Change-Id: Iac8ccd17d9e46ebb2cb55db7adb06c469bbd4ea0 Reviewed-on: Reviewed-by: Noel Grandin <> Tested-by: Noel Grandin <>
2017-05-21cppcanvas: this assert can stayThorsten Behrens1-2/+1
Both sequences are supposed to be 4 - input there does not result from parsing the EMF+ file. Change-Id: Idcfa70d717b75ce226a8bae0ebe073002555ef17 Reviewed-on: Tested-by: Jenkins <> Reviewed-by: Thorsten Behrens <>
2017-05-14tdf#43157: convert cppcanvas module away from OSL_ASSERTChris Sherlock3-3/+3
2nd attempt, this time I've decided makes more sense to just produce a warning. Change-Id: Iaf373a2d38e8f47266b00ea869de38a584788d45 Reviewed-on: Tested-by: Jenkins <> Reviewed-by: Chris Sherlock <>
2017-05-12EMF+ tdf#31814 Add support of reading EmfPlusBoundaryPointDataBartosz Kosiorek3-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: Tested-by: Jenkins <> Reviewed-by: Bartosz Kosiorek <>
2017-05-09EMF+ We already implemented most used EMF+ records.Bartosz Kosiorek1-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: Tested-by: Jenkins <> Reviewed-by: Bartosz Kosiorek <>
2017-05-09tdf#31814 EMF+ Fix an issue when not all elements were displayedBartosz Kosiorek2-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: Reviewed-by: Bartosz Kosiorek <> Tested-by: Bartosz Kosiorek <>