summaryrefslogtreecommitdiff
path: root/chart2/source
AgeCommit message (Collapse)Author
2014-11-04remove some unused code in chart2Noel Grandin
Change-Id: I99afd30ad413b57d27c89421b5010f6d88856496
2014-11-03coverity#1242524, arguments in wrong orderMarkus Mohrhard
2014-11-01coverity#704564 Unchecked dynamic_castCaolán McNamara
Change-Id: Ife93dd20c08bf2a30298384099cd36592091c336
2014-10-31Removed duplicated includesAndrea Gelmini
Change-Id: I5362d997bfa086c9fb1726efcb15132a966684f6 Reviewed-on: https://gerrit.libreoffice.org/12160 Tested-by: LibreOffice gerrit bot <gerrit@libreoffice.org> Reviewed-by: Michael Stahl <mstahl@redhat.com>
2014-10-29coverity#1242433 SdrMakeOutliner alway derefs pModCaolán McNamara
so change from a pointer to a reference Change-Id: I81eb2c9e4df8353fbbdad7058c6ca7ea22286e62
2014-10-25coverity#1249459 Unchecked dynamic_castCaolán McNamara
Change-Id: I5e212976f1be310c3a802679a0d8b93c2866d591
2014-10-25coverity#1249460 Unchecked dynamic_castCaolán McNamara
Change-Id: I21f85d0e18ab8a30320f3db294efa9fbdd4dd55d
2014-10-24coverity#704564 Unchecked dynamic_castCaolán McNamara
Change-Id: I2dfdc5e1e852011b532b484e566896133208ee86
2014-10-23Fraction: Revert "fdo#81356: convert Fraction to boost::rational<long> - wip"Jan Holesovsky
This reverts commit 47a2d7642d249d70b5da0c330a73f3a0032e4bba. Conflicts: cui/source/tabpages/transfrm.cxx svx/source/svdraw/svdedtv1.cxx svx/source/svdraw/svdibrow.cxx sw/source/filter/ww1/w1filter.cxx tools/source/generic/rational.cxx Change-Id: I4849916f5f277a4afef0e279b0135c76b36b9d15
2014-10-23Fraction: Revert "fdo#84854 it seems long is not enough on 32 bit"Jan Holesovsky
This reverts commit 582ef22d3e8e30ffd58f092d37ffda30bd07bd9e. Conflicts: svx/source/svdraw/svdedtv1.cxx svx/source/svdraw/svdibrow.cxx sw/source/filter/ww1/w1filter.cxx Change-Id: I80abc7abdeddc267eaabc9f8ab49611bb3f8ae83
2014-10-23loplugin: cstylecastNoel Grandin
Change-Id: I4aafc170895d8bab47206c7b07b4f1f6105d42d3
2014-10-20chart2: fix memory leak due to cyclic reference in VAxisBaseMichael Stahl
The VAxisBase::m_xNumberFormatsSupplier refers to the ChartModel itself, and apparently that is a cyclic reference. Naively using the ChartModel's m_xNumberFormatsSupplier in ChartView::impl_createDiagramAndContent() because it will later be passed to AxisHelper::getExplicitNumberFormatKeyForAxis(), which expects to be able to convert it to a ChartModel. Since passing around the ChartModel as an XNumberFormattingSupplier is sort of un-intuitive anyway, refactor some methods to use XChartDocument instead, and only create the VPolarAxis / VCartesianAxis with the ChartModel's m_xNumberFormatsSupplier. The drawback is that if ChartModel::attachNumberFormatsSupplier() is called after ChartView::update() has created the axes, it may not have an effect on them; not sure if that is a real or hypothetical problem. Change-Id: Ib5f0d5882b85adaf44f80e086f19178b3e64882f
2014-10-20fdo#84938: replace TIB_ constants with enumNoel Grandin
Change-Id: I435ce2331fb49e7ce9fe97bdfddfaef706759a84 Reviewed-on: https://gerrit.libreoffice.org/12023 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2014-10-17coverity#1247647 Uncaught exceptionCaolán McNamara
Change-Id: I1e47f3970f944aa3556e4ddb9b7918d9ee23c0eb
2014-10-16fdo#84854 it seems long is not enough on 32 bitDavid Tardon
Fraction used BigInt internally for computations, rational does nothing like that. Change-Id: I3e9b25074f979bc291208f7c6362c3c40eb77ff5
2014-10-15More -Werror,-Wunused-private-fieldStephan Bergmann
...detected with a modified trunk Clang with > Index: lib/Sema/SemaDeclCXX.cpp > =================================================================== > --- lib/Sema/SemaDeclCXX.cpp (revision 219190) > +++ lib/Sema/SemaDeclCXX.cpp (working copy) > @@ -1917,9 +1917,10 @@ > const Type *T = FD.getType()->getBaseElementTypeUnsafe(); > // FIXME: Destruction of ObjC lifetime types has side-effects. > if (const CXXRecordDecl *RD = T->getAsCXXRecordDecl()) > - return !RD->isCompleteDefinition() || > - !RD->hasTrivialDefaultConstructor() || > - !RD->hasTrivialDestructor(); > + return !RD->hasAttr<WarnUnusedAttr>() && > + (!RD->isCompleteDefinition() || > + !RD->hasTrivialDefaultConstructor() || > + !RD->hasTrivialDestructor()); > return false; > } > > @@ -3517,9 +3518,11 @@ > bool addFieldInitializer(CXXCtorInitializer *Init) { > AllToInit.push_back(Init); > > +#if 0 > // Check whether this initializer makes the field "used". > if (Init->getInit()->HasSideEffects(S.Context)) > S.UnusedPrivateFields.remove(Init->getAnyMember()); > +#endif > > return false; > } to warn about members of SAL_WARN_UNUSED-annotated class types, and warn about initializations with side effects (cf. <http://lists.cs.uiuc.edu/pipermail/cfe-dev/2014-October/039602.html> "-Wunused-private-field distracted by side effects"). Change-Id: I3f3181c4eb8180ca28e1fa3dffc9dbe1002c6628
2014-10-14Take the current axis alignment state.Kohei Yoshida
Change-Id: I02c28f44af0a1510bdb780d1e6c70561b7ba24e5
2014-10-14Revert "Revert "Make VCartesianAxis::get2DAxisMainLine() truly const method.""Stephan Bergmann
This reverts commit bf302000ea8849ac7ecd9162f3014bac9da4406e, as clarified on IRC, --enable-chart-tests is known flaky and not intended to be used by anybody but moggi.
2014-10-14Revert "Make VCartesianAxis::get2DAxisMainLine() truly const method."Stephan Bergmann
This reverts commit 410c5bd58c36d9064a79520d5bc31cb32ead1326, it causes CppunitTest_chart2_xshape to fail.
2014-10-13More on documenting the methods.Kohei Yoshida
Change-Id: Ibdbc1b26521e56de5dcba1b9572bb0221fc0fff5
2014-10-13Scope level reduction & explain what the max axis index really is.Kohei Yoshida
The concept of axis index and dimension index is quite confusing in the chart2 code.... Change-Id: I035af0a225396bd5394089d6f584f78dcaf0ad31
2014-10-13Explain what doAutoScaling() does.Kohei Yoshida
Change-Id: I15ca9ca0c3a22094f0a969adf2b3f1145d116052
2014-10-13Make VCartesianAxis::get2DAxisMainLine() truly const method.Kohei Yoshida
No internal state gets modified by this method now. Change-Id: I10475ad3483cb3a92e41208e1a55a6b925c10223
2014-10-10Make the variable name shorter and sweeter...Kohei Yoshida
Change-Id: Ibea26f3c99b50f2011050960794d9736566ee7ce
2014-10-10prepareScaleAutomatismForDimensionAndIndex -> prepareAutomaticAxisScaling.Kohei Yoshida
Change-Id: Ic66d2fcf78603bdd4fa1e08933900f49c66384b7
2014-10-10Hide the 3rd nested looping for clarity.Kohei Yoshida
Also aScaleAutomatism -> aAutoScaling. The word 'automatism' is a bit awkward. Change-Id: I9856f904d5693e79e19176ba5fff881267d5c7e7
2014-10-10Make this loop a bit easier to follow...Kohei Yoshida
Change-Id: Ib978f47925e1f7d77e02586bff50d24fa5b27f2c
2014-10-10Make this a hard prerequisite.Kohei Yoshida
Change-Id: Ib1c859b6033b2cf148e1ed74a437dda85872dffe
2014-10-10Use boost::ptr_vector for this...Kohei Yoshida
which also exposed few cases of mis-used const-ness here and there... Change-Id: Ie4374459643fc050ded646762d72ad622cf3f843
2014-10-10coverity#1223085 Unchecked return valueCaolán McNamara
there were three of these, not just one i.e. see also commit 276a051ef5dc144a202633779259a4ecd43c81a8 Date: Sun Oct 5 13:05:04 2014 -0500 coverity#1223085 Unchecked return value Change-Id: I07ee033ae31a346a08f68a6edfa480505fe6c11a
2014-10-10coverity#707673 Uninitialized scalar fieldCaolán McNamara
Change-Id: Ic631ebc74e94f4ccf7e3915db3952714fd4d28d4
2014-10-10Fix CppunitTest_chart2_xshapeStephan Bergmann
...regression from d7e477c4518bb3f638af59fa48cae04eba8c76cc "Use boost::optional for these members." Change-Id: I8e02033d75fd52489be03b34fbce494ae49ab7e3
2014-10-10Further fallout from git mis-merge?Stephan Bergmann
...in a80f74add22c303327dd3203d9be5eeb901a73a4 "Put SeriesPlotterContainer into CreateShapeParam2D." Change-Id: I9e2f700a1999070069ff87fc3030c00a045372c6
2014-10-09Fix git's wrong 3-way merge.Kohei Yoshida
Change-Id: I30520ff78f47bcfa487cea6f40904bf718610967
2014-10-09Put SeriesPlotterContainer into CreateShapeParam2D.Kohei Yoshida
Change-Id: I537c131c0c927d2cbbd7cd44f07e9e4748b8ed6c
2014-10-09Explain what updateScreenValues() does.Kohei Yoshida
Change-Id: I323827f259c9f09c5b4bfe882c63096dea640dbf
2014-10-09Use typedef rather than raw nested std::vector everywhere...Kohei Yoshida
Change-Id: Id4b58d13b229776f2ef1f270e937547ece9e1248
2014-10-09Make its own header the first header.Kohei Yoshida
Change-Id: I81ff1cb9200bdccd3f50dc08fa68fc69929e7c90
2014-10-09Clean this up a bit.Kohei Yoshida
Change-Id: Ief207c72a22eee6a745f885912ea929ade717321
2014-10-09Description for m_aAllTickInfos.Kohei Yoshida
Especially wrt why it is nested vectors. Change-Id: I143785778ef62abbdc047a2d18c713f43a6522d9
2014-10-09Make AxisProperties non-virtual; nobody derives from it.Kohei Yoshida
Change-Id: I1cb5203374b6d13a914c335d73a7db6d8a47c4bb
2014-10-09B2DVector supports operator!=. Let's use it.Kohei Yoshida
Change-Id: Iecf58e0f24909ec2fd2780a5f4b219105c7c4b47
2014-10-09Make it clear which bounding rectangle it returns & const.Kohei Yoshida
Change-Id: I9c8080225cfbe0f38133ce2503bd2d0c1d0f5e13
2014-10-09xInverseScaling is set but not used. Remove it.Kohei Yoshida
Change-Id: I1d185df4b14e152f4da9dfdbc2e71504c2cb1f3b
2014-10-09Re-organize method / class descriptions.Kohei Yoshida
Change-Id: Ib7fe33581c3aec2f58f8a16a07215b69080e0a51
2014-10-09Make this data member const, as it never gets modified once constructed.Kohei Yoshida
Change-Id: Ic01fd0591bbb524b4a6a0ec1dfe448e33f4d6b18
2014-10-09TickFactory_2D -> TickFactory2D.Kohei Yoshida
For consistent naming. Change-Id: Ic94681d4d18c0330ea5a8393c1b3d863c81e1ea8
2014-10-09Make thie method consistent with the others.Kohei Yoshida
Change-Id: I028f0dd833cb2307e842805636458f51c71dbd66
2014-10-09Simplify AxisProperties and its surrounding code.Kohei Yoshida
Change-Id: Ie47421d778c2beb8eb8e132532220054f183a5f1
2014-10-09Use boost::optional for these members.Kohei Yoshida
Change-Id: I60c52bf99d0e63e27f7c576e2582c26e19809c8c
generated by cgit (git 2.34.1) at 2025-02-15 11:29:16 +0000