Make toggling of word wrap for the log window nicer for linux. Unfortunately this method is only implemented for wxGTK (what not wxMSW? Shocker!).

Made it so that Source/Core/DolphinWX/Globals.h is only used for the wxWidgets build, and a little other file inclusion clean up.
Fix an issue where a user runs a game from the open menu or toolbar items with a clean user directory, and henceforth double clicking on the empty game list (where it says double click to browse for games), that game is run again.


git-svn-id: https://dolphin-emu.googlecode.com/svn/trunk@6037 8ced0084-cf51-0410-be5f-012b33b47a6e

Source/Core/DebuggerUICommon/Src/DebuggerUIUtil.h

@@ -20,17 +20,6 @@
 
 #include <wx/wx.h>
 
-#define wxUSE_XPM_IN_MSW 1
-#define USE_XPM_BITMAPS 1
-
-// Defined in CodeWindow.cpp
 extern wxFont DebuggerFont;
 
-// define this to use XPMs everywhere (by default, BMPs are used under Win)
-// BMPs use less space, but aren't compiled into the executable on other platforms
-
-#if USE_XPM_BITMAPS && defined (__WXMSW__) && !wxUSE_XPM_IN_MSW
-#error You need to enable XPM support to use XPM bitmaps with toolbar!
-#endif // USE_XPM_BITMAPS
-
 #endif

Source/Core/DolphinWX/Src/BootManager.cpp

@@ -35,7 +35,6 @@
 #include <string>
 #include <vector>
 
-#include "Globals.h"
 #include "Common.h"
 #include "IniFile.h"
 #include "BootManager.h"
@@ -46,6 +45,7 @@
 #include "SysConf.h"
 #include "Core.h"
 #if defined(HAVE_WX) && HAVE_WX
+	#include "Globals.h"
 	#include "ConfigMain.h"
 	#include "Frame.h"
 	#include "CodeWindow.h"

Source/Core/DolphinWX/Src/Frame.cpp

@@ -717,6 +717,8 @@ void CFrame::OnGameListCtrl_ItemActivated(wxListEvent& WXUNUSED (event))
 
 		m_GameListCtrl->Update();
 	}
+	else if (!m_GameListCtrl->GetGameNames().size())
+		m_GameListCtrl->BrowseForDirectory();
 	else
 		// Game started by double click
 		BootGame(std::string(""));

Source/Core/DolphinWX/Src/FrameAui.cpp

@@ -789,8 +789,6 @@ void CFrame::ReloadPanes()
 	// Open notebook pages
 	AddRemoveBlankPage();
 	g_pCodeWindow->OpenPages();
-	if (g_pCodeWindow->bShowOnStart[0]) ToggleLogWindow(true);
-	if (g_pCodeWindow->bShowOnStart[1]) ToggleConsole(true);
 }
 
 void CFrame::DoLoadPerspective()

Source/Core/DolphinWX/Src/Globals.h

@@ -16,19 +16,14 @@
 // http://code.google.com/p/dolphin-emu/
 
 
-
 // This file holds global data for DolphinWx and DebuggerWx
 
 
-
 #ifndef _GLOBALS_H
 #define _GLOBALS_H
 
 #include "Common.h"
 
-// Constant Colors
-const unsigned long COLOR_GRAY = 0xDCDCDC;
-
 enum
 {
 	Toolbar_DebugGo,
@@ -248,49 +243,31 @@ enum
 	LIST_CTRL = 1000
 };
 
-#define wxUSE_XPM_IN_MSW 1
-#define USE_XPM_BITMAPS 1
+#include <wx/wx.h>
+#include <wx/toolbar.h>
+#include <wx/log.h>
+#include <wx/image.h>
+#include <wx/aboutdlg.h>
+#include <wx/filedlg.h>
+#include <wx/spinctrl.h>
+#include <wx/srchctrl.h>
+#include <wx/listctrl.h>
+#include <wx/progdlg.h>
+#include <wx/imagpng.h>
+#include <wx/button.h>
+#include <wx/stattext.h>
+#include <wx/choice.h>
+#include <wx/cmdline.h>
+#include <wx/busyinfo.h>
 
-// For compilers that support precompilation, includes <wx/wx.h>.
-//#include <wx/wxprec.h>
-
-//#ifndef WX_PRECOMP
-#if defined(HAVE_WX) && HAVE_WX
-	#include <wx/wx.h>
-	//#endif
-
-	#include <wx/toolbar.h>
-	#include <wx/log.h>
-	#include <wx/image.h>
-	#include <wx/aboutdlg.h>
-	#include <wx/filedlg.h>
-	#include <wx/spinctrl.h>
-	#include <wx/srchctrl.h>
-	#include <wx/listctrl.h>
-	#include <wx/progdlg.h>
-	#include <wx/imagpng.h>
-	#include <wx/button.h>
-	#include <wx/stattext.h>
-	#include <wx/choice.h>
-	#include <wx/cmdline.h>
-	#include <wx/busyinfo.h>
-
-	// Define this to use XPMs everywhere (by default, BMPs are used under Win)
-	// BMPs use less space, but aren't compiled into the executable on other platforms
-	#if USE_XPM_BITMAPS && defined (__WXMSW__) && !wxUSE_XPM_IN_MSW
-	#error You need to enable XPM support to use XPM bitmaps with toolbar!
-	#endif // USE_XPM_BITMAPS
-
-	// custom message macro
-	#define EVT_HOST_COMMAND(id, fn) \
-		DECLARE_EVENT_TABLE_ENTRY(\
+// custom message macro
+#define EVT_HOST_COMMAND(id, fn) \
+	DECLARE_EVENT_TABLE_ENTRY(\
 			wxEVT_HOST_COMMAND, id, wxID_ANY, \
 			(wxObjectEventFunction)(wxEventFunction) wxStaticCastEvent(wxCommandEventFunction, &fn), \
 			(wxObject*) NULL \
 			),
 
-	extern const wxEventType wxEVT_HOST_COMMAND;
+extern const wxEventType wxEVT_HOST_COMMAND;
 
-#endif // HAVE_WX
 #endif // _GLOBALS_H
-

Source/Core/DolphinWX/Src/ISOFile.h

@@ -21,6 +21,10 @@
 #include "Volume.h"
 #include "VolumeCreator.h"
 
+#if defined(HAVE_WX) && HAVE_WX
+#include <wx/image.h>
+#endif
+
 class PointerWrap;
 class GameListItem
 {

Source/Core/DolphinWX/Src/LogWindow.cpp

@@ -15,20 +15,12 @@
 // Official SVN repository and contact information can be found at
 // http://code.google.com/p/dolphin-emu/
 
-
-#include <wx/wx.h>
-#include <wx/button.h>
-#include <wx/textctrl.h>
-#include <wx/listbox.h>
-#include <wx/checklst.h>
-#include <wx/strconv.h>
-
-#include "Core.h" // for Core::GetState()
 #include "LogWindow.h"
 #include "ConsoleListener.h"
 #include "Console.h"
+#include "IniFile.h"
 #include "FileUtil.h"
-
+#include "DebuggerUIUtil.h"
 
 // Milliseconds between msgQueue flushes to wxTextCtrl
 #define UPDATETIME 200
@@ -71,6 +63,9 @@ CLogWindow::CLogWindow(CFrame *parent, wxWindowID id, const wxPoint& pos,
 	CreateGUIControls();
 
 	LoadSettings();
+
+	m_LogTimer = new wxTimer(this, IDTM_UPDATELOG);
+	m_LogTimer->Start(UPDATETIME);
 }
 
 void CLogWindow::CreateGUIControls()
@@ -91,23 +86,27 @@ void CLogWindow::CreateGUIControls()
 	m_verbosity->SetFont(wxFont(7, wxFONTFAMILY_DEFAULT, wxFONTSTYLE_NORMAL, wxFONTWEIGHT_NORMAL));
 
 	// Font
-	m_FontChoice = new wxChoice(this, IDM_FONT, wxDefaultPosition, wxDefaultSize, 0, NULL, 0, wxDefaultValidator);
+	m_FontChoice = new wxChoice(this, IDM_FONT,
+		   	wxDefaultPosition, wxDefaultSize, 0, NULL, 0, wxDefaultValidator);
 
 	m_FontChoice->Append(wxT("Default font"));
 	m_FontChoice->Append(wxT("Monospaced font"));
 	m_FontChoice->Append(wxT("Selected font"));
 	m_FontChoice->SetSelection(0);
 	DefaultFont = GetFont();
-	MonoSpaceFont.SetNativeFontInfoUserDesc(wxString::FromAscii("lucida console windows-1252"));
-	Font.push_back(DefaultFont);
-	Font.push_back(MonoSpaceFont);
-	Font.push_back(DebuggerFont);
+	MonoSpaceFont.SetNativeFontInfoUserDesc(wxT("lucida console windows-1252"));
+	LogFont.push_back(DefaultFont);
+	LogFont.push_back(MonoSpaceFont);
+	LogFont.push_back(DebuggerFont);
 
 	// Options
 	wxCheckBox * m_WrapLine = new wxCheckBox(this, IDM_WRAPLINE, wxT("Word Wrap"));
-	m_writeFileCB = new wxCheckBox(this, IDM_WRITEFILE, wxT("Write to File"), wxDefaultPosition, wxDefaultSize, 0);
-	m_writeConsoleCB = new wxCheckBox(this, IDM_WRITECONSOLE, wxT("Write to Console"), wxDefaultPosition, wxDefaultSize, 0);
-	m_writeWindowCB = new wxCheckBox(this, IDM_WRITEWINDOW, wxT("Write to Window ->"), wxDefaultPosition, wxDefaultSize, 0);
+	m_writeFileCB = new wxCheckBox(this, IDM_WRITEFILE,
+		   	wxT("Write to File"), wxDefaultPosition, wxDefaultSize, 0);
+	m_writeConsoleCB = new wxCheckBox(this, IDM_WRITECONSOLE,
+		   	wxT("Write to Console"), wxDefaultPosition, wxDefaultSize, 0);
+	m_writeWindowCB = new wxCheckBox(this, IDM_WRITEWINDOW,
+		   	wxT("Write to Window ->"), wxDefaultPosition, wxDefaultSize, 0);
 
 	m_checks = new wxCheckListBox(this, IDM_LOGCHECKS, wxDefaultPosition, wxDefaultSize);
 
@@ -115,7 +114,6 @@ void CLogWindow::CreateGUIControls()
 	m_Log = CreateTextCtrl(this, IDM_LOG, wxTE_RICH | wxTE_MULTILINE | wxTE_READONLY | wxTE_DONTWRAP);
 	m_cmdline = new wxTextCtrl(this, IDM_SUBMITCMD, wxEmptyString, wxDefaultPosition, wxDefaultSize,
 		wxTE_PROCESS_ENTER | wxTE_PROCESS_TAB);
-	//m_cmdline->SetFont(DebuggerFont);
 
 	// Sizers
 	sUber = new wxBoxSizer(wxHORIZONTAL);			// whole plane
@@ -126,8 +124,10 @@ void CLogWindow::CreateGUIControls()
 	wxStaticBoxSizer* sbLeftOptions = new wxStaticBoxSizer(wxVERTICAL, this, wxT("Options"));
 
 	wxBoxSizer* sLogCtrl = new wxBoxSizer(wxHORIZONTAL);
-	sLogCtrl->Add(new wxButton(this, IDM_TOGGLEALL, wxT("Toggle all"), wxDefaultPosition, wxDefaultSize, wxBU_EXACTFIT), 1);
-	sLogCtrl->Add(new wxButton(this, IDM_CLEARLOG, wxT("Clear"), wxDefaultPosition, wxDefaultSize, wxBU_EXACTFIT), 1);
+	sLogCtrl->Add(new wxButton(this, IDM_TOGGLEALL, wxT("Toggle all"),
+			   	wxDefaultPosition, wxDefaultSize, wxBU_EXACTFIT), 1);
+	sLogCtrl->Add(new wxButton(this, IDM_CLEARLOG, wxT("Clear"),
+			   	wxDefaultPosition, wxDefaultSize, wxBU_EXACTFIT), 1);
 
 	sbLeftOptions->Add(m_FontChoice, 0, (wxDOWN), 1);
 	sbLeftOptions->Add(m_WrapLine, 0, (wxDOWN), 1);
@@ -146,13 +146,7 @@ void CLogWindow::CreateGUIControls()
 	sUber->Add(sRight, 1, wxEXPAND);
 	this->SetSizer(sUber);
 
-	// Settings
-	//	SetAffirmativeId(IDM_SUBMITCMD);
-	UpdateChecks();
 	m_cmdline->SetFocus();
-
-	m_LogTimer = new wxTimer(this, IDTM_UPDATELOG);
-	m_LogTimer->Start(UPDATETIME);
 }
 
 CLogWindow::~CLogWindow()
@@ -220,8 +214,8 @@ void CLogWindow::LoadSettings()
 	m_verbosity->SetSelection(verbosity - 1);
 	ini.Get("Options", "Font", &font, 0);
 	m_FontChoice->SetSelection(font);
-	if (m_FontChoice->GetSelection() < (int)Font.size())
-		m_Log->SetDefaultStyle(wxTextAttr(wxNullColour, wxNullColour, Font[m_FontChoice->GetSelection()]));
+	if (m_FontChoice->GetSelection() < (int)LogFont.size())
+		m_Log->SetDefaultStyle(wxTextAttr(wxNullColour, wxNullColour, LogFont[m_FontChoice->GetSelection()]));
 	ini.Get("Options", "WriteToFile", &m_writeFile, true);
 	m_writeFileCB->SetValue(m_writeFile);
 	ini.Get("Options", "WriteToConsole", &m_writeConsole, true);
@@ -257,7 +251,6 @@ void CLogWindow::OnSubmit(wxCommandEvent& WXUNUSED (event))
 	if (!m_cmdline) return;
 	Console_Submit(m_cmdline->GetValue().To8BitData());
 	m_cmdline->SetValue(wxEmptyString);
-	NotifyUpdate();
 }
 
 void CLogWindow::OnClear(wxCommandEvent& WXUNUSED (event))
@@ -271,8 +264,6 @@ void CLogWindow::OnClear(wxCommandEvent& WXUNUSED (event))
 	m_LogSection.Leave();
 
 	m_console->ClearScreen();
-	NOTICE_LOG(CONSOLE, "Console cleared");
-	NotifyUpdate();
 }
 
 // Enable or disable all boxes for the current verbosity level and save the changes.
@@ -336,8 +327,8 @@ wxTextCtrl* CLogWindow::CreateTextCtrl(wxPanel* parent, wxWindowID id, long Styl
 	TC->SetBackgroundColour(*wxBLACK);
 	if (m_FontChoice)
 	{
-		if (m_FontChoice->GetSelection() < (int)Font.size())
-			TC->SetDefaultStyle(wxTextAttr(wxNullColour, wxNullColour, Font[m_FontChoice->GetSelection()]));
+		if (m_FontChoice->GetSelection() < (int)LogFont.size())
+			TC->SetDefaultStyle(wxTextAttr(wxNullColour, wxNullColour, LogFont[m_FontChoice->GetSelection()]));
 	}
 	return TC;
 }
@@ -361,31 +352,36 @@ void CLogWindow::OnOptionsCheck(wxCommandEvent& event)
 		break;
 
 	case IDM_WRAPLINE:
-		// SetWindowStyleFlag doesn't fully work, we need to redraw the window
-		//m_Log->SetWindowStyleFlag(m_Log->GetWindowStyleFlag() ^ wxTE_DONTWRAP);
-		/* Notice:	To retain the colors when changing word wrapping we need to
-					loop through every letter with GetStyle and then reapply them letter by letter */
+#ifdef __WXGTK__
+		m_Log->SetWindowStyleFlag(m_Log->GetWindowStyleFlag() ^ (wxTE_WORDWRAP | wxTE_DONTWRAP));
+#else
+		// Unfortunately wrapping styles can only be changed dynamically with wxGTK
+		// Notice:	To retain the colors when changing word wrapping we need to
+		//			loop through every letter with GetStyle and then reapply them letter by letter
 		// Prevent m_Log access while it's being destroyed
 		m_LogAccess = false;
 		UnPopulateRight();
 		Text = m_Log->GetValue();
 		m_Log->Destroy();
-		switch (event.IsChecked())
-		{
-			case 0: m_Log = CreateTextCtrl(this, IDM_LOG, wxTE_RICH | wxTE_MULTILINE | wxTE_READONLY | wxTE_DONTWRAP); break;
-			case 1: m_Log = CreateTextCtrl(this, IDM_LOG, wxTE_RICH | wxTE_MULTILINE | wxTE_READONLY | wxTE_WORDWRAP); break;
-		}
+		if (event.IsChecked())
+			m_Log = CreateTextCtrl(this, IDM_LOG,
+					wxTE_RICH | wxTE_MULTILINE | wxTE_READONLY | wxTE_WORDWRAP);
+		else
+			m_Log = CreateTextCtrl(this, IDM_LOG,
+					wxTE_RICH | wxTE_MULTILINE | wxTE_READONLY | wxTE_DONTWRAP);
 		m_Log->SetDefaultStyle(wxTextAttr(*wxWHITE));
 		m_Log->AppendText(Text);
 		PopulateRight();
 		m_LogAccess = true;
+#endif
 		break;
 
 	case IDM_FONT:
 		// Update selected font
-		Font[Font.size()-1] = DebuggerFont;
-		m_Log->SetStyle(0, m_Log->GetLastPosition(), wxTextAttr(wxNullColour, wxNullColour, Font[event.GetSelection()]));
-		m_Log->SetDefaultStyle(wxTextAttr(wxNullColour, wxNullColour, Font[event.GetSelection()]));
+		LogFont[LogFont.size()-1] = DebuggerFont;
+		m_Log->SetStyle(0, m_Log->GetLastPosition(),
+			   	wxTextAttr(wxNullColour, wxNullColour, LogFont[event.GetSelection()]));
+		m_Log->SetDefaultStyle(wxTextAttr(wxNullColour, wxNullColour, LogFont[event.GetSelection()]));
 		break;
 
 	case IDM_WRITEFILE:
@@ -471,21 +467,13 @@ void CLogWindow::OnLogTimer(wxTimerEvent& WXUNUSED(event))
 {
 	if (!m_LogAccess) return;
 
-	//m_Log->Freeze();
 	UpdateLog();
-	// Better auto scroll than wxTE_AUTO_SCROLL
+	// Scroll to the last line
 	if (msgQueue.size() > 0)
 	{
 		m_Log->ScrollLines(1);
 		m_Log->ShowPosition( m_Log->GetLastPosition() );
 	}
-	//m_Log->Thaw();
-}
-
-void CLogWindow::NotifyUpdate()
-{
-	UpdateChecks();
-	//UpdateLog();
 }
 
 void CLogWindow::UpdateLog()
@@ -494,11 +482,8 @@ void CLogWindow::UpdateLog()
 	if (!m_Log) return;
 
 	m_LogTimer->Stop();
-	wxString collected_text;
 
 	m_LogSection.Enter();
-	// Rough estimate
-	collected_text.reserve(100 * msgQueue.size());
 	int msgQueueSize = (int)msgQueue.size();
 	for (int i = 0; i < msgQueueSize; i++)
 	{
@@ -536,13 +521,10 @@ void CLogWindow::UpdateLog()
 			// White timestamp
 			m_Log->SetStyle(j, j + 9, wxTextAttr(*wxWHITE));
 		}
-		collected_text.Append(msgQueue.front().second);
 		msgQueue.pop();
 	}
 	m_LogSection.Leave();
-	// Write all text at once, needs multiple SetStyle, may not be better
-	//if (collected_text.size()) m_Log->AppendText(collected_text);
-	// Return in 0.2 seconds
+
 	m_LogTimer->Start(UPDATETIME);
 }
 

Source/Core/DolphinWX/Src/LogWindow.h

@@ -17,12 +17,9 @@
 
 #ifndef LOGWINDOW_H_
 #define LOGWINDOW_H_
-#include "Main.h" // for wxGetApp
+
 #include "LogManager.h"
 #include "Frame.h"
-#include "DebuggerUIUtil.h"
-
-#include "IniFile.h" // Common
 #include "Thread.h"
 #include <queue>
 
@@ -60,7 +57,6 @@ public:
 		const wxString& name = wxT("Log")
 		);
 	~CLogWindow();
-	void NotifyUpdate();
 
 	void SaveSettings();
 	void LoadSettings();
@@ -71,7 +67,7 @@ public:
 private:
 	CFrame *Parent;
 	wxFont DefaultFont, MonoSpaceFont;
-	std::vector<wxFont> Font;
+	std::vector<wxFont> LogFont;
 	wxTimer *m_LogTimer;
 	FileLogListener *m_fileLog;
 	ConsoleListener *m_console;

Source/Core/DolphinWX/Src/Main.cpp

@@ -37,7 +37,6 @@
 #include "Main.h"
 #include "ConfigManager.h"
 #include "CodeWindow.h"
-#include "LogWindow.h"
 #include "JitWindow.h"
 #include "ExtendedTrace.h"
 #include "BootManager.h"

Source/Core/DolphinWX/Src/MainNoGUI.cpp

@@ -37,9 +37,7 @@
 #endif
 
 #include "Core.h"
-#include "Globals.h"
 #include "Host.h"
-#include "ISOFile.h"
 #include "CPUDetect.h"
 #include "cmdline.h"
 #include "Thread.h"