From c5483edae01e4145c80439c5070e0bf2fad51186 Mon Sep 17 00:00:00 2001 From: Stephan Bergmann Date: Fri, 7 Sep 2018 10:48:41 +0200 Subject: Drop workaround for old MSVC that no longer appears necessary with MSVC 2017 The comments talk about unspecific "compilation and/or linking errors" with MSVC 2008 and 2013, but building with latest MSVC 2017 at least works fine now, so assume the workaround is no longer needed. (Theoretically, external code being compiled with older MSVC could run into problems now, but lets take that risk of becoming slightly build-time incompatible here.) Change-Id: I9e20be355c765c3a881502773963a61305eb00f3 Reviewed-on: https://gerrit.libreoffice.org/60135 Tested-by: Jenkins Reviewed-by: Stephan Bergmann --- include/salhelper/simplereferenceobject.hxx | 22 ++++++---------------- 1 file changed, 6 insertions(+), 16 deletions(-) (limited to 'include') diff --git a/include/salhelper/simplereferenceobject.hxx b/include/salhelper/simplereferenceobject.hxx index d8f86f52d00a..2745f0d848fd 100644 --- a/include/salhelper/simplereferenceobject.hxx +++ b/include/salhelper/simplereferenceobject.hxx @@ -51,8 +51,8 @@ namespace salhelper { The same problem as with operators new and delete would also be there with operators new[] and delete[]. But since arrays of reference-counted - objects are of no use, anyway, it seems best to simply declare and not - define (private) operators new[] and delete[]. + objects are of no use, anyway, it seems best to simply + define operators new[] and delete[] as deleted. */ class SALHELPER_DLLPUBLIC SimpleReferenceObject { @@ -102,23 +102,13 @@ private: */ void operator =(SimpleReferenceObject) SAL_DELETED_FUNCTION; - /// @cond INTERNAL - -#ifdef _MSC_VER -/* We can't now have these private with MSVC2008 at least, it leads to - compilation errors in xmloff and other places. -*/ -protected: -#endif - /** not implemented (see general class documentation) + /** see general class documentation */ - static void * operator new[](std::size_t); + static void * operator new[](std::size_t) SAL_DELETED_FUNCTION; - /** not implemented (see general class documentation) + /** see general class documentation */ - static void operator delete[](void * pPtr); - - /// @endcond + static void operator delete[](void * pPtr) SAL_DELETED_FUNCTION; }; } -- cgit