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

Remove redundant check for m_webView in wxOSX code

Browse Source 
wxCHECK_MSG() is enough, there is really no need to check for the same
condition twice, this is just confusing.
This commit is contained in:
Vadim Zeitlin
2017-10-21 23:35:57 +02:00
parent 74e60b39f8
commit 9884b0b4ea
1 changed files with 2 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
src/osx/webview_webkit.mm
View File

@@ -411,12 +411,8 @@ wxString wxWebViewWebKit::GetSelectedText() const
bool wxWebViewWebKit::RunScript(const wxString& javascript, wxString* output) bool wxWebViewWebKit::RunScript(const wxString& javascript, wxString* output)
{ {
if ( !m_webView )
{
wxCHECK_MSG( m_webView, false, wxCHECK_MSG( m_webView, false,
wxS("wxWebView must be created before running JS scripts") ); wxS("wxWebView must be created before calling RunScript()") );
return false;
}
wxJSScriptWrapper wrapJS(javascript, &m_runScriptCount); wxJSScriptWrapper wrapJS(javascript, &m_runScriptCount);