summaryrefslogtreecommitdiff
path: root/embedserv
AgeCommit message (Collapse)AuthorFilesLines
2016-04-21Fix typosAndrea Gelmini1-1/+1
Change-Id: Ic4fbc8e7220eb4641f67ef4f85d4853608a369f4 Reviewed-on: https://gerrit.libreoffice.org/24257 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Eike Rathke <erack@redhat.com> Tested-by: Eike Rathke <erack@redhat.com>
2016-04-12cleanup: remove unused com/sun/star/uno includesJochen Nitschke1-1/+0
Sequence.h(xx), Any.h(xx) and Type.h(xx) and remove unused using-declarations from these files. Add a few missing includes provided by them. Change-Id: I6b91b6d1fdf9d0496dd546c0aab9bdcc6831a5d4 Reviewed-on: https://gerrit.libreoffice.org/23805 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-03-04-Werror,-WshadowStephan Bergmann1-3/+3
In the second part, looks odd that those two lines are a perfect copy of what is already done near the start of the function, but I have no insight at all into that code, so just leave it at that. Change-Id: I6b1d973f77a3d9389880ddec500968144ba615f2
2016-03-04Silence -Werror,-Wnon-virtual-dtor in external includesStephan Bergmann2-0/+2
Change-Id: I2ef42eca934588b219de52dfc7c373395118cc32
2016-03-04-Werror,-WshadowStephan Bergmann1-1/+1
Change-Id: Ie6a35ce1fece5de31f455d88ab3e12010b8047f5
2016-03-04-Werror,-Wunused-macrosStephan Bergmann1-1/+0
According to <https://msdn.microsoft.com/en-us/library/windows/desktop/aa383681%28v=vs.85%29.aspx> ("Enabling STRICT"), it is used in combination with including windows.h, so maybe this #define STRICT (which was there ever since a934115b3ea90d7883681a6ca8922a4b859be36f "Initial revision," w/o also including windows.h) was always unnecessary (as reported by clang-cl). Change-Id: I02cf2584fe2fd52e9513791b18fdf6e7982e89b3
2016-03-04Silence -Werror,-Wnon-virtual-dtorStephan Bergmann1-1/+3
Change-Id: I46f7b68aded50abc0015b458507adbf84d0ffe76
2016-03-04-Werror,-WshadowStephan Bergmann1-2/+1
Change-Id: I40f92daa60a08d93ea7bbe309fd16fa7ef27e78b
2016-02-09Remove excess newlinesChris Sherlock12-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>
2016-01-26Silence -Werror,-Wmissing-field-initializers in END_OBJECT_MAP (clang-cl)Stephan Bergmann1-1/+7
Change-Id: Ia9ba7fc6cd94e2d726cd9b1dc72c140af16dd700
2016-01-26Silence more warnings, clang-cl's /W4 now also includes -WextraStephan Bergmann2-0/+2
Change-Id: I20fa3b02570bdbc3ccd297401a7669b0fda5c624
2016-01-25more Windows build fixesNoel Grandin1-1/+1
after my OInterfaceContainer2 change Change-Id: Ic365cf09631ff58c7d1b54795e7024923ebe02d1
2016-01-25more Windows build fixesNoel Grandin1-1/+1
after my OInterfaceContainer2 change Change-Id: Idc8d9aa218fbeea0e6b00216bb12b0e50efea57c
2016-01-25more Windows build fixesNoel Grandin1-1/+1
after my OInterfaceContainer2 change Change-Id: I8dabb07088c03795d6ac1989ad5ef318d011ccff
2016-01-25more Windows build fixesNoel Grandin1-1/+1
after my OInterfaceContainer2 change Change-Id: I2d59fd166672a2d0b6c05e5bf9368c8fa5cf111c
2016-01-25more Windows build fixesNoel Grandin1-0/+1
after my OInterfaceContainer2 change Change-Id: I869126a90e67bd00e953182ccafa6569009316ea
2016-01-25more Windows build fixesNoel Grandin2-2/+2
Change-Id: I0b42df81c2848fb7d8a4722a984caaaa458da78f
2016-01-25InterfaceContainer2 with vector instead of SequenceNoel Grandin3-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
2016-01-07cppcheck: noExplicitConstructorCaolán McNamara5-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>
2015-12-08Resolve Linker Warning LNK4104Mike Kaganski2-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>
2015-12-02-Werror,-Wint-to-pointer-castStephan Bergmann2-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
2015-12-02Silence clang-cl warnings in ATL headersStephan Bergmann2-0/+24
Change-Id: Id94279d0086bc29569783f4e0b5d975be162e823
2015-11-28-Werror,-Wunused-variableStephan Bergmann1-1/+0
Change-Id: I6800b97cfbf2e986c341d8893658f511951bc8f0
2015-11-24-Werror,-Wunused-variableStephan Bergmann1-10/+0
Change-Id: If674f0a3199a43cec84eb8cfb3b3c1d334a288ca
2015-11-24Use Get/SetWindowLongPtr, for 64-bit buildStephan Bergmann1-2/+2
Change-Id: I02e5ca8dad87c24f8bf6e192a6b37ffc82a5a04a
2015-11-24-Werror,-Wbitwise-op-parenthesesStephan Bergmann1-1/+1
Change-Id: I36bd91dc2bb77a09f59e865527db1e9c6724a652
2015-11-24Expand single use of SendCommand macroStephan Bergmann1-6/+4
Change-Id: I8af259f55e8fc318ccdbc69bba00ae0d7d3aacdb
2015-11-24-Werror,-WreorderStephan Bergmann1-1/+1
Change-Id: I8350b7e60e54eb0f0be5fa33a4ca4de40551618c
2015-11-24-Werror,-Wlogical-op-parenthesesStephan Bergmann1-1/+1
Change-Id: I39f59a88c34c37bad520dd44f3e8fcaf4a016eee
2015-11-24-Werror,-WreorderStephan Bergmann1-1/+1
Change-Id: I71e3d8878a8c7190ef239e4529d23eccaadc80d6
2015-11-24-Werror,-Wint-to-pointer-castStephan Bergmann1-2/+2
Change-Id: I6db7d7595fda972e7c5d5c24d235678671478975
2015-11-18Fix css.awt.XSystemDependentMenuPeer.getMenuHandleStephan Bergmann1-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>
2015-11-18The #else branch is unused nowStephan Bergmann1-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>
2015-11-18-Werror,-Wlogical-op-parenthesesStephan Bergmann1-2/+3
Change-Id: I8e22b6399bbdb07c376299d5a23c8061af18578d
2015-11-18-Werror,-Wduplicate-decl-specifier (extern "C" already in STDAPI)Stephan Bergmann1-2/+2
Change-Id: I634debc05a24e0d519200fc91a77d50f36b9af38
2015-11-18-Wmissing-braces is globally disabled in com_GCC_defs.mk anywayStephan Bergmann1-4/+0
Change-Id: Ie2a1e6785ac925f268e63775f2282df3f3c6bcd4
2015-11-15use initialiser for Sequence<OUString>Noel Grandin1-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>
2015-11-01no need to use OUString constructor in call to createInstanceNoel Grandin1-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>
2015-10-31use uno::Reference::set method instead of assignmentNoel Grandin2-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>
2015-10-12Replace "SAL_OVERRIDE" with "override" in LIBO_INTERNAL_ONLY codeStephan Bergmann1-3/+3
Change-Id: I2ea407acd763ef2d7dae2d3b8f32525523ac8274
2015-10-01com::sun::star->css in embedserv/Noel Grandin7-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>
2015-09-13cppcheck: noExplicitConstructorCaolán McNamara1-1/+1
Change-Id: I39194062ba68c4cb1ccc9b93c629f005ccd02497
2015-09-05cppcheck: fix some redundantCondition warningsJulien Nabet1-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>
2015-08-05tdf#88206 replace cppu::WeakImplHelper* etc.Takeshi Abe3-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>
2015-07-09remove obsolete check for pre-MSVC2013 ATL library nameMichael Stahl1-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>
2015-06-29Revert "return and use std::vector from OInterfaceContainerHelper"Noel Grandin1-5/+7
This reverts commit e57314f61f67b093510c5a8a8f34a62126ba8734.
2015-06-29return and use std::vector from OInterfaceContainerHelperNoel Grandin1-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>
2015-06-15remove unnecessary check for null when calling deleteNoel Grandin2-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
2015-05-04css::uno::Reference can automatically up-cast nowStephan Bergmann1-1/+1
follow-up to 8568c7226cfb732e17fa42b365b9fce3a8264691 "tdf#66232: Cannot open files by COM2" Change-Id: I5a0858dd6f48136f9f07b0d04c5634a6173dd154
2015-05-03tdf#66232: Cannot open files by COM2Mike Kaganski1-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>