Amebis/WinStd
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

COM: Throw in constructors and operators

Browse Source 
Signed-off-by: Simon Rozman <simon@rozman.si>
This commit is contained in:
Simon Rozman 2023-03-06 16:04:24 +01:00
parent 12046216e9
commit fc0060d3bd
1 changed files with 19 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

29
include/WinStd/COM.h
View File

@ -93,7 +93,9 @@ namespace winstd
com_obj(_In_ _Other *other) com_obj(_In_ _Other *other)
{ {
assert(other); assert(other);
other->QueryInterface(__uuidof(T), (void**)&m_h); HRESULT hr = other->QueryInterface(__uuidof(T), (void**)&m_h);
if (FAILED(hr))
throw com_runtime_error(hr, "QueryInterface failed");
} }
/// ///
@ -104,7 +106,9 @@ namespace winstd
template <class _Other> template <class _Other>
com_obj(_In_ com_obj<_Other> &other) com_obj(_In_ com_obj<_Other> &other)
{ {
other->QueryInterface(__uuidof(T), (void**)&m_h); HRESULT hr = other->QueryInterface(__uuidof(T), (void**)&m_h);
if (FAILED(hr))
throw com_runtime_error(hr, "QueryInterface failed");
} }
/// ///
@ -183,26 +187,32 @@ namespace winstd
/// ///
/// Constructs BSTR from OLE string /// Constructs BSTR from OLE string
/// ///
bstr(_In_ LPCOLESTR src) noexcept bstr(_In_ LPCOLESTR src)
{ {
m_h = SysAllocString(src); m_h = SysAllocString(src);
if (!m_h)
throw std::bad_alloc();
} }
/// ///
/// Constructs BSTR from OLE string with length /// Constructs BSTR from OLE string with length
/// ///
bstr(_In_ LPCOLESTR src, _In_ UINT len) noexcept bstr(_In_ LPCOLESTR src, _In_ UINT len)
{ {
m_h = SysAllocStringLen(src, len); m_h = SysAllocStringLen(src, len);
if (!m_h)
throw std::bad_alloc();
} }
/// ///
/// Constructs BSTR from std::basic_string /// Constructs BSTR from std::basic_string
/// ///
template<class _Traits, class _Ax> template<class _Traits, class _Ax>
bstr(_In_ const std::basic_string<wchar_t, _Traits, _Ax> &src) noexcept bstr(_In_ const std::basic_string<wchar_t, _Traits, _Ax> &src)
{ {
m_h = SysAllocStringLen(src.c_str(), (UINT)src.length()); m_h = SysAllocStringLen(src.c_str(), (UINT)src.length());
if (!m_h)
throw std::bad_alloc();
} }
/// ///
@ -880,7 +890,7 @@ namespace winstd
/// ///
/// Copy from SAFEARRAY /// Copy from SAFEARRAY
/// ///
variant& operator=(_In_ const SAFEARRAY *pSrc) noexcept variant& operator=(_In_ const SAFEARRAY *pSrc)
{ {
assert(pSrc != NULL); assert(pSrc != NULL);
VariantClear(this); VariantClear(this);
@ -891,10 +901,9 @@ namespace winstd
SafeArrayGetVartype(const_cast<LPSAFEARRAY>(pSrc), &vt); SafeArrayGetVartype(const_cast<LPSAFEARRAY>(pSrc), &vt);
vt |= VT_ARRAY; vt |= VT_ARRAY;
parray = pCopy; parray = pCopy;
} else return *this;
assert(0); }
throw com_runtime_error(hr, "SafeArrayCopy failed");
return *this;
} }
public: public: