summaryrefslogtreecommitdiff
path: root/cppcanvas
AgeCommit message (Collapse)Author
2014-05-22cppcheck: Function parameter 'rStartPos' should be passed by referenceThomas Arnhold
Change-Id: Ie453e1a95b54de07463b6f5052a23d6632400b8d
2014-05-22cppcheck: Function parameter 'rLineInfo' should be passed by referenceThomas Arnhold
Change-Id: I5bd3f808a0ca61731c5ceb57100d1b30a3163e98
2014-05-15Resolves fdo#70681: fixincludeguards.pl: all that's leftThomas Arnhold
Change-Id: I3e51a62710bb46c8255fd228d41d9300c90a1fb5 Reviewed-on: https://gerrit.libreoffice.org/9360 Reviewed-by: Thomas Arnhold <thomas@arnhold.org> Tested-by: Thomas Arnhold <thomas@arnhold.org>
2014-05-10Blind fix for red Tinderboxes, followingJulien Nabet
Change-Id: I590a682af7fc1879ae63cda3aea73063ba4c84f4
2014-05-09'!= false' is redundant, and confusing - kill it.Jan Holesovsky
Change-Id: I1a52b9ce7b590e8cf274a2daaa1c55cfc1efe48b
2014-04-28coverity#738576 Uninitialized pointer fieldCaolán McNamara
Change-Id: Id945cf9549b7f633f7deb1f237a2bf6536f221c1
2014-04-28coverity#738575 Uninitialized pointer fieldCaolán McNamara
Change-Id: Ib65f0e95ef2a837b6c5791d9e2dce5fe8025da33
2014-04-28coverity#738574 Uninitialized scalar fieldCaolán McNamara
Change-Id: Ia7cca6e2a2bc3240a32509697a999461491a2747
2014-04-28coverity#738573 Uninitialized scalar fieldCaolán McNamara
Change-Id: I8b112fa0095cc6ed15c9198a312a0b6fce3a8d28
2014-04-17remove executable bitAndras Timar
Change-Id: I91f6e861f6d0dbea1b57ac73857e0c706b27f681
2014-04-10CppunitTest_cppcanvas_emfplus: disable this in case there is no display to useMiklos Vajna
This test is already disabled on non-Linux, but it fails in case there is no display to use. For now just disable it in that case. Change-Id: I29c52e803a1fca5f2bdeeb655c573ad8fef622e8
2014-04-10fdo#77229 testcaseMiklos Vajna
We didn't have EMF+ rendering testcases so far, let's see if it works out to render into a bitmap and then just assert pixel position colors there. It's better than nothing for missing shapes at least. Change-Id: I2d1c63fef1127f69af7156ed6c99553845f77c9f
2014-04-09fdo#77229 EMF+ rendering: improve EmfPlusSetClipPath's CombineModeExclude caseMiklos Vajna
This is still not perfect, but at least we now don't do the opposite of what was asked. Change-Id: I5e144c5ec2987902e65b2eb472259d9c39bbbd11
2014-04-09Clean up function declarationsStephan Bergmann
Change-Id: I54f8a16cd64375d1398b6582175b0941ba68696e
2014-04-07cppcanvas: sal_Bool->boolNoel Grandin
Change-Id: I3dc076f3593d4d083d5e25e41e007eadc97d2978
2014-04-07Replace SV_DECL/IMPL_REF macros with SvRef templateStephan Bergmann
Change-Id: I0ef2e67f6d61e0ce118c0f5e926b8194ef9d8058
2014-04-01Explicitly mark overriding destructors as "virtual"Stephan Bergmann
It appears that the C++ standard allows overriding destructors to be marked "override," but at least some MSVC versions complain about it, so at least make sure such destructors are explicitly marked "virtual." Change-Id: I0e1cafa7584fd16ebdce61f569eae2373a71b0a1
2014-03-26First batch of adding SAL_OVERRRIDE to overriding function declarationsStephan Bergmann
...mostly done with a rewriting Clang plugin, with just some manual tweaking necessary to fix poor macro usage. Change-Id: I71fa20213e86be10de332ece0aa273239df7b61a
2014-03-18Find places where OUString and OString are passed by value.Noel Grandin
It's not very efficient, because we generally end up copying it twice - once into the parameter and again into the destination OUString. So I create a clang plugin that finds such places and generates a warning so that we can convert them to pass-by-reference. Change-Id: I5341a6ea9e3190f4b4c05c42c85595e3dcd83361
2014-02-26cppuhelper: retrofit std::exception into overriding exception specsStephan Bergmann
Change-Id: I56e32131b7991ee9948ce46765632eb823d463b3
2014-02-25Remove visual noise from cppcanvasAlexander Wilms
Change-Id: I961e6a9e227962096b57569a8b8160841cacd950
2014-02-25Update little-to-big endian GetSwapFloat functionsDouglas Mencken
rSt >> pPtr[3] >> pPtr[2] >> pPtr[1] >> pPtr[0] results in error: no match for 'operator>>' in 'rSt >> *(pPtr + 3u)' Change-Id: I1820bac8c72f7633ab01a17bf5d073b988e1b58c Reviewed-on: https://gerrit.libreoffice.org/8207 Tested-by: Michael Stahl <mstahl@redhat.com> Reviewed-by: Michael Stahl <mstahl@redhat.com>
2014-02-24harmonize Tell() Seek() type.Norbert Thiebaud
Change-Id: I2e472aa0279d0763762d3c660207cd74da512626
2014-02-23Remove unneccessary commentsAlexander Wilms
Change-Id: I939160ae72fecbe3d4a60ce755730bd4c38497fb Reviewed-on: https://gerrit.libreoffice.org/8182 Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2014-02-21vcl: sal_Bool -> boolStephan Bergmann
Change-Id: I2a3e3d3e3266ea0f0fafdd91362076a4aa160f0e
2014-02-17fix debug=2 buildCaolán McNamara
Change-Id: Id7c6db5c6cbefef427d9c2cc4bd210811b5a7991
2014-02-12remove SvStream::operator>> methodsNoel Grandin
in favour of ReadXXX methods. Change-Id: Ic2c0a7b6b92ff4c236ae99b39d77f3d935b301e3 Reviewed-on: https://gerrit.libreoffice.org/7915 Tested-by: Michael Stahl <mstahl@redhat.com> Reviewed-by: Michael Stahl <mstahl@redhat.com>
2014-02-12convert SvStream::operator>> methods to ReadXXX methodsNoel Grandin
First, I updated the clang rewriter to do the conversion. Then I lightly hand-tweaked the output for the few places where the rewriter messed up, mostly when dealing with calls on "this". Change-Id: I40a6a977959cd97415c678eafc8507de8aa3b1a9 Reviewed-on: https://gerrit.libreoffice.org/7879 Reviewed-by: Michael Stahl <mstahl@redhat.com> Tested-by: Michael Stahl <mstahl@redhat.com>
2014-02-04convert specialised SvStream::operator>> methods to ReadXXX methodsNoel Grandin
as preparation for converting the SvStream::operator>> methods on primitive types Change-Id: I62f134bced15c687d6e0d46924f56e8d1c3d95b9 Reviewed-on: https://gerrit.libreoffice.org/7798 Tested-by: LibreOffice gerrit bot <gerrit@libreoffice.org> Reviewed-by: Michael Stahl <mstahl@redhat.com>
2014-01-30EMF+: Only draw endcap outline if not filled, fix endcap scaling.Andrzej Hunt
Change-Id: I4520eea08e43ccd657c1db03b258ef84612da971 Reviewed-on: https://gerrit.libreoffice.org/7726 Reviewed-by: Jan Holesovsky <kendy@collabora.com> Tested-by: Jan Holesovsky <kendy@collabora.com>
2014-01-28Apparently broken bitmask operationStephan Bergmann
Change-Id: Id4dfa3d4676b50a86fd6044d501de15b122bb7f1
2014-01-23coverity#1158489 Uninitialized scalar fieldCaolán McNamara
Change-Id: Ibe8600f8ad38859cd79d52528f970d99f0b8f536
2014-01-14longparas: remove now broken STRING_LEN usesCaolán McNamara
Change-Id: Ie2de4680e07e26416fc52d4c22b5bececb190cc1
2014-01-13longparas: convert ValidateKashidas to sal_Int32Caolán McNamara
we can use -1 as the failure return code as return value is always checked and isolated from the rest of the code. Change-Id: I87a743de82d681a7e4eabc61a576c4e50f1279a8
2014-01-01EMF+: actually use lineJoin attribute for polygons.Andrzej Hunt
Change-Id: I5b369703333332598353d9817f0253bbe5fc3601
2014-01-01EMF+: mapping can rotate the width vector, so use resulting length.Andrzej Hunt
Otherwise (i.e. with non-vertical lines) the width will be incorrect, and can result in lines disappearing as the resulting X component can be tiny/approaching zero. Change-Id: Icf3b7c10c627594600b517b8ff445f8df87c56f8
2013-12-27EMF+: Add EMFPlusPointR detection to EMFPPath.Andrzej Hunt
Change-Id: Iff1ae9806ee05654762b313ed00ada206e51337f
2013-12-26EMF+: Integer coordinate values are signed.Andrzej Hunt
Change-Id: I5babcec50d48dc2a6288a01685db61fbb7058680
2013-12-18vcl get rid of xub_StrLen and STRING_LEN in outdev3Norbert Thiebaud
a new log section (sal.rtl.xub) is used to display alert in case of suspicious len == 0xFFFF (aka STRING_LEN) Change-Id: I3ed2aa7896e12592be9e003580dd6c8eda4add5e Reviewed-on: https://gerrit.libreoffice.org/7117 Tested-by: LibreOffice gerrit bot <gerrit@libreoffice.org> Reviewed-by: Norbert Thiebaud <nthiebaud@gmail.com>
2013-12-17Adapt all (non-extension, SharedLibrary) .components to environment="..."Stephan Bergmann
Change-Id: I56f38bd786f3a026cb2908f28540dc9c4003af83
2013-12-06EMF+: Fill line cap object if EmfPlusCustomLineCapDataFillPath is set.Matúš Kukan
Change-Id: I7b53a8f18e1fb24b1ae0322bdf0980e431a0725f
2013-12-05EMF+: Log the stride too.Jan Holesovsky
Change-Id: I903d8e5629c578f292307bf3abdb2f58ef9dd92c
2013-12-04EMF+: Do not ignore source rectangle of the image to be rendered.Matúš Kukan
Change-Id: I4f861c7653c89d8f1ca73696e885f33f7bfc3b06
2013-11-29Further XCanvas != XBitmapCanvas cleanup.Thorsten Behrens
It seems I missed a few places in 21ec9beae29b19b8ec6f0a16fd0e708e4f210208, getting that straight now hopefully. Change-Id: I34fc8df2d1ee65bef0639b3de9487ff311bed89a
2013-11-29Revert "Resolves: fdo#71527 make presenter console not crash/useless"Thorsten Behrens
This reverts commit 639aa8e72639fd01e9004977f1cfaafc13b1e45f. Change-Id: I0ac1a7ada530791e2d34d2f0802213c869ea48a1
2013-11-28Resolves: fdo#71527 make presenter console not crash/uselessCaolán McNamara
Basically reverts the pieces of 21ec9beae29b19b8ec6f0a16fd0e708e4f210208 to make XSpriteCanvas a XBitmapCanvas again Otherwise PresenterHelper::loadBitmap is not an XBitmapCanvas and so VclFactory::createBitmap cannot succeed Change-Id: I197adf98e915102f383ee050a8ea16d1e41cecf2
2013-11-27EMF+: Line thickness has to be considered when drawing the caps.Jan Holesovsky
Change-Id: I6043ee3c214f453afaef06125993c73be624c07e
2013-11-26WaE: potentially uninitialized local variable 'pPattern' usedTor Lillqvist
Change-Id: Ic000edc60a9c5ae7d64fab9bf8b6c2a88bcec223
2013-11-26-Werror,-Wunused-const-variableStephan Bergmann
Change-Id: I25fa3ea9e0101de7b17c17ef44da6d7b26a46017
2013-11-25EMF+: Set the stroke attributes on the custom line caps.Jan Holesovsky
This finally makes the rendering of the custom line caps nice & complete. Change-Id: If35ef1c44f34f5d5e6c50789c907105d03e96fca