diff options
author | Mike Kaganski <mike.kaganski@collabora.com> | 2019-05-14 19:15:59 +0200 |
---|---|---|
committer | Mike Kaganski <mike.kaganski@collabora.com> | 2019-05-15 07:49:40 +0200 |
commit | 51fb20cda19dfd0f26228d10ba1296ceca4e3601 (patch) | |
tree | 4c9a81234eb15957020b24183940d8e0d2199d79 /shell | |
parent | 31405b20fed5f3e98be25501a25e25963f3102f9 (diff) |
Use proper VARIANT_BOOL values: VARIANT_TRUE is actually -1
Who was that idiot who initially used wrong values??? Hmm... oh damn,
that was me :-(
Change-Id: I3bfde7511bcf8adfa38ae68372f21511f21efca1
Reviewed-on: https://gerrit.libreoffice.org/72308
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
Tested-by: Mike Kaganski <mike.kaganski@collabora.com>
(cherry picked from commit 31cc28ca7c02f28b64e9d757c9fc17e2a81ba352)
Reviewed-on: https://gerrit.libreoffice.org/72332
Diffstat (limited to 'shell')
-rw-r--r-- | shell/source/win32/spsupp/COMOpenDocuments.cxx | 13 |
1 files changed, 10 insertions, 3 deletions
diff --git a/shell/source/win32/spsupp/COMOpenDocuments.cxx b/shell/source/win32/spsupp/COMOpenDocuments.cxx index e731ceccaadc..93f6a17aaa8b 100644 --- a/shell/source/win32/spsupp/COMOpenDocuments.cxx +++ b/shell/source/win32/spsupp/COMOpenDocuments.cxx @@ -16,6 +16,9 @@ #include <spsuppServ.hpp> #include <stdio.h> +namespace +{ + // Display confirmation dialog, return false on negative answer bool SecurityWarning(const wchar_t* sProgram, const wchar_t* sDocument) { @@ -76,6 +79,10 @@ HRESULT LOStart(const wchar_t* sModeArg, const wchar_t* sFilePath, bool bDoSecur return S_OK; } +VARIANT_BOOL toVBool(bool b) { return b ? VARIANT_TRUE : VARIANT_FALSE; } + +} // namespace + // IObjectSafety methods void COMOpenDocuments::COMObjectSafety::SetMaskedOptions(DWORD iMask, DWORD iOptions) @@ -309,7 +316,7 @@ STDMETHODIMP COMOpenDocuments::CreateNewDocument2( { // TODO: resolve the program from varProgID (nullptr -> default?) HRESULT hr = LOStart(L"-n", bstrTemplateLocation, m_aObjectSafety.GetSafe_forUntrustedCaller() || m_aObjectSafety.GetSafe_forUntrustedData()); - *pbResult = VARIANT_BOOL(SUCCEEDED(hr)); + *pbResult = toVBool(SUCCEEDED(hr)); return hr; } @@ -360,7 +367,7 @@ STDMETHODIMP COMOpenDocuments::ViewDocument3( { // TODO: resolve the program from varProgID (nullptr -> default?) HRESULT hr = LOStart(L"--view", bstrDocumentLocation, m_aObjectSafety.GetSafe_forUntrustedCaller() || m_aObjectSafety.GetSafe_forUntrustedData()); - *pbResult = VARIANT_BOOL(SUCCEEDED(hr)); + *pbResult = toVBool(SUCCEEDED(hr)); return hr; } @@ -422,7 +429,7 @@ STDMETHODIMP COMOpenDocuments::EditDocument3( { // TODO: resolve the program from varProgID (nullptr -> default?) HRESULT hr = LOStart(L"-o", bstrDocumentLocation, m_aObjectSafety.GetSafe_forUntrustedCaller() || m_aObjectSafety.GetSafe_forUntrustedData()); - *pbResult = VARIANT_BOOL(SUCCEEDED(hr)); + *pbResult = toVBool(SUCCEEDED(hr)); return hr; } |