summaryrefslogtreecommitdiffstats
path: root/embedserv
Commit message (Collapse)AuthorAgeFilesLines
* Remove excess newlinesChris Sherlock2016-02-0912-34/+0
| | | | | | | | | | | | | | | | | | | A ridiculously fast way of doing this is: for i in $(pcregrep -l -M -r --include='.*[hc]xx$' \ --exclude-dir=workdir --exclude-dir=instdir '^ {3,}' .) do perl -0777 -i -pe 's/^ {3,}/ /gm' $i done Change-Id: Iebb93eccbee9e4fc5c4380474ba595858a27ac2c Reviewed-on: https://gerrit.libreoffice.org/22224 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Chris Sherlock <chris.sherlock79@gmail.com>
* Silence -Werror,-Wmissing-field-initializers in END_OBJECT_MAP (clang-cl)Stephan Bergmann2016-01-261-1/+7
| | | | Change-Id: Ia9ba7fc6cd94e2d726cd9b1dc72c140af16dd700
* Silence more warnings, clang-cl's /W4 now also includes -WextraStephan Bergmann2016-01-262-0/+2
| | | | Change-Id: I20fa3b02570bdbc3ccd297401a7669b0fda5c624
* more Windows build fixesNoel Grandin2016-01-251-1/+1
| | | | | | after my OInterfaceContainer2 change Change-Id: Ic365cf09631ff58c7d1b54795e7024923ebe02d1
* more Windows build fixesNoel Grandin2016-01-251-1/+1
| | | | | | after my OInterfaceContainer2 change Change-Id: Idc8d9aa218fbeea0e6b00216bb12b0e50efea57c
* more Windows build fixesNoel Grandin2016-01-251-1/+1
| | | | | | after my OInterfaceContainer2 change Change-Id: I8dabb07088c03795d6ac1989ad5ef318d011ccff
* more Windows build fixesNoel Grandin2016-01-251-1/+1
| | | | | | after my OInterfaceContainer2 change Change-Id: I2d59fd166672a2d0b6c05e5bf9368c8fa5cf111c
* more Windows build fixesNoel Grandin2016-01-251-0/+1
| | | | | | after my OInterfaceContainer2 change Change-Id: I869126a90e67bd00e953182ccafa6569009316ea
* more Windows build fixesNoel Grandin2016-01-252-2/+2
| | | | Change-Id: I0b42df81c2848fb7d8a4722a984caaaa458da78f
* InterfaceContainer2 with vector instead of SequenceNoel Grandin2016-01-253-4/+4
| | | | | | | | | | create an InterfaceContainer2 class to replace InterfaceContainer. It uses a std::vector instead of a Sequence for the mutable listener list, which provides far better performance. Switch all our internal use-sites to the new class. Change-Id: I6b56cfa511ded2395faa22e68fab3b2f16c3cb88
* cppcheck: noExplicitConstructorCaolán McNamara2016-01-075-9/+11
| | | | | | | | Change-Id: Ifdcb1cc12645d6cdba81d16f94f0268696187cde Reviewed-on: https://gerrit.libreoffice.org/21201 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
* Resolve Linker Warning LNK4104Mike Kaganski2015-12-082-5/+6
| | | | | | | | | | | | | See https://msdn.microsoft.com/en-us/library/e821azx1.aspx Some export symbols should be PRIVATE Also, don't export DLLMain. It is internal function used by CRT, and not intended to be exported. Change-Id: I48139b985aa2ee499c2005d8b0d966a2cc6dcd0c Reviewed-on: https://gerrit.libreoffice.org/20419 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Matúš Kukan <matus@libreoffice.org>
* -Werror,-Wint-to-pointer-castStephan Bergmann2015-12-022-6/+6
| | | | | | | | | | Appart from the by-design type-punned use in CreateWindowEx, m_uID is (only) used in a call to MAKEWPARAM (to create a WPARAM to pass into a SendMessage call), so the most realistic type to use seems to be WORD. And CHatchWindow::Init is only called with a value of 2000 (from docholder.cxx), anyway. Change-Id: I4b6554d1ca9bb3926378c9e25a5473609f5951c5
* Silence clang-cl warnings in ATL headersStephan Bergmann2015-12-022-0/+24
| | | | Change-Id: Id94279d0086bc29569783f4e0b5d975be162e823
* -Werror,-Wunused-variableStephan Bergmann2015-11-281-1/+0
| | | | Change-Id: I6800b97cfbf2e986c341d8893658f511951bc8f0
* -Werror,-Wunused-variableStephan Bergmann2015-11-241-10/+0
| | | | Change-Id: If674f0a3199a43cec84eb8cfb3b3c1d334a288ca
* Use Get/SetWindowLongPtr, for 64-bit buildStephan Bergmann2015-11-241-2/+2
| | | | Change-Id: I02e5ca8dad87c24f8bf6e192a6b37ffc82a5a04a
* -Werror,-Wbitwise-op-parenthesesStephan Bergmann2015-11-241-1/+1
| | | | Change-Id: I36bd91dc2bb77a09f59e865527db1e9c6724a652
* Expand single use of SendCommand macroStephan Bergmann2015-11-241-6/+4
| | | | Change-Id: I8af259f55e8fc318ccdbc69bba00ae0d7d3aacdb
* -Werror,-WreorderStephan Bergmann2015-11-241-1/+1
| | | | Change-Id: I8350b7e60e54eb0f0be5fa33a4ca4de40551618c
* -Werror,-Wlogical-op-parenthesesStephan Bergmann2015-11-241-1/+1
| | | | Change-Id: I39f59a88c34c37bad520dd44f3e8fcaf4a016eee
* -Werror,-WreorderStephan Bergmann2015-11-241-1/+1
| | | | Change-Id: I71e3d8878a8c7190ef239e4529d23eccaadc80d6
* -Werror,-Wint-to-pointer-castStephan Bergmann2015-11-241-2/+2
| | | | Change-Id: I6db7d7595fda972e7c5d5c24d235678671478975
* Fix css.awt.XSystemDependentMenuPeer.getMenuHandleStephan Bergmann2015-11-181-1/+1
| | | | | | | | | | | | | | | | ...to return an ANY containing a 64-bit HYPER instead of a 32-bit LONG for SYSTEM_WIN32 (which covers both 32-bit and 64-bit versions of Windows), so that the HMENU value (which is effectively a void*) is not truncated for the 64-bit build. This should effectively be URE ABI compatible for the 32-bit build, as there the HYPER value will fit into 32 bits, so extracting a LONG from the ANY should still work. Change-Id: Ief3de5924f672e8f6bbe6df08c15439456ca036f Reviewed-on: https://gerrit.libreoffice.org/20045 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
* The #else branch is unused nowStephan Bergmann2015-11-181-4/+0
| | | | | | | | | | | | | | ...after a3ce60eb3c8bb5c69ab41556398efa8316286477 "We only support MSVC 2008 (_MSC_VER 1500) or later" removed the && (_MSC_VER > 1310) part from the #if Change-Id: I472461b29a37fcf996262c0a83cb10f4cb8443a9 Reviewed-on: https://gerrit.libreoffice.org/20042 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
* -Werror,-Wlogical-op-parenthesesStephan Bergmann2015-11-181-2/+3
| | | | Change-Id: I8e22b6399bbdb07c376299d5a23c8061af18578d
* -Werror,-Wduplicate-decl-specifier (extern "C" already in STDAPI)Stephan Bergmann2015-11-181-2/+2
| | | | Change-Id: I634debc05a24e0d519200fc91a77d50f36b9af38
* -Wmissing-braces is globally disabled in com_GCC_defs.mk anywayStephan Bergmann2015-11-181-4/+0
| | | | Change-Id: Ie2a1e6785ac925f268e63775f2282df3f3c6bcd4
* use initialiser for Sequence<OUString>Noel Grandin2015-11-151-2/+1
| | | | | | | | | | | | | replaced using: git grep -lP 'Sequence.*OUString.*\(\s*1\s*\)' | xargs perl -0777 -pi -e "s/Sequence<\s*OUString\s*> (\w+)\(\s*1\s*\); .*\[0\] = (\S+);/Sequence<OUString> \1 { \2 };/g" Change-Id: I20ad0489da887a9712982531c3b127339bb8b3b9 Reviewed-on: https://gerrit.libreoffice.org/19969 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* no need to use OUString constructor in call to createInstanceNoel Grandin2015-11-011-3/+1
| | | | | | | Change-Id: Iaf3d83ba1490cb1d97a5bd4d1f7cd6943d4a7296 Reviewed-on: https://gerrit.libreoffice.org/19704 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* use uno::Reference::set method instead of assignmentNoel Grandin2015-10-312-15/+15
| | | | | | | Change-Id: I150f213ab033f5fc1b50abe4e1529292206c9feb Reviewed-on: https://gerrit.libreoffice.org/19689 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* Replace "SAL_OVERRIDE" with "override" in LIBO_INTERNAL_ONLY codeStephan Bergmann2015-10-121-3/+3
| | | | Change-Id: I2ea407acd763ef2d7dae2d3b8f32525523ac8274
* com::sun::star->css in embedserv/Noel Grandin2015-10-017-182/+137
| | | | | | | Change-Id: Iaf1e30ae7dc9710a16a031ea623d531e9e65a6a5 Reviewed-on: https://gerrit.libreoffice.org/19025 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
* cppcheck: noExplicitConstructorCaolán McNamara2015-09-131-1/+1
| | | | Change-Id: I39194062ba68c4cb1ccc9b93c629f005ccd02497
* cppcheck: fix some redundantCondition warningsJulien Nabet2015-09-051-1/+1
| | | | | | | Change-Id: If53d33fb4d9ca9dc1babb9043bd07c5d29fe74c2 Reviewed-on: https://gerrit.libreoffice.org/18353 Reviewed-by: Julien Nabet <serval2412@yahoo.fr> Tested-by: Julien Nabet <serval2412@yahoo.fr>
* tdf#88206 replace cppu::WeakImplHelper* etc.Takeshi Abe2015-08-053-6/+6
| | | | | | | | | with the variadic variants in embedserv and eventattacher. Change-Id: I556ad208a7ffcf85c484e298cdbb1324b8a6a659 Reviewed-on: https://gerrit.libreoffice.org/17499 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
* remove obsolete check for pre-MSVC2013 ATL library nameMichael Stahl2015-07-091-1/+1
| | | | | | | Change-Id: I52bba4d4c0558e6b87530e4577f4b1ef458b7012 Reviewed-on: https://gerrit.libreoffice.org/16888 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: David Ostrovsky <david@ostrovsky.org>
* Revert "return and use std::vector from OInterfaceContainerHelper"Noel Grandin2015-06-291-5/+7
| | | | This reverts commit e57314f61f67b093510c5a8a8f34a62126ba8734.
* return and use std::vector from OInterfaceContainerHelperNoel Grandin2015-06-291-7/+5
| | | | | | | | | | since most of the time we don’t need a heavyweight uno::Sequence. Adds a new method getElementsAsVector(). Change-Id: I9e72bef0c0c723ffd0dd7d4152db5baec6784a7a Reviewed-on: https://gerrit.libreoffice.org/15747 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
* remove unnecessary check for null when calling deleteNoel Grandin2015-06-152-7/+3
| | | | | | | | 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
* css::uno::Reference can automatically up-cast nowStephan Bergmann2015-05-041-1/+1
| | | | | | | follow-up to 8568c7226cfb732e17fa42b365b9fce3a8264691 "tdf#66232: Cannot open files by COM2" Change-Id: I5a0858dd6f48136f9f07b0d04c5634a6173dd154
* tdf#66232: Cannot open files by COM2Mike Kaganski2015-05-031-4/+6
| | | | | | | | | | | | | | | | | | | | | As suggested by comment 13 of the bug, the problem is caused by commit b248624126c271c88381d3dad6e04fc954f65779 by Noel Grandin. 1. DocumentHolder::DocumentFrame(): XDesktop2 cannot be directly queried for XFrame2. And in this case, simple XFrame is enough. 2. DocumentHolder::CloseFrame(): m_xFrame->dispose() violates an assert because of NULL reference. Reverted to previous code. 3. Following commit debde4fdc443f522562ee29def4c27512d64609a by Andras Timar (supposed to fix the bug), found two places where previously used to be UNO_QUERY, but now UNO_QUERY_THROW is used. These places are reverted to use UNO_QUERY. Change-Id: Ifb9e74367d5dd99ba12166abdd443b29a8018aeb Reviewed-on: https://gerrit.libreoffice.org/15607 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Andras Timar <andras.timar@collabora.com>
* remove unnecessary use of void in function declarationsNoel Grandin2015-04-1511-38/+38
| | | | | | | | | | | | | | | | 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
* Fix various Windows-only XServiceInfo implementationsStephan Bergmann2015-03-242-63/+22
| | | | | | ...to match what is recorded in the .component files Change-Id: Ibc5bb0575f3869317e4d14614cf1dad0af728ddc
* gbuild: set _WIN32_WINNT globally instead of WINVERMichael Stahl2015-03-051-1/+0
| | | | | | | | | | ... and remove the definitions in makefiles, except in fpicker, which defines a different value for the Vista file picker. The WINVER value is derived automatically from _WIN32_WINNT by SDK header sdkddkver.h. Change-Id: I73358a9bf5d070f71821654cae2a29335a754a21
* Typo: trough->throughJulien Nabet2015-02-211-1/+1
| | | | Change-Id: I5af784709df88492695d1ac9c9a5b020e909f362
* boost::unordered_map->std::unordered_mapCaolán McNamara2015-01-041-3/+3
| | | | Change-Id: I5d458f43616edc395faa8c27edaddc7d515166db
* cppuhelper: clean up public headers with include-what-you-useMichael Stahl2014-11-181-0/+1
| | | | Change-Id: I41ba46831f24b2960a1fe982b74a2b623e682e0b
* sal: clean up public headers with include-what-you-useMichael Stahl2014-11-171-0/+1
| | | | | | | | | 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
* Fix common typos. No automatic tools. Handmade…Andrea Gelmini2014-11-121-1/+1
| | | | | | | Change-Id: I1ab4e23b0539f8d39974787f226e57a21f96e959 Reviewed-on: https://gerrit.libreoffice.org/12164 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>