summaryrefslogtreecommitdiff
path: root/cppcanvas
AgeCommit message (Collapse)AuthorFilesLines
2014-09-25coverity#1241427 Unchecked dynamic_castCaolán McNamara1-0/+1
Change-Id: Ie011d0bcc504c6c17a525abb368e69009a3d2d2e
2014-09-23fdo#82577: Handle WindowNoel Grandin2-2/+2
Put the VCL Window class in the vcl namespace. Avoids clash with the X11 Window typedef. Change-Id: Ib1beb7ab4ad75562a42aeb252732a073d25eff1a
2014-09-18fdo#82577: Handle FontNoel Grandin3-8/+8
Put the VCL Font class in the vcl namespace. Avoids clash with the X11 Font typedef. Change-Id: I1a84f7cad8b31697b9860a3418f7dff794ff6537
2014-08-21vcl: convert push flags to type-safe enum-like classNoel Grandin3-3/+3
Change-Id: Ib49a0dd5ecee0225f51bea2ff1c0ab5326595a47
2014-08-20vcl: use enum for complex text layout constantsNoel Grandin1-15/+7
Since these constants are bitfield flags, we define some methods to make working with them reasonably type safe. Move the definitions to outdevstate.hxx, since we need the values there, and that appears to be the "root most" header file. Also dump TEXT_LAYOUT_BIDI_LTR constant, since it means the same thing as TEXT_LAYOUT_DEFAULT (ie. 0), and leaving it in causes people to write weird code thinking that it's a real flag. Change-Id: Iddab86cd6c78181ceb8caa48e77e1f5a8e526343 Reviewed-on: https://gerrit.libreoffice.org/10676 Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2014-08-08Missing test dependenciesStephan Bergmann1-0/+4
Change-Id: I7ea8f2b02b66a15e86e20948536c34017d6a3999
2014-08-07coverity#1228877 Structurally dead codeCaolán McNamara1-11/+11
Change-Id: If3a3de8bae4c2d1b8e39d3255cd8ee7e985b4404
2014-07-29fdo#81799 we need cppuhelper here tooDavid Tardon1-0/+1
Change-Id: I719557813180c3f862ef4b00e24142fdbbb0daf0
2014-07-28Use better coordinates for unit test of fdo#77229Arnaud Versini1-1/+1
Change-Id: I5fae217687e12313451da5bf4b4e6f914551b628 Reviewed-on: https://gerrit.libreoffice.org/10447 Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk> Tested-by: Miklos Vajna <vmiklos@collabora.co.uk>
2014-07-20vcl consitent use of long for corrdinateNorbert Thiebaud4-12/+12
most of length in vcl are calculated in 'long' but array of X position tend to be in sal_Int32. As a prep work to be able to support 'double' as the base type of Device Coordinate, harmonize the use of 'long' for non-float coordinate. Change-Id: I7cb33301ff6a5e2c62247b36a4e07e168a58a323
2014-07-20emf+: recognise some more object typesTomaž Vajngerl1-0/+18
Change-Id: I33fec62e4bc38eeaf014eeb1210db2904af033f6
2014-07-20emf+: emulate hatch with color blendTomaž Vajngerl1-2/+116
Change-Id: I2ac8f790c79c269d4c1fa650e703c3645c567ca4
2014-07-20bnc#881024 Handle 0 font height just like outdev & drawinglayerTomaž Vajngerl1-1/+12
Change-Id: I80055e4101873e0ddd408ac1f0ee9c75cc3bf6b3
2014-07-20emf+: Log type name instead of the type numberTomaž Vajngerl1-1/+47
Change-Id: I5c4f7c5713a559815bd46328907673d114fee011
2014-07-01New loplugin:stringconcatStephan Bergmann1-4/+3
Change-Id: Id7c517fb37bc28797c45fc0dde83e866f2aa4aac
2014-06-17improve the inlinesimplememberfunctions clang pluginNoel Grandin4-15/+3
Change-Id: I6d5a952901648e01904ef5c37f953c517304d31e
2014-06-12Fix vclplug_* dependencies for gb_CppunitTest_use_vclStephan Bergmann1-3/+1
...and replace gb_CppunitTest_unset_headless with gb_CppunitTest_use_vcl_non_headless. Change-Id: Ibd859f98ed8df09988ca26ffcee26e06e25d8313
2014-06-12Lets assume gb_CppunitTest_unset_headless was cargo cult hereStephan Bergmann1-2/+0
Change-Id: I9d4255f0ddcaf375706556d0c0baf2a5a0762bbf
2014-06-04update_pch: add a bunch of pch filesThomas Arnhold3-0/+111
connectivity: 3m52s -> 2m47s cppcanvas: 28s -> 13s cppuhelper: 20s -> 14s dbaccess: 2m38s -> 2m01s hwpfilter: 16s -> 13s sot: 21s -> 16s Change-Id: I49286bfe6be73dd1b861be632b95e17a99e82f8a
2014-06-04DeInitVCL at end of testsStephan Bergmann2-0/+2
This required some changes to the framework: * Init-/DeInitVCL is no longer done per individual test in BootstrapFixture, but once per CppunitTest invocation in a new vclbootstrapprotector (similarly to the exisiting unobootstrapprotector). CppunitTests that need VCL now need to declare gb_CppunitTest_use_vcl. * For things to work properly, the UNO component context needs to be disposed from within DeInitVCL (cf. Desktop's Application::DeInit called from DeInitVCL). The easiest solution was to introduce an Application::setDeInitHook (where the hook is called from DeInitVCL) specifically for vclbootstrapprotector to call. * PythonTests don't (yet) call DeInitVCL; they still hook into BootstrapFixture's original test_init functionality (to call InitVCL), and do not make use of the vclbootstrapprotector. Change-Id: I4f3a3c75db30b58c1cd49d81c51db14902ed68b2
2014-05-30Fix memory leak in test codeStephan Bergmann1-2/+4
Change-Id: I318878b3fa883eebbc081a6faf6a594dad2c6e26
2014-05-22cppcheck: Function parameter 'rStartPos' should be passed by referenceThomas Arnhold2-6/+6
Change-Id: Ie453e1a95b54de07463b6f5052a23d6632400b8d
2014-05-22cppcheck: Function parameter 'rLineInfo' should be passed by referenceThomas Arnhold1-2/+2
Change-Id: I5bd3f808a0ca61731c5ceb57100d1b30a3163e98
2014-05-15Resolves fdo#70681: fixincludeguards.pl: all that's leftThomas Arnhold23-68/+68
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 Nabet1-2/+2
Change-Id: I590a682af7fc1879ae63cda3aea73063ba4c84f4
2014-05-09'!= false' is redundant, and confusing - kill it.Jan Holesovsky1-1/+1
Change-Id: I1a52b9ce7b590e8cf274a2daaa1c55cfc1efe48b
2014-04-28coverity#738576 Uninitialized pointer fieldCaolán McNamara1-6/+20
Change-Id: Id945cf9549b7f633f7deb1f237a2bf6536f221c1
2014-04-28coverity#738575 Uninitialized pointer fieldCaolán McNamara1-6/+17
Change-Id: Ib65f0e95ef2a837b6c5791d9e2dce5fe8025da33
2014-04-28coverity#738574 Uninitialized scalar fieldCaolán McNamara1-6/+20
Change-Id: Ia7cca6e2a2bc3240a32509697a999461491a2747
2014-04-28coverity#738573 Uninitialized scalar fieldCaolán McNamara1-2/+8
Change-Id: I8b112fa0095cc6ed15c9198a312a0b6fce3a8d28
2014-04-17remove executable bitAndras Timar1-0/+0
Change-Id: I91f6e861f6d0dbea1b57ac73857e0c706b27f681
2014-04-10CppunitTest_cppcanvas_emfplus: disable this in case there is no display to useMiklos Vajna1-0/+2
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 Vajna4-0/+161
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 Vajna1-1/+23
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 Bergmann1-4/+0
Change-Id: I54f8a16cd64375d1398b6582175b0941ba68696e
2014-04-07cppcanvas: sal_Bool->boolNoel Grandin3-5/+5
Change-Id: I3dc076f3593d4d083d5e25e41e007eadc97d2978
2014-04-07Replace SV_DECL/IMPL_REF macros with SvRef templateStephan Bergmann1-1/+1
Change-Id: I0ef2e67f6d61e0ce118c0f5e926b8194ef9d8058
2014-04-01Explicitly mark overriding destructors as "virtual"Stephan Bergmann1-5/+5
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 Bergmann19-129/+129
...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 Grandin2-2/+2
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 Bergmann2-7/+7
Change-Id: I56e32131b7991ee9948ce46765632eb823d463b3
2014-02-25Remove visual noise from cppcanvasAlexander Wilms2-9/+9
Change-Id: I961e6a9e227962096b57569a8b8160841cacd950
2014-02-25Update little-to-big endian GetSwapFloat functionsDouglas Mencken1-2/+5
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 Thiebaud1-4/+4
Change-Id: I2e472aa0279d0763762d3c660207cd74da512626
2014-02-23Remove unneccessary commentsAlexander Wilms3-20/+20
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 Bergmann3-6/+6
Change-Id: I2a3e3d3e3266ea0f0fafdd91362076a4aa160f0e
2014-02-17fix debug=2 buildCaolán McNamara1-31/+1
Change-Id: Id7c6db5c6cbefef427d9c2cc4bd210811b5a7991
2014-02-12remove SvStream::operator>> methodsNoel Grandin1-2/+2
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 Grandin2-94/+94
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 Grandin3-9/+9
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>