summaryrefslogtreecommitdiff
path: root/unotools
AgeCommit message (Collapse)Author
2015-06-18Fix typosAndrea Gelmini
Change-Id: Ibfa971952cd330ddf430e7e951c5c235d2ae6bd8 Reviewed-on: https://gerrit.libreoffice.org/16314 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Michael Stahl <mstahl@redhat.com>
2015-06-16Fix typosAndrea Gelmini
Change-Id: Id8c7cf460cba5af89891c470e1aaa61b227d356e Reviewed-on: https://gerrit.libreoffice.org/16310 Reviewed-by: Michael Stahl <mstahl@redhat.com> Tested-by: Michael Stahl <mstahl@redhat.com>
2015-06-15remove unnecessary check for null when calling deleteNoel Grandin
Idea originally from caolan. Found using the following command: find . -name *.cxx | xargs /opt/local/bin/grep -zlP '(?m)if\s*\(\s*\w+\s*\)\s*delete\s+\w+\;' Change-Id: I3338f4e22193a6dfd6219c8c75835224a3392763
2015-06-08add (and call before dtor) Commit to SvtSearchOptionsCaolán McNamara
i.e. follow up to commit 465359c35fcd8c30a2bedd3d0beb07c0c1c36cba Author: Michael Stahl <mstahl@redhat.com> Date: Wed Mar 11 16:39:24 2015 +0100 do not call virtual Commit() from dtors of utl::ConfigItem subclasses ~SvtAppFilterOptions_Impl() demonstrates that this is a bad idea, by not invoking its subclasses' ImplCommit() but its own. to avoid assert in SvtSearchOptions dtor Change-Id: I0eaf93450e1daddb81a9463f2c1df48a334dfb22
2015-06-08loplugin:cstylecast: deal with remaining pointer castsStephan Bergmann
Change-Id: Ic05bc081ec190cc80d8821630e94f51c8db225b6
2015-05-28new clang plugin: loopvartoosmallNoel Grandin
Idea from bubli - look for loops where the index variable is of such size that it cannot cover the range revealed by examining the length part of the condition. So far, I have only run the plugin up till the VCL module. Also the plugin deliberately excludes anything more complicated than a straightforward incrementing for loop. Change-Id: Ifced18b01c03ea537c64168465ce0b8287a42015
2015-05-26cppcheck: noExplicitConstructorCaolán McNamara
Change-Id: If947733a205e8ece1845079be95cbc2d6cbd5029
2015-05-22loplugin:constantfunctionNoel Grandin
Change-Id: I7cb5b0c2cf9ade557173ca596ea5d42d853ff448
2015-05-15tdf#91064: Revert "tdf#89790 DOCX: saving LO version number in app.xml"Joren De Cuyper
This reverts commit 480ca7434a330b2678f9ef287cffd6d9cf27bed5. Change-Id: I69a16425fc36979d49f409bbd7921495a22a35dc Reviewed-on: https://gerrit.libreoffice.org/15737 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Joren De Cuyper <jorendc@libreoffice.org>
2015-05-10NOINSTANCE is implied now in IMPL_STATIC_LINK...Stephan Bergmann
Change-Id: Ifb032457d6c1b279c4183282ef2b271c706dd71a
2015-05-08lopluign:staticmethods: Handle DECL_LINKStephan Bergmann
Change-Id: Ib27854a8470f3ff5b208cb949a7bd02f2a86c969
2015-05-07fix for CONFIG_MODE constant -> enum conversionNoel Grandin
in my commit b35828ba7dc33af81d17c8c870ec981d18a57b08 "convert CONFIG_MODE constants to scoped enum" Change-Id: Iafb6b1643920eb066a2d50150f408161331f9d04
2015-05-06fix windows buildNoel Grandin
warning C4334: '<<' : result of 32-bit shift implicitly converted to 64 bits (was 64-bit shift intended?) after my commit 60e659ae355d073b05b3b9c8d2ac3bdc5f76020d "convert IMPL_FONT_ATTR constants to scoped enum" Change-Id: I83f1ea5869fa60c89dbc706020f68cc2c4277f48
2015-05-06Add comments about ImplFontAttrsTor Lillqvist
Change-Id: I32d8b8f4301188245721d31fd234bb1784a1ae8d
2015-05-06convert USER_OPT constants to scoped enumNoel Grandin
Change-Id: I85a1c9f89ac79e0e54fcd8dfd1908325fae9a8ac
2015-05-06convert DEFAULTFONT_ constants to scoped enumNoel Grandin
Change-Id: Ia33e957f6cf530e2639b3c86d9482f642652cb46
2015-05-06convert SUBSFONT_ to scoped enumNoel Grandin
Change-Id: Ic66191ac4cdfa753dc784e7bd1a6dd20f50def6b
2015-05-06convert FONTTOSUBSFONT constants to scoped enumNoel Grandin
Change-Id: I994bfc76d709d01406404991dbfbd49b15bd03f4
2015-05-06convert IMPL_FONT_ATTR constants to scoped enumNoel Grandin
Change-Id: I2b5986ea69dc09e6ba18b23a80029f6551b48152
2015-05-06convert CVC_ flags to scoped enumNoel Grandin
Change-Id: Iadc9b6240121e9fee91de299b94a9f8629656507
2015-05-06convert CONFIG_MODE constants to scoped enumNoel Grandin
Change-Id: I1b0fb976e7bf8d7de06da0c30012e8bba1320498
2015-05-05loplugin:staticmethodsNoel Grandin
Change-Id: I97160fc51dc16ff92b06d44570298eeec637e132
2015-05-01tdf#89790 - DOCX: saving LO version number in correct tag in app.xmlRadu Ioan
Converted existing method that provides application name and version in two functions Change-Id: I93b70da7c78ee5db3d6eb710a2a2a9f4b7b4fb86 Reviewed-on: https://gerrit.libreoffice.org/15151 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk>
2015-04-30Gradually typed LinkStephan Bergmann
Turn the Link class into a template abstracting over the link's argument and return types, but provide default template arguments that keep the generic, unsafe "void* in, sal_IntPtr out" behvior. That way, individual uses of the Link class can be updated over time. All the related macros are duplicated with ..._TYPED counterparts, that additionally take the RetType (except for LINK_TYPED, which manages to infer the relevant types from the supplied Member). (It would have been attractive to change the "untyped" LinkStubs from taking a void* to a properly typed ArgType parameter, too, but that would cause -fsanitize=function to flag uses of "untyped" Link::Call.) Change-Id: I3b0140378bad99abbf240140ebb4a46a05d2d2f8
2015-04-29rhbz#1213173: connectivity: Calc driver: prevent document being disposedMichael Stahl
... by adding a XCloseListener that vetoes any attempt to close it. The Calc document can be opened by the user in the UI and closed again. Change-Id: Ied427b67274d925c911e516c0a50a4c0b2b18db9
2015-04-29convert STR_EVENT constants to scoped enumNoel Grandin
- also add needed infrastructure to o3tl::enumarray so we can use std::find on it - move the Impl inside the .cxx file, no need to expose it in the header Change-Id: I7758a6175849f46723d97b1e657f846524c3b7cd
2015-04-28SvtSecurityOptions_Impl::m_nSecLevel is of type sal_Int32Stephan Bergmann
Change-Id: I82576e00de006caa6f7f6e414e5399d035aed1bf
2015-04-24loplugin:simplifyboolStephan Bergmann
Change-Id: I56557cd02bbb97d3c3aa4053608cf2dd48c35e7a
2015-04-23Improved loplugin:literaltoboolconversion looking into cond. exprs.Stephan Bergmann
...automatic rewriter fixes Change-Id: I6b04ca80f08f8a71ff94e309fd52f44d736751ee
2015-04-22Various #include <sal/log.hxx> fixupsStephan Bergmann
rtl/string.hxx and rtl/ustring.hxx both unnecessarily #include <sal/log.hxx> (and don't make use of it themselves), but many other files happen to depend on it. Cleaned up some, but something like grep -FwL sal/log.hxx $(git grep -Elw \ 'SAL_INFO|SAL_INFO_IF|SAL_WARN|SAL_WARN_IF') -- \*.cxx) shows lots more files that potentially need fixing before the include can be removed from rtl/string.hxx and rtl/ustring.hxx. Change-Id: Ibf033363e83d37851776f392dc0b077381cd8b90
2015-04-21try to find a unique naming scheme for temp filesMarkus Mohrhard
This should hopefully avoid name clashes during the test runs. During the test runs two tests can try to access the same temp file which causes one test to fail. Using the pid as part of the temp file name should avoid that problem. Change-Id: I696c3d34e33a5d8e9234238d86f1201ecdfa66ec Reviewed-on: https://gerrit.libreoffice.org/15455 Tested-by: Jenkins <ci@libreoffice.org> Tested-by: Christian Lohmaier <lohmaier+LibreOffice@googlemail.com> Reviewed-by: Christian Lohmaier <lohmaier+LibreOffice@googlemail.com>
2015-04-16improve commentStephan Bergmann
Change-Id: I451ab0fb0e7ee1a6d20091caa9c1142b16796dc7
2015-04-16tdf#89394. fix hangs because of unavailable Windows UNC path.Jan Kantert
UNC: \\server.domain\file.odt is stored as file://server.domain/file.odt. Windows local: C:\folder\file.odt is stored as file:///C:/folder/file.odt. Linux: /home/user/file.odt is stored as file:///home/user/file.odt. I cannot find any case where we only have file:// (but not a third slash) on a local file. Change-Id: Ib15b3380cb40f1cd37123be7a38426b74fbde5cf Signed-off-by: Stephan Bergmann <sbergman@redhat.com>
2015-04-15remove unnecessary use of void in function declarationsNoel Grandin
ie. void f(void); becomes void f(); I used the following command to make the changes: git grep -lP '\(\s*void\s*\)' -- *.cxx \ | xargs perl -pi -w -e 's/(\w+)\s*\(\s*void\s*\)/$1\(\)/g;' and ran it for both .cxx and .hxx files. Change-Id: I314a1b56e9c14d10726e32841736b0ad5eef8ddd
2015-04-07convert SvtModuleOptions::EModule to scoped enumNoel Grandin
Change-Id: I9665fc003cb3a44f4db857b24584fa3a70487259
2015-04-07convert SvtModuleOptions::EFactory to enum classNoel Grandin
and fix issues in iterating through o3tl::enumarray Change-Id: Ia59ef9be44d8c92c2e406fa71aa92269578e26e3
2015-04-04Typo: creat->createJulien Nabet
Change-Id: I5e57e6d506504933096f4c8e371b0ddc93c04cbd
2015-04-03Fix "unknown factory" log, declare and use FACTORYNAME_BASICJulien Nabet
Also use FACTORYNAME_STARTMODULE in SvtModuleOptions::ClassifyFactoryByServiceName Change-Id: I1ab87f13a9a1d071bf6abaae137fc6470101d711 Reviewed-on: https://gerrit.libreoffice.org/15113 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2015-04-02Change INetProtocol enumerators to CamelCaseStephan Bergmann
...to avoid clashes with macros and poor warnings about hiding global FILE Change-Id: I444e8b958f029201b228937e088efabc4b6d3050
2015-04-02Change INetProtocol to scoped enumerationStephan Bergmann
...and fix o3tl::enumarray::operator [] const overload Change-Id: I749b1b9d68686b03a97074253478d9d2d9d32b0b
2015-04-02loplugin:staticmethodsNoel Grandin
Change-Id: I3ae555fd9f0dc3c11ea301ee1cb2c799fbec820d
2015-04-01Replace remaining getCppuType et al with cppu::UnoTypeStephan Bergmann
Change-Id: I6b0974024e61e02639106d4ca3abcdcdb8e6fa87
2015-03-31Reduce to static_cast any reinterpret_cast from void pointersStephan Bergmann
Change-Id: I9e12cea5aba3b78e1f6ed284aedb7531d107edbf
2015-03-28Clean up C-style casts from pointers to voidStephan Bergmann
Change-Id: I8c40cbe7c6928d89eb280344436a4b68a4884bd1
2015-03-26const_cast: convert some C-style casts and remove some redundant onesStephan Bergmann
Change-Id: I72a618a0deca795b87ec009105252f4594ecba90
2015-03-22introduce ODFSaneDefaultVersion; ODFDefaultVersion is a mess, mess, mess..Eike Rathke
A running ODFVER_LATEST is fine for configuration purposes, but not for determining how to store a feature. Change-Id: I8ebc13f900f62f8c36d2cdc50a18855fc6693527
2015-03-21Char highlight: option about export LO character background to MSO formatsZolnai Tamás
Export as highlighting or shading. Highlighting is the default. Change-Id: Ib2a38e4cd0d49317288bb565383b690d4222c8ed
2015-03-19loplugin:constantfunction: unotoolsNoel Grandin
Change-Id: Icf543973dc9edde270016c8af7e4c08c57eff650
2015-03-11do not call virtual Commit() from dtors of utl::ConfigItem subclassesMichael Stahl
~SvtAppFilterOptions_Impl() demonstrates that this is a bad idea, by not invoking its subclasses' ImplCommit() but its own. Change-Id: Ic675e9cd3be1494c740a4f289be3bdbf0d1a122b
2015-03-11utl::ConfigItem::Commit() should call ClearModified()Michael Stahl
Rename the virtual function, and add a new non-virtual Commit() to do that. Change-Id: I09421df781ba965d6ff638b46cd8214fb3a00022