bool wxLaunchDefaultBrowser(const wxString& url)
{
+ bool success = true;
+
wxString finalurl = url;
//if it isn't a full url, try appending http:// to it
wxRegKey keyDDE(key, wxT("DDEExec"));
if ( keyDDE.Exists() )
{
- wxString ddeTopic = wxRegKey(keyDDE, wxT("topic"));
+ wxRegKey keyTopic(keyDDE, wxT("topic"));
+ wxString ddeTopic = keyTopic.QueryDefaultValue();
// we only know the syntax of WWW_OpenURL DDE request
if ( ddeTopic == wxT("WWW_OpenURL") )
{
- wxString ddeCmd = keyDDE;
+ wxString ddeCmd = keyDDE.QueryDefaultValue();
// this is a bit naive but should work as -1 can't appear
// elsewhere in the DDE topic, normally
(LPShellExecute) ::GetProcAddress(hShellDll,
wxString::Format(wxT("ShellExecute%s"),
-#ifdef __WXUNICODE__
+#if wxUSE_UNICODE
wxT("W")
#else
wxT("A")
// Windows sometimes doesn't open the browser correctly when using mime
// types, so do ShellExecute - i.e. start <url> (from James Carroll)
nResult = (int) (*lpShellExecute)(NULL, NULL, finalurl.c_str(),
- NULL, wxT(""), SW_SHOWNORMAL);
+ NULL, wxT(""), SW_SHOWNORMAL);
// Unload Shell32.dll
::FreeLibrary(hShellDll);
#else
if ( cmd.empty() || wxExecute(cmd + wxT(" ") + finalurl) == -1)
return false;
}
-
+
#else // !wxUSE_MIMETYPE && !(WXMSW && wxUSE_NATIVE_CONFIG)
- return false;
+ success = false;
#endif
//success - hopefully
- return true;
+ return success;
}
// ----------------------------------------------------------------------------
#endif // !wxGTK
#endif // wxUSE_GUI
-