diff options
author | David Ostrovsky <david@ostrovsky.org> | 2015-09-06 21:33:15 +0200 |
---|---|---|
committer | Michael Stahl <mstahl@redhat.com> | 2015-09-09 14:55:06 +0000 |
commit | 98483599b76fe6d0113a657f057ff5ad799ade96 (patch) | |
tree | e41729cc04b852bbb1bac9762669bc9af991fa9b /sw/inc/IMark.hxx | |
parent | 1544d1257582af96bee633e2ce1ab2b39b4de7d1 (diff) |
sw: Help vs 2015 with virtual base inheritance
MSVC 14.0 was failing to compile IMark class an friends with:
error C2280: 'SwModify &SwModify::operator =(const SwModify &)':
attempting to reference a deleted function: [1].
After defining copy ctor and assignment operator in SwModify and
derived classes as deleted the next failure was:
error C2512: 'sw::mark::IMark::IMark': no appropriate default
constructor available: [2].
Defining default ctor as default in IMark and derived classes
seems to make all compilers happy.
[1] http://paste.openstack.org/show/447854
[2] http://paste.openstack.org/show/447856
Change-Id: I2822e8ec4b98ca18d0a0128ed2ceeb38f347c5bf
Reviewed-on: https://gerrit.libreoffice.org/18362
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Michael Stahl <mstahl@redhat.com>
Diffstat (limited to 'sw/inc/IMark.hxx')
-rw-r--r-- | sw/inc/IMark.hxx | 24 |
1 files changed, 24 insertions, 0 deletions
diff --git a/sw/inc/IMark.hxx b/sw/inc/IMark.hxx index 8d7bf0605a12..56850f8601c8 100644 --- a/sw/inc/IMark.hxx +++ b/sw/inc/IMark.hxx @@ -36,6 +36,9 @@ namespace sw { namespace mark : virtual public SwModify // inherited as interface , public ::boost::totally_ordered<IMark> { + protected: + IMark() = default; + public: //getters virtual const SwPosition& GetMarkPos() const =0; @@ -67,21 +70,33 @@ namespace sw { namespace mark virtual OUString ToString( ) const =0; virtual void dumpAsXml(struct _xmlTextWriter* pWriter) const = 0; + private: + IMark(IMark&) = delete; + IMark &operator =(IMark const&) = delete; }; class SW_DLLPUBLIC IBookmark : virtual public IMark { + protected: + IBookmark() = default; + public: virtual const OUString& GetShortName() const =0; virtual const vcl::KeyCode& GetKeyCode() const =0; virtual void SetShortName(const OUString&) =0; virtual void SetKeyCode(const vcl::KeyCode&) =0; + private: + IBookmark(IBookmark&) = delete; + IBookmark &operator =(IBookmark const&) = delete; }; class SW_DLLPUBLIC IFieldmark : virtual public IMark { + protected: + IFieldmark() = default; + public: typedef ::std::map< OUString, ::com::sun::star::uno::Any> parameter_map_t; //getters @@ -94,14 +109,23 @@ namespace sw { namespace mark virtual void SetFieldname(const OUString& rFieldname) =0; virtual void SetFieldHelptext(const OUString& rFieldHelptext) =0; virtual void Invalidate() = 0; + private: + IFieldmark(IFieldmark&) = delete; + IFieldmark &operator =(IFieldmark const&) = delete; }; class SW_DLLPUBLIC ICheckboxFieldmark : virtual public IFieldmark { + protected: + ICheckboxFieldmark() = default; + public: virtual bool IsChecked() const =0; virtual void SetChecked(bool checked) =0; + private: + ICheckboxFieldmark(ICheckboxFieldmark&) = delete; + ICheckboxFieldmark &operator =(ICheckboxFieldmark const&) = delete; }; // Apple llvm-g++ 4.2.1 with _GLIBCXX_DEBUG won't eat boost::bind for this |