summaryrefslogtreecommitdiffstats
path: root/sdext
Commit message (Collapse)AuthorAgeFilesLines
* sal: clean up public headers with include-what-you-useMichael Stahl2014-11-1710-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
* coverity#1019392 Uncaught exceptionCaolán McNamara2014-11-141-0/+5
| | | | Change-Id: Iab953700c9aa95fc3acd5f25233a942a31b7f5ad
* fdo#86023 - O[U]String needs a 'clear' methodBrij Mohan Lal Srivastava2014-11-144-9/+9
| | | | | | | Added clear() method to OString and OUString class, Updated appropriate call-sites. Change-Id: I0ba97fa6dc7af3e31b605953089a4e8e9c3e61ac Signed-off-by: Stephan Bergmann <sbergman@redhat.com>
* Revert "use the new OUString::fromUtf8 method"Stephan Bergmann2014-11-064-7/+9
| | | | | | | | This reverts commit 05050cdb23de586870bf479a9df5ced06828d498, not all places that use e.g. OStringToOUString to convert potential UTF-8 are guaranteed to fulfil the prerequisites necessary to use fromUtf8 (and some places like e.g. in codemaker are happy with the best-effort effect of OStringToOUString's OSTRING_TO_OUSTRING_CVTFLAGS).
* use the new OUString::fromUtf8 methodNoel Grandin2014-11-064-9/+7
| | | | Change-Id: I771004b7ccab3344a67e827e45bc34c22ffa5f77
* Resolves: fdo#76581 copy-and-paste -> slideshow crash in presenter consoleCaolán McNamara2014-11-061-8/+20
| | | | | | | | | | crash started after... commit 0218b0e2fa510d7acd7413dbedefd9ad50257f76 Date: Mon Jul 15 19:45:44 2013 +0300 fdo#65457 - Provide visual clues in presenter view. Change-Id: I19d84800bd5924f2dcc9e5debcf18ef95577105c
* coverity#1019393 Uncaught exceptionCaolán McNamara2014-11-051-28/+36
| | | | Change-Id: I818a91a3090392efed46700b5e2d77d7f1e3bdb9
* fdo#38835 strip out OUString globalsNoel Grandin2014-11-053-10/+8
| | | | | | | they are largely unnecessary these days, since our OUString infrastructure gained optimised handling for static char constants. Change-Id: I07f73484f82d0582252cb4324d4107c998432c37
* remove unnecessary 'using namespace rtl' declarationsNoel Grandin2014-10-2913-13/+0
| | | | | | | | | It turns out that almost none of them were necessary. Change-Id: I1311ed28409c682b57ea8d149bcbaf2c49133e83 Reviewed-on: https://gerrit.libreoffice.org/12133 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
* More -Werror,-Wunused-private-fieldStephan Bergmann2014-10-151-1/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ...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
* convert to floating point before divideCaolán McNamara2014-10-061-2/+2
| | | | Change-Id: I0328ca6594b3809210ea124dcb744bad3b5eb92e
* coverity#1242725 Result is not floating-pointNorbert Thiebaud2014-10-041-2/+2
| | | | Change-Id: I007ef5efa07f9124a08e049bca01aac87acfa48e
* remove HAVE_GCC_PRAGMA_DIAGNOSTIC_SCOPE check and macroMichael Stahl2014-10-021-4/+0
| | | | | | | This is supported in GCC 4.6.0 already: https://gcc.gnu.org/onlinedocs/gcc-4.6.0/gcc/Diagnostic-Pragmas.html Change-Id: I2f67e588eea3a323a2e9c81e39e56ab2e715a817
* coverity#1242882 Result is not floating-pointCaolán McNamara2014-10-021-2/+2
| | | | Change-Id: I5072c29188e33b5711e67188441c497bf8759891
* coverity#982849 try and fix Unrecoverable parse warningCaolán McNamara2014-09-261-1/+1
| | | | Change-Id: Iba0f175c7c079f2171bab6e5a7419c7d5966c541
* what is it with the explicit voidsCaolán McNamara2014-09-261-8/+8
| | | | Change-Id: I7d3995f16351cb1686003f2ba7a044ac99d6d10b
* fdo#82577: Handle FontNoel Grandin2014-09-181-1/+1
| | | | | | | Put the VCL Font class in the vcl namespace. Avoids clash with the X11 Font typedef. Change-Id: I1a84f7cad8b31697b9860a3418f7dff794ff6537
* sdext: check the copy on the stack, not the memberMichael Stahl2014-09-081-1/+1
| | | | Change-Id: Id8aaccf5fa31ad670215db61853bb729302b1fec
* Related fdo#82088: dropping aliases again :)Stefan Weiberg2014-09-066-164/+159
| | | | | | | Change-Id: I41f6472423b60e3699ed6158cc5eccb10de1538d Reviewed-on: https://gerrit.libreoffice.org/11295 Reviewed-by: Norbert Thiebaud <nthiebaud@gmail.com> Tested-by: Norbert Thiebaud <nthiebaud@gmail.com>
* Resolves: fdo#83137 NULL pActionCaolán McNamara2014-08-271-1/+1
| | | | Change-Id: I74c4e32aae1e3aa74d197f132c900627062b6d01
* set names on a bunch more threads...Michael Stahl2014-08-251-0/+2
| | | | Change-Id: I4c2f2f0bcce52839033876ec991529721e06a3c8
* fix string out-of-bounds assert on rhbz496762-1.pdfCaolán McNamara2014-08-201-1/+4
| | | | Change-Id: I224e3c9befd1a9ab3ba0cc2676cd6b4ba51214c8
* fdo#75757: remove inheritance to std::vectorTakeshi Abe2014-08-091-12/+18
| | | | | | | | | ... for PaneStyleContainer and ViewStyleContainer. Change-Id: I6a6e2b1733266f54a94a49ab491cf854eb5df492 Reviewed-on: https://gerrit.libreoffice.org/10847 Reviewed-by: David Tardon <dtardon@redhat.com> Tested-by: David Tardon <dtardon@redhat.com>
* avoid memory leakDavid Tardon2014-08-011-3/+5
| | | | Change-Id: I9ac97c4e5c7b01b5e05cb1132c0d245f5843799e
* fix parsing of cmdline argsDavid Tardon2014-08-011-6/+6
| | | | Change-Id: I4005c08d4163d5d26f227e01c2520379f717343a
* Does not look like Library_pdfimport needs popplerStephan Bergmann2014-07-031-1/+0
| | | | Change-Id: I1dd162f939ce4f0689ea14724c1e7cec8a6c16b0
* Improve test codeStephan Bergmann2014-07-031-11/+12
| | | | | | (never call CPPUNIT_ASSERT etc., which work by throwing exceptions, from a dtor) Change-Id: I293d54eb40c2ad9205d485ccff0ffd2161257142
* Check exit code of spawned xpdfimportStephan Bergmann2014-07-031-1/+34
| | | | Change-Id: I4bea1ebe5c9915ad5e3a4f8fecb516bc056d060d
* Remove double assignmentPeter Senna Tschudin2014-06-301-1/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The semantic match that finds this problem is follows: (http://coccinelle.lip6.fr/) // <smpl> @r@ expression i,f; position p1,p2; @@ ( (<+...i++...+>) = ...; | (<+...++i...+>) = ...; | (<+...i--...+>) = ...; | (<+...--i...+>) = ...; | i = <+...f(...)...+>; | i@p1 = ...; ( i = <+...i...+>; | i = <+...f(...)...+>; | i@p2 = ...; ) ) @@ expression i; position r.p1,r.p2; @@ * i@p1 = ...; i@p2 = ...; // </smpl> Change-Id: Ifacb878d58486f2645560592484ce24b2bc0bd6f Reviewed-on: https://gerrit.libreoffice.org/9968 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
* loplugin:unreffun: also warn about redundant redeclarationsStephan Bergmann2014-06-272-6/+1
| | | | Change-Id: I9a812220b58cf6da00d854e65794f7c673ab239d
* prevent invalid memory accessMarkus Mohrhard2014-06-211-1/+1
| | | | | | e.g. fdo26139-1.pdf Change-Id: Ic5854e2056c246d48e2962dfe0a91f67c2c30c04
* loplugin:staticcallStephan Bergmann2014-06-134-6/+6
| | | | Change-Id: Iacd3efa9e5d6103ad92e15884e8aa010d7b2ae93
* loplugin: inlinesimplememberfunctionsNoel Grandin2014-06-097-40/+8
| | | | Change-Id: I42119f656ca528286fb25d2d36c0af54b7d04a6b
* fixincludeguards: fix include guardsThomas Arnhold2014-06-061-2/+2
| | | | | | | Change-Id: Ie6e8d4272b0b1d0d2ce93bcbc2e818a9eac1a56b Reviewed-on: https://gerrit.libreoffice.org/9629 Reviewed-by: Thomas Arnhold <thomas@arnhold.org> Tested-by: Thomas Arnhold <thomas@arnhold.org>
* sdext: remove SAL_THROW macroNoel Grandin2014-06-0510-17/+7
| | | | Change-Id: Ib8649fb4d5aa2249d99d5c9d9ecf9d2307a93c72
* update_pch: add a bunch of pch filesThomas Arnhold2014-06-046-0/+288
| | | | | | | | | | | | | | desktop: 1m51s -> 54s framework: 1m55s -> 1m04s package: 32s -> 16s sdext: 1m31s -> 47s svgio: 32s -> 15s uui: 49s -> 20s vbahelper: 1m44s -> 27s xmlscript: 15s -> 10s xmlsecurity: 45s -> 23s Change-Id: Ia437969c091bf877983ababc5ea2d044bbc0bee0
* coverity#982645 bMissingDefaultsToNormal is always trueCaolán McNamara2014-06-042-11/+7
| | | | Change-Id: Ica58be0b5512eb8f386e51ec6e84d0ec09c3027f
* DeInitVCL at end of testsStephan Bergmann2014-06-041-0/+1
| | | | | | | | | | | | | | | | | | | | | 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
* Use boost::hash_value to avoid undef conv of neg/large floating -> size_tStephan Bergmann2014-06-021-23/+24
| | | | Change-Id: I85853ffc6083d46c02c08f14d5d3321803c31301
* remove more unnecesary OUString constructor useNoel Grandin2014-05-293-14/+13
| | | | | | when throwing exceptions Change-Id: I6edfb6b6745499f802b0e3c0e096a36fb7c32aac
* remove unnecessary NULL parameter passed to UNO ExceptionNoel Grandin2014-05-291-1/+1
| | | | | | .. now that we have a default value for that parameter Change-Id: I54d6b0a0a01ce2f2e5168ada0c427424de0477bc
* remove unnecessary use of Reference constructor in throwNoel Grandin2014-05-272-5/+5
| | | | | | | | | | | Convert code like this: throw IOException("xx", Reference< XInterface >(static_cast<OWeakObject*>(this)) ); to this: throw IOException("xx", static_cast<OWeakObject*>(this) ); Change-Id: Ife9f645f0f1810a8e80219126193015502c43dbb
* Remove unnecessary semicolonsPeter Senna Tschudin2014-05-271-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | A simplified version of the semantic match that finds this problem is follows: (http://coccinelle.lip6.fr/) // <smpl> @r1@ statement S; position p,p1; @@ S@p1;@p @script:python r2@ p << r1.p; p1 << r1.p1; @@ if p[0].line != p1[0].line_end: cocci.include_match(False) @@ position r1.p; @@ -;@p // </smpl> Change-Id: Ib9708d37fbb4c6060f88d5dae3814a2d37b2091e Reviewed-on: https://gerrit.libreoffice.org/9493 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
* coverity#983807 Uncaught exceptionCaolán McNamara2014-05-241-36/+14
| | | | Change-Id: Ic303dae356690db009b5917168b77da2f2d8d433
* Prefer cppu::UnoType<T>::get() to ::getCppuType((T*)0) part20Julien Nabet2014-05-231-17/+17
| | | | Change-Id: If87cdfb2c605254f6d69baa4ca5aec09091caa68
* Correct linker library sequence for pdfimportJan-Marek Glogowski2014-05-201-1/+1
| | | | | | Otherwise I get a lot of missing fontconfig symbols. Change-Id: I9bc3e395d54380a36cd6dd73cf95b7f4e4cab54e
* we need fontconfig nowCaolán McNamara2014-05-201-0/+1
| | | | Change-Id: I0482f613d7c908488e7d9f84f891bd521e91cc15
* fdo#78689 PDF Import: get font's ascent value from different sourceVort2014-05-206-47/+57
| | | | | | | Change-Id: I19018d25ef53bbea225bb5a9ef806ce5c1b4adc7 Reviewed-on: https://gerrit.libreoffice.org/9410 Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
* coverity#984442 Use after freeCaolán McNamara2014-05-161-1/+1
| | | | Change-Id: I28822f6b74cbf8ace8f7a194248840cb401679be
* fdo#78549 PDF Import: fix incorrect transformation matrix writingVort2014-05-151-2/+2
| | | | | | | Change-Id: I60d77e01ea7774c234d1ad1b81b90a7db3461e22 Reviewed-on: https://gerrit.libreoffice.org/9310 Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
generated by cgit (git 2.34.1) at 2024-07-22 05:12:46 +0000