From 9884b0b4ea44b54f45a2ee6d45771fb63a61d2ee Mon Sep 17 00:00:00 2001 From: Vadim Zeitlin Date: Sat, 21 Oct 2017 23:35:57 +0200 Subject: [PATCH] Remove redundant check for m_webView in wxOSX code wxCHECK_MSG() is enough, there is really no need to check for the same condition twice, this is just confusing. --- src/osx/webview_webkit.mm | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/src/osx/webview_webkit.mm b/src/osx/webview_webkit.mm index 2e7fb12f14..ea75162a9d 100644 --- a/src/osx/webview_webkit.mm +++ b/src/osx/webview_webkit.mm @@ -411,12 +411,8 @@ wxString wxWebViewWebKit::GetSelectedText() const bool wxWebViewWebKit::RunScript(const wxString& javascript, wxString* output) { - if ( !m_webView ) - { - wxCHECK_MSG( m_webView, false, - wxS("wxWebView must be created before running JS scripts") ); - return false; - } + wxCHECK_MSG( m_webView, false, + wxS("wxWebView must be created before calling RunScript()") ); wxJSScriptWrapper wrapJS(javascript, &m_runScriptCount);