summaryrefslogtreecommitdiff
path: root/chart2
AgeCommit message (Collapse)Author
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-17java: final fields that can be staticNoel Grandin
Change-Id: I8c06be7bc0b8a38c662209f0de72a00550e25447
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-11fdo#84647 : Fixed default value for TickLableSkip value.sushil_shinde
1. 'TextCanOverlap' property was stored as false if TickLableSkip is not equal to one. 2. For OOXML charts TickLableSkip can be between 1 to 999999999. 3. We can not apply zero or less than zero value to TickLableSkip. ( As per specification) 4. In axis model default value for TickLableSkip was zero which is incorrect. 5. Added unit test to check 'TextCanOverlap' property for chart. Change-Id: Ib3104b1d932f6e9376c149eabb201c8e9ad23da9 Reviewed-on: https://gerrit.libreoffice.org/11901 Reviewed-by: Kohei Yoshida <libreoffice@kohei.us> Tested-by: Kohei Yoshida <libreoffice@kohei.us>
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-10loplugin:passstuffbyrefStephan Bergmann
Change-Id: If17bfdfdb51d5b9b8db563fccf09bb7608fcd0f8
2014-10-10fdo#54361: Add additional unit test case.Muthu Subramanian
Change-Id: Ifa3fd6f074d1957a468a1dad5dc2c223d38d24d4
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
2014-10-09fdo#81356: convert Fraction to boost::rational<long> - wipJuan Picca
* Added rational util functions used by Fraction class not available in the boost::rational class. * Replaced usage of Fraction by boost::rational<long> * Removed code that relies on: 1. fraction.IsValid() -- rational only allow valid values, ie denominator() != 0 2. rational.denominator() == 0 -- always false 3. rational.denominator() < 0 -- always false but implementation detail: http://www.boost.org/doc/libs/release/libs/rational/rational.html#Internal%20representation * Simplified code that relies on: 1. rational.denominator() != 0 -- always true * BUGS EXIST because Fraction allows the creation of invalid values but boost::rational throws the exception boost::bad_rational Change-Id: I84970a4956afb3f91ac0c8f726547466319420f9 Reviewed-on: https://gerrit.libreoffice.org/11551 Reviewed-by: David Tardon <dtardon@redhat.com> Tested-by: David Tardon <dtardon@redhat.com>
2014-10-09typo: Lable -> LabelAndras Timar
Change-Id: I8b9e9f75dd22ee5bff678c5bc0e1fa9381a103de
2014-10-08coverity#1237352 Dereference after null checkCaolán McNamara
Change-Id: Ia9cb84bb0924915d4611c5f18d2a0d19b4198d94
2014-10-08spelling: instanciated -> instantiatedNoel Grandin
Change-Id: I99f3010e30f81786b938dc11736ea1597cd5530d
2014-10-07Annotate code.Kohei Yoshida
Change-Id: If26d21593f36a39de3bc5b81717e5000795c97ae
2014-10-05coverity#1223085 Unchecked return valueNorbert Thiebaud
Change-Id: I4149684a6f704893e130bd1776850634fdb67ceb
2014-10-04coverity#703928 Unchecked return valueNorbert Thiebaud
Change-Id: I26ba62448796f1d80d2b4bbaf2499d08058c3add
2014-10-04coverity#1242729 Result is not floating-pointNorbert Thiebaud
Change-Id: I7a1c8c964dc05d90ac6f15affe75ed14c79c0eb0
generated by cgit (git 2.34.1) at 2025-01-22 17:21:31 +0000