summaryrefslogtreecommitdiffstats
path: root/chart2/source/inc
Commit message (Collapse)AuthorAgeFilesLines
* use more concrete types in chart2, DiagramNoel Grandin2022-01-212-1/+5
| | | | | | | Change-Id: Ib78f327c8bf1b71c3bb038c029b10e6fe54edaa1 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/128708 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* use more concrete types in chart2, ChartModelNoel Grandin2022-01-203-3/+12
| | | | | | | Change-Id: Ie38d941f855978b04995162040d8871a2577255c Reviewed-on: https://gerrit.libreoffice.org/c/core/+/128641 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* use more concrete types in chart2, InternalDataProviderNoel Grandin2022-01-192-7/+9
| | | | | | | Change-Id: Iee2cc56b00f0464e652a14eace2640e5d798eb5f Reviewed-on: https://gerrit.libreoffice.org/c/core/+/128629 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* use more concrete types in chart2, UncachedDataSequenceNoel Grandin2022-01-191-2/+4
| | | | | | | Change-Id: Ic5ca09567b74840c28d1c892a315476a6b7e9e0f Reviewed-on: https://gerrit.libreoffice.org/c/core/+/128630 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* use more concrete types in chart2, BaseCoordinateSystemNoel Grandin2022-01-195-9/+142
| | | | | | | Change-Id: I1c246a3ea37595647ac254f492170a9e18540794 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/128623 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* use more concrete types in chart2, ChartTypeTemplateNoel Grandin2022-01-193-14/+15
| | | | | | | | | | and return data using a little struct rather than std::pair, to make the call sites read a little better. Change-Id: Ieb6fc24e053f50823789167ed0112aa04a083725 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/128613 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* use more concrete types in chart2, DiagramNoel Grandin2022-01-191-4/+3
| | | | | | | Change-Id: Id21836b1258b8757e634fc9ab670ef1180748469 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/128612 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* use more concrete types in chart2, DiagramNoel Grandin2022-01-192-2/+7
| | | | | | | Change-Id: I870f6d9fa4c0b51cf7c887199079fdbabbca1fec Reviewed-on: https://gerrit.libreoffice.org/c/core/+/128597 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* use more concrete types in chart2, NameContainerNoel Grandin2022-01-191-9/+3
| | | | | | | | | and simplify the NameContainer class, since it has only one use site Change-Id: I5b68b8985c22bb5eb37fa559aba1d5314c3d3aa0 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/128575 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* use more concrete types in chart2, ChartTypeTemplateNoel Grandin2022-01-191-1/+3
| | | | | | | Change-Id: I1bfbc81ca0d44efc669e5bc2b525cfa8b51be1ea Reviewed-on: https://gerrit.libreoffice.org/c/core/+/128561 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* use more concrete types in chart2, ChartTypeTemplateNoel Grandin2022-01-181-0/+251
| | | | | | | Change-Id: Ife0be0fde2f8d273411cbdbbafa2173ac80f80f7 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/128560 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* loplugin:finalclasses in chart2Noel Grandin2022-01-1718-37/+37
| | | | | | | Change-Id: I6cc88d50df7a6c77285f8534de43fd0c8d251bf7 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/128505 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* use more concrete types in chart2, ChartViewNoel Grandin2022-01-161-1/+1
| | | | | | | Change-Id: I51d503cd71ef2adc0e590ef836dabf9906495bbc Reviewed-on: https://gerrit.libreoffice.org/c/core/+/128475 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* use more concrete types in chart2, DiagramNoel Grandin2022-01-161-0/+183
| | | | | | | Change-Id: Iae84e16b3406f4c08139764a1838e38b13cebee7 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/128471 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* improvements to ModifyEventForwarderNoel Grandin2022-01-141-35/+8
| | | | | | | | | | | | | (*) Firstly, the XWeak machinery in here was never useful. This class has always used hard references to its listeners. So just remove all that. (*) Convert from osl::Mutex to std::mutex and use the latest utility classes in comphelper with that. Change-Id: Id129527cf53da6d7a9f6e1c76b556c57fe1a0270 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/128393 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* use more concrete types in chart2Noel Grandin2022-01-144-9/+14
| | | | | | | | | | | in this case, ModifyEventForwarder. Instead of doing UNO_QUERY and implicitly relying on it's implementation, make it obvious what we are doing Change-Id: Ie02a11f115da9c8af7d4ca85bda9fb9a69c5d52f Reviewed-on: https://gerrit.libreoffice.org/c/core/+/128392 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* Recheck modules [a-c]* with IWYUGabor Kelemen2022-01-133-2/+2
| | | | | | | | | See tdf#42949 for motivation Change-Id: I74a3c3a67c3639376e2251c3eb917fa4139dfbd0 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/127808 Tested-by: Jenkins Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
* remove unused codeNoel Grandin2022-01-121-16/+0
| | | | | | | Change-Id: Ieb5100dab1ed8af23e765926a8a1f33d75132406 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/128311 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* use vectors to build up point data, instead of SequenceNoel Grandin2022-01-112-2/+20
| | | | | | | | | which shaves 1% off the load time of a large chart Change-Id: Ieb8f029f760f41c3bef63bbc4cd221c1473f0f49 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/128283 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* bypass SvxShape when creating rectangles in chart2Noel Grandin2022-01-061-0/+7
| | | | | | | Change-Id: I461449959fffa8948bf4321a6c3114505d87a149 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/127997 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* use more SvxShape in chart2Noel Grandin2022-01-031-3/+3
| | | | | | | Change-Id: I4a445ca3501d32cd8ff04f12239cce52457b9228 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/127889 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* add new SvxShape hierarchy classNoel Grandin2022-01-031-1/+1
| | | | | | | | | | to serve as a shared/common base class, so I can plumb SvxShape types through chart2. Change-Id: I8758fff1599c92a214b334ee44cf7b4e582c1915 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/127873 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* use more SvxShape in chart2Noel Grandin2022-01-021-0/+1
| | | | | | | Change-Id: I97f57652d2ce17efc9b69e07bbca78c318451f1d Reviewed-on: https://gerrit.libreoffice.org/c/core/+/127858 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* use concrete types in chart2, SvxDrawPageNoel Grandin2022-01-021-6/+6
| | | | | | | Change-Id: I0bdaa74595c0201a04490e293619c2c399ab03bd Reviewed-on: https://gerrit.libreoffice.org/c/core/+/127838 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* use concrete types in chart2, createTextNoel Grandin2022-01-021-0/+6
| | | | | | | Change-Id: Ic3b5920192c62cbdd025b480b01d04fb57d0be00 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/127837 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* use concrete types in chart2, SvxShapePolyPolygonNoel Grandin2021-12-301-4/+4
| | | | | | | Change-Id: Ica53484d44e6992278b8d188ffb536e238aeb195 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/127725 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* use comphelper::WeakComponentImplHelper in PopupRequestNoel Grandin2021-12-291-9/+3
| | | | | | | Change-Id: I199e40de2ac3215d03a90d75988a7775f87890b3 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/127693 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* use comphelper::WeakComponentImplHelper in RangeHighlighterNoel Grandin2021-12-291-6/+6
| | | | | | | Change-Id: I38d14bb253981db18885355c64accaa91c3accd7 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/127694 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* use more cppu::BaseMutexNoel Grandin2021-12-197-50/+12
| | | | | | | Change-Id: Ib9da716de0addc70ca58d9085ff351c2ff8c706f Reviewed-on: https://gerrit.libreoffice.org/c/core/+/127083 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* speed up property access in chartsNoel Grandin2021-12-051-1/+4
| | | | | | | | | | uno::Any copy and assign is fairly hot, so try and do those operations in one step Change-Id: I110566ba98788d5a658d229433de291c8cdf146c Reviewed-on: https://gerrit.libreoffice.org/c/core/+/126377 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* O[U]String::replaceAt overloads that take string_viewNoel Grandin2021-11-231-1/+1
| | | | | | | | | | which results in lots of nice string_view improvements picked up by the plugins Change-Id: Ib0ec3887816b3d4436d003b739d9814f83e244b2 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/125657 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* loplugin:stringliteraldefine in chart2Noel Grandin2021-11-183-39/+53
| | | | | | | | | | and drop namespace declarations that were surrounding the #define - namespace decls have no affect on #defines Change-Id: Ic66ba7837ba3a5c48a0cf414af30985c480d73af Reviewed-on: https://gerrit.libreoffice.org/c/core/+/125278 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* chart2: disambiguate ShiftedCategoryPositionMiklos Vajna2021-11-161-1/+1
| | | | | | | | | | | There are two of these: one sal_Bool in a .idl file and a bool in a .hxx file. They need to be printed differently with SAL_DEBUG(), so rename the C++ one to avoid confusion. Change-Id: I0f79908a7b74742410813166370cb8ba3017e727 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/125256 Tested-by: Jenkins Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
* tdf#136111 fix scaling problem on chart driven by a macroAndras Timar2021-11-101-0/+1
| | | | | | | | | | | Change-Id: I9a55bcfceb9259f0d5dc944c00a34b3e4a891e0e Reviewed-on: https://gerrit.libreoffice.org/c/core/+/124940 Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com> Reviewed-by: Dennis Francis <dennis.francis@collabora.com> (cherry picked from commit 0644f44daef7caa8a246221d762fbc0f6af3672c) Reviewed-on: https://gerrit.libreoffice.org/c/core/+/124864 Tested-by: Jenkins Reviewed-by: Andras Timar <andras.timar@collabora.com>
* Prepare for removal of non-const operator[] from Sequence in chart2Mike Kaganski2021-10-291-1/+2
| | | | | | | Change-Id: I71e0b458c4dd2a85e36a485e3efa72b1077b0e54 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/124346 Tested-by: Jenkins Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
* In O[U]StringBuffer, make string_view params replacements for OUString onesStephan Bergmann2021-10-111-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ...for LIBO_INTERNAL_ONLY, instead of having them as additional overloads. That way, loplugin:bufferadd and loplugin:stringviewparam found many further opportunities for simplification (all addressed here). Some notes: * There is no longer an implicit conversion from O[U]String to O[U]StringBuffer (as that goes via user-defined conversions through string_view now), which was most noticeable in copy initializations like OStringBuffer buf = someStr; that had to be changed to direct initialization, OStringBuffer buf(someStr); But then again, it wasn't too many places that were affected and I think we can live with that. * I made the O[U]StringBuffer ctors taking string_view non-explicit, mainly to get them in line with their counterparts taking O[U]String. * I added an OUStringBuffer::lastIndexOf string_view overload that was missing (relative to OUStringBuffer::indexOf). * loplugin:stringconstant needed some addition to keep the compilerplugins/clang/test/stringconstant.cxx checks related to OStringBuffer::append and OStringBuffer::insert working. * loplugin:stringviewparam no longer needs the special O[U]StringBuffer-related code that had been introduced in 1250aecd71fabde4dba990bfceb61bbe8e06b8ea "loplugin:stringviewparam extend to new.." Change-Id: Ib1bb8c4632d99b744e742605a9fef6eae959fd72 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/122904 Tested-by: Jenkins Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
* loplugin:moveparam in chart2Noel Grandin2021-10-111-4/+4
| | | | | | | Change-Id: I6a47abf80f0f45bbb55bc9e0fe817f3c3650aff3 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/123351 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* [API CHANGE] oox: fix import of chart date categoriesDennis Francis2021-09-061-2/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Before this fix, date categories imported in oox's DataSourceContext were stored as formatted strings according to number format code in <c:formatCode> under the <c:cat> tree. As a result chart2 could not recognize them as dates. This causes problems like: * The axis that is linked to date categories cannot use the scaling/range-selection(min/max)/increments specs mentioned as axis properties. This results in distorted/unreadable chart renders w.r.t the date axis. * No re-formatting is attempted as per the number format provided for axis. This patch introduces a role qualifer argument to the XDataProvider interface method createDataSequenceByValueArray to support categories of date type via this method. When exporting to oox, write date categories and format code under <c:cat> <c:numRef> <c:numCache> This patch also fixes some discrepancies in date axis interval computation (auto mode) found by already existing unit tests. Change-Id: Ibc53b0a56fdddba80ba452d5567ce98d80460ea7 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/121525 Tested-by: Jenkins Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
* tdf#114441 :Convert use of sal_uLong to better integer typesYildiray2021-08-181-1/+1
| | | | | | | | Change-Id: Ifc7c1ce21f9dbaf77a0ae2ffa598e219809a250d Reviewed-on: https://gerrit.libreoffice.org/c/core/+/120541 Tested-by: Jenkins Reviewed-by: Hossein <hossein@libreoffice.org> Reviewed-by: Michael Stahl <michael.stahl@allotropia.de>
* tdf#143148 Replaced #include guards with #pragma once.Ankur Khandelwal2021-08-101-4/+1
| | | | | | | | | Replace include guards with #pragma once in header file. Change-Id: I078f97ef5f7ab44ee915c78ab5f4569bde9f6640 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/120208 Tested-by: Michael Stahl <michael.stahl@allotropia.de> Reviewed-by: Michael Stahl <michael.stahl@allotropia.de>
* create comphelper::OMultiTypeInterfaceContainerHelper2 and use itNoel Grandin2021-08-071-2/+2
| | | | | | | | | | based on OInterfaceContainerHelper2 which is considerably faster than the original OInterfaceContainerHelper Change-Id: I9c8b6d0e5382018824bf7188a26343703abf2d51 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/120161 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* Pass context and resource string down to boost::locale separatelyNoel Grandin2021-08-051-2/+2
| | | | | | | | | | because this is often on a hot path, and we can avoid the splitting and joining of strings like this. Change-Id: Ia36047209368ca53431178c2e8723a18cfe8260a Reviewed-on: https://gerrit.libreoffice.org/c/core/+/119220 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* OPropertySet doesn't need a pimplNoel Grandin2021-07-221-6/+33
| | | | | | | | | also inline some of the standalone lambda/functor classes Change-Id: I0a19a1d3cb7f5b3d02348f1ab57bae08b3ab089a Reviewed-on: https://gerrit.libreoffice.org/c/core/+/119379 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* Purge out rtl::math::setNanBaiXiaochun2021-06-301-5/+5
| | | | | | | Change-Id: If8794c105e279afd4437517d3e61f2f900eb6490 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/118123 Tested-by: Jenkins Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
* use string_view in the Translate APINoel Grandin2021-06-171-1/+2
| | | | | | | Change-Id: I0bb0ea9d39ed623928060ffd3f2e2bc36ba33209 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/117272 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* add Toggleable as a separate thing to a ButtonCaolán McNamara2021-05-211-4/+4
| | | | | | | | | and inherit ToggleButton from both it and Button Change-Id: If0e500aca8d0ffa087cb5e2bfc1786372fbff4eb Reviewed-on: https://gerrit.libreoffice.org/c/core/+/115921 Tested-by: Jenkins Reviewed-by: Caolán McNamara <caolanm@redhat.com>
* tdf#124295 - Always select a 3D scheme in the chart wizardAndreas Heinisch2021-05-191-1/+1
| | | | | | | Change-Id: Ic0a39b4cb6f7af6fbdd99fc93677a0c2d966234d Reviewed-on: https://gerrit.libreoffice.org/c/core/+/115671 Tested-by: Jenkins Reviewed-by: Andreas Heinisch <andreas.heinisch@yahoo.de>
* use a single global item pool for chart2 draw modelNoel Grandin2021-05-071-2/+0
| | | | | | | | | which exposed a bug in SvxUnoNameItemTable::replaceByName Change-Id: If3207df6d46a2185b78ea5c3e1c145527b42d7cd Reviewed-on: https://gerrit.libreoffice.org/c/core/+/115126 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
* throw() -> noexcept, part 2/3: Automatic loplugin:noexcept rewriteStephan Bergmann2021-05-021-1/+1
| | | | | | | Change-Id: I076f16d0536b534abf0ced4d76051eadb4c0e033 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/114949 Tested-by: Jenkins Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
* tdf#133423 Fix 'central' formula for moving averageSamuel Mehrbrodt2021-03-101-3/+2
| | | | | | | Change-Id: I3685c310fdbe4809cb7863628be96645364c6a7a Reviewed-on: https://gerrit.libreoffice.org/c/core/+/112191 Tested-by: Jenkins Reviewed-by: Samuel Mehrbrodt <samuel.mehrbrodt@allotropia.de>