summaryrefslogtreecommitdiff
path: root/cppcanvas
AgeCommit message (Collapse)AuthorFilesLines
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>
2014-01-30EMF+: Only draw endcap outline if not filled, fix endcap scaling.Andrzej Hunt1-17/+32
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 Bergmann1-1/+1
Change-Id: Id4dfa3d4676b50a86fd6044d501de15b122bb7f1
2014-01-23coverity#1158489 Uninitialized scalar fieldCaolán McNamara1-1/+8
Change-Id: Ibe8600f8ad38859cd79d52528f970d99f0b8f536
2014-01-14longparas: remove now broken STRING_LEN usesCaolán McNamara1-7/+9
Change-Id: Ie2de4680e07e26416fc52d4c22b5bececb190cc1
2014-01-13longparas: convert ValidateKashidas to sal_Int32Caolán McNamara1-0/+1
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
generated by cgit v1.2.3 (git 2.39.1) at 2024-05-23 21:08:02 +0000