summaryrefslogtreecommitdiff
path: root/tools
AgeCommit message (Collapse)AuthorFilesLines
2014-12-10long is 32 bit on Windows x86_64 platformDavid Ostrovsky1-1/+1
So that this is always wrong (on this platform) to write: reinterpret_cast<foo>(reinterpret_cast<long>(bar)) it should be: renterpret_cast<foo>(reinterpret_cast<sal_uIntPtr>(bar)) Change-Id: Ia286246ee1616988f755c2d2054b26efacc51af0 Reviewed-on: https://gerrit.libreoffice.org/13366 Reviewed-by: Michael Stahl <mstahl@redhat.com> Tested-by: Michael Stahl <mstahl@redhat.com>
2014-12-05fdo#38835 strip out OString globalsNoel Grandin1-63/+33
Change-Id: Id2eb27132fd6c1734e50c02617ce1bbb75e294a0
2014-12-05loplugin:cstylecast, involving pointer to incomplete typeStephan Bergmann1-4/+4
Change-Id: I73850d2b2ea6ff4376e25e0be39c65c35328a9a6
2014-12-03Fold URE: WindowsStephan Bergmann1-101/+0
...assuming the delayLoadHook in cli_ure/source/native/native_bootstrap.cxx is no longer necessary and loading of cppuhelper from the program dir cannot fail regardless in whatever scenario the cli_cppuhelper library itself is loaded. Change-Id: I13f32b327bca4cce9780864f5e57cdad3860afe5
2014-11-25assert on loading timestamp in kde80951-1.htmlCaolán McNamara1-2/+2
Change-Id: I32a9d6cc625537ab6e0940d93bf9a7544df370c8
2014-11-19Mark Color::AsRGBHexString() as constMiklos Vajna1-1/+1
Change-Id: Idf00e228c480afe2bbbb64682ed01f2c778b6d9d
2014-11-18Apply pimpl to ResStringArray, to hide its internals.Kohei Yoshida1-9/+48
Change-Id: I23969735f35e221a1aad39081ee63d02066d856f
2014-11-17Build fix on Linux.Kohei Yoshida1-0/+1
Change-Id: Iea96991a3bef08c17a8a0cbc347408dadc0061af
2014-11-17Apply pimpl to Fraction.Kohei Yoshida1-85/+133
Change-Id: I60eb597a6374b807b2264af6d5841a42e9b1c0f3
2014-11-17Make these methods non-inline.Kohei Yoshida1-0/+91
Change-Id: I0b24e34dec6c452659b224b45a6849dafe708c3b
2014-11-17sal: clean up public headers with include-what-you-useMichael Stahl9-0/+10
Sadly cannot forward declare "struct {...} TimeValue;". rtl/(u)?string.hxx still include sal/log.hxx but removing osl/diagnose.h was painful enough for now... Change-Id: Id41e17f3870c4f24c53ce7b11f2c40a3d14d1f05
2014-11-14coverity#982314 Logically dead codeCaolán McNamara1-4/+1
Change-Id: Iba420f0726864cb093e17fc9f4ce5bd7a33f7721
2014-11-14fdo#86023 - O[U]String needs a 'clear' methodBrij Mohan Lal Srivastava3-5/+5
Added clear() method to OString and OUString class, Updated appropriate call-sites. Change-Id: I0ba97fa6dc7af3e31b605953089a4e8e9c3e61ac Signed-off-by: Stephan Bergmann <sbergman@redhat.com>
2014-11-12Fix common typos. No automatic tools. Handmade…Andrea Gelmini3-4/+4
Change-Id: I1ab4e23b0539f8d39974787f226e57a21f96e959 Reviewed-on: https://gerrit.libreoffice.org/12164 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2014-11-11There are three positive return codes from inflateCaolán McNamara1-1/+1
#define Z_OK 0 #define Z_STREAM_END 1 #define Z_NEED_DICT 2 and we don't support dictionaries, so a Z_NEED_DICT return creates an infinite loop Change-Id: Iafb1da594962b3cb456a3223cc6d4122791718c5
2014-11-03checker type doesn't work, try event typeCaolán McNamara1-1/+1
Change-Id: I8b92c583bf1d6167b713648f417ba5569b3593ba
2014-11-01tools & vcl: move OutputDevice::ImplRotatePos() to Point::RotateAbout()Chris Sherlock2-0/+80
OutputDevice has a private function that rotates a point around another point. However, there is no real reason why OutputDevice should be responsible for this - it's really the responsibility of the Point class in the tools module. Therefore, I've moved this functionality out of OutputDevice and into Point, but I've renamed it from the rather confusing name "ImplRotatePos" to "RotateAround", which is what it actually does. Change-Id: If12fb40a7b476653224d4edfc01887bc91a80c7d Reviewed-on: https://gerrit.libreoffice.org/12171 Reviewed-by: Chris Sherlock <chris.sherlock79@gmail.com> Tested-by: Chris Sherlock <chris.sherlock79@gmail.com>
2014-11-01coverity#1250404 Operands don't affect resultCaolán McNamara1-0/+1
trying to auto-ignore this one, so don't touch it manually in the UI so I can experiment if this syntax will correctly auto-triage in all coverity instances Change-Id: I4ae42a6828041f40c6ab8cb3cfc631c8dd38e8c8
2014-10-31Removed duplicated includesAndrea Gelmini1-2/+0
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-28WaE: C4101: 'unused' : unreferenced local variableTor Lillqvist1-1/+1
Change-Id: I511c2df37c7feda8a3ffa311074bcf2fe669a73a
2014-10-28fix build on android-arm tinderboxDavid Tardon1-2/+2
Change-Id: If528f3cbaab66ceec07e35b479b4fe2ed55aa3c7
2014-10-28fdo#81356: use boost::rational internally in FractionJuan Picca2-304/+190
Change-Id: I6f40eafee7652209395bd471e3508fe3a3d19d73 Reviewed-on: https://gerrit.libreoffice.org/12085 Reviewed-by: David Tardon <dtardon@redhat.com> Tested-by: David Tardon <dtardon@redhat.com>
2014-10-28coverity#1242531 Untrusted loop boundCaolán McNamara1-0/+6
Change-Id: I94308a517028a62ecf7018f6d7a4e9d37c0f56a6
2014-10-27cid#705051 Dereference after null checkNoel Grandin1-1/+1
Change-Id: I6b7adb593efa137bb09b3b8d49539f6cd405dd20
2014-10-24Explicitly include the cmath std::abs overloadsStephan Bergmann1-0/+1
Change-Id: I501edd8f3301f70e6130d844dcd64cac8b18598f
2014-10-24Fix Fraction(-2147483648.0) for 32-bit wide longStephan Bergmann2-1/+8
Change-Id: I6465377de1edff5d0ccc0049fb6d24f6f1223fd2
2014-10-24Fract: enable testsJuan Picca2-9/+1
Change-Id: I00650503f15f2dcd45b07e3c529965336e8391e6 Reviewed-on: https://gerrit.libreoffice.org/12080 Reviewed-by: David Tardon <dtardon@redhat.com> Tested-by: David Tardon <dtardon@redhat.com>
2014-10-23Fraction: Revert "fdo#81356: convert Fraction to boost::rational<long> - wip"Jan Holesovsky6-276/+616
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 "just check if the fraction is < 0"Jan Holesovsky1-1/+1
This reverts commit 8afab33d161ddd39a3e6094b5daa08b156ef0421.
2014-10-23Fraction: Revert "simplify condition"Jan Holesovsky1-1/+1
This reverts commit ad1f796b1dc710a02ade17c95e99d11412f1280a.
2014-10-23Fraction: Revert "fdo#84854 it seems long is not enough on 32 bit"Jan Holesovsky2-9/+9
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-23coverity#440777 Resource leak in objectCaolán McNamara1-16/+20
rework to avoid the warning Change-Id: Id104c01b04e07ffaaf503a842c978acbc8a22a19
2014-10-18work around windows whiningNorbert Thiebaud1-3/+4
Change-Id: If2922bca25c93edc25e1171001def8ce3011cf4c
2014-10-17another blind attempt to fix the testDavid Tardon1-2/+2
Change-Id: I7ac53b340cf3a9f72a90414e09ddd37148ecd28a
2014-10-17correct castDavid Tardon1-2/+2
Change-Id: Ic79ce25bf4ceb4d715aabdabf3824304b02b1f38
2014-10-17now why did i do this?David Tardon1-1/+1
Change-Id: Icf1fc5643271672761b565c30382e70c689c29b4
2014-10-17fix long long -> BigInt for LONG_MAX valueDavid Tardon1-7/+7
Change-Id: I1cf551299bae925b5a5cf0a488b6cc3497d010bf
2014-10-17fix copypastaDavid Tardon1-1/+0
Change-Id: I08383e3086eef8e7c8136293d51b7108807a7def
2014-10-17fix comparisonDavid Tardon1-4/+4
Change-Id: Ia7f65d69c186056527a2beb72a18f4a66e4c6586
2014-10-17correct function nameDavid Tardon1-1/+1
Change-Id: Ibc3ab15bc830d68bf9d3d1d9ac9595d82c7fb6aa
2014-10-17allow construction of BigInt from sal_Int64 on 32 bitDavid Tardon3-0/+142
Change-Id: Ib68920fc9bd693d2f2679b4fc27d9956dc42fc86
2014-10-16fdo#84854 it seems long is not enough on 32 bitDavid Tardon2-9/+9
Fraction used BigInt internally for computations, rational does nothing like that. Change-Id: I3e9b25074f979bc291208f7c6362c3c40eb77ff5
2014-10-16Strip svidl down to what is still usedStephan Bergmann1-79/+0
...the generated .ilb, .lst, and .sid outputs requested by SdiTarget were apparently unused. Change-Id: I1abb2abc7945070451fce4a98a11d955515e3f63
2014-10-14fdo#75757: remove inheritance to std::mapTakeshi Abe1-18/+18
from UniqueIndexImpl. Change-Id: Iaa9040dff117ed5b05955c9f6eef31878dccf3b0 Reviewed-on: https://gerrit.libreoffice.org/11951 Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2014-10-14simplify conditionDavid Tardon1-1/+1
Change-Id: Idceac888c57cc27142877d0cf83dd525ab3f92ca
2014-10-14just check if the fraction is < 0David Tardon1-1/+1
Change-Id: I23f631898f29d8285d3da333686da8d3f28a00f8
2014-10-11initialize nPolyCountCaolán McNamara1-3/+3
Change-Id: I774f0ea2576063b7d1f7005e7df41ae52fa9037c
2014-10-10coverity#1240260 Operands don't affect resultCaolán McNamara1-1/+1
see can we silence these false positives coverity#1240259 Operands don't affect result coverity#1240254 Operands don't affect result coverity#1240264 Operands don't affect result coverity#1240267 Operands don't affect result Change-Id: Ieca40474c231e33a516b70f7693346ac73babd61
2014-10-09fdo#81356: convert Fraction to boost::rational<long> - wipJuan Picca6-616/+275
* 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-09remove SvRefBase::QueryDeleteNoel Grandin1-10/+0
Move it's functionality into the only place that needs it, in the dbase driver. Removes an extra virtual call from a widely used class. The dbase driver seems to be using to perform some kind of whacky object recycling, so it's not like we want this functionality to be used somewhere else. Change-Id: I41018f71e0b0a79fdd3d527536f0ac95c788e614 Reviewed-on: https://gerrit.libreoffice.org/11786 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>