Fix syntax error, add file changed/saved indicator

Makefile

@@ -14,11 +14,7 @@ all:
 
 dev: all
 
-build:
+run: all
-	@mkdir -p build
-	./cmake.sh Tyro
-
-run: build
 ifneq ($(OS),Darwin)
 	./$(PROGRAM)
 else
@@ -27,11 +23,11 @@ endif
 
 
 ifeq ($(OS),Darwin)
-run-app:
+run-app: all
 	open -a $(PWD)/build/Tyro.app
 endif
 
-run-grind:
+run-grind: all
 	valgrind $(PROGRAM)
 
 # Make optimized and striped executable
@@ -80,8 +76,6 @@ run-tests: tests
 tests: $(TEST_RUNNER)
 	./build/test_runner
 	
-
-	
 tests-verbose: $(TEST_RUNNER)
 	./build/test_runner -s
 

src/widgets/EditPane.cpp

@@ -194,7 +194,20 @@ bool EditPane::SaveFile(const wxString &filename)
 	if (this->FileWritable())
 	{
 		this->SetSavePoint();
-		return wxStyledTextCtrl::SaveFile(filename);
+		bool saved = wxStyledTextCtrl::SaveFile(filename);
+
+		if (saved)
+		{
+			auto parent = (wxAuiNotebook*) this->GetParent();
+			auto currentPage = parent->GetCurrentPage();
+			auto idx = parent->GetPageIndex(currentPage);
+			wxString currentTitle = parent->GetPageText(idx);
+
+			currentTitle.Replace("*", "");
+			parent->SetPageText(idx, currentTitle);
+		}
+
+		return saved;
 	}
 
 	return false;
@@ -270,6 +283,25 @@ void EditPane::BindEvents()
 		}
 	}, wxID_ANY);
 
+	// On modification, update parent tab to show "dirtyness"
+	this->Bind(wxEVT_STC_MODIFIED, [=](wxStyledTextEvent& event) {
+		auto parent = (wxAuiNotebook*) this->GetParent();
+		auto currentPage = parent->GetCurrentPage();
+		auto idx = parent->GetPageIndex(currentPage);
+		wxString currentTitle = parent->GetPageText(idx);
+
+		if (this->IsModified() && ! currentTitle.Contains("*"))
+		{
+			parent->SetPageText(idx, currentTitle + "*");
+		}
+
+		if (currentTitle.Contains("*") && ! this->IsModified())
+		{
+			currentTitle.Replace("*", "");
+			parent->SetPageText(idx, currentTitle);
+		}
+	}, wxID_ANY);
+
 	// this->Bind(wxEVT_STC_CHARADDED, &EditPane::OnCharAdded, this, wxID_ANY);
 }
 
@@ -405,19 +437,19 @@ void EditPane::_ApplyTheme(JsonValue &lexer_map)
 		}
 
 		// Set bold, if it applies
-		if  this->theme_config->GetThemeValue("bold", key).isBool())
+		if (this->theme_config->GetThemeValue("bold", key).isBool())
 		{
 			this->StyleSetBold(i, this->theme_config->GetThemeValue("bold", key).asBool());
 		}
 		
 		// Italic
-		if  this->theme_config->GetThemeValue("italic", key).isBool())
+		if (this->theme_config->GetThemeValue("italic", key).isBool())
 		{
 			this->StyleSetItalic(i, this->theme_config->GetThemeValue("italic", key).asBool());
 		}
 		
 		// Underline
-		if  this->theme_config->GetThemeValue("underline", key).isBool())
+		if (this->theme_config->GetThemeValue("underline", key).isBool())
 		{
 			this->StyleSetUnderline(i, this->theme_config->GetThemeValue("underline", key).asBool());
 		}

src/widgets/TyroMenu.cpp

@@ -1,20 +1,17 @@
 #include "src/widgets/TyroMenu.h"
-#include "src/settings/LangConfig.h"
-
-static LangConfig *lang_config = nullptr;
 
 /**
  * Constructor
  */ 
 TyroMenu::TyroMenu()
 {
-	fileMenu = new wxMenu();
+	this->fileMenu = new wxMenu();
-	editMenu = new wxMenu();
+	this->editMenu = new wxMenu();
-	viewMenu = new wxMenu();
+	this->viewMenu = new wxMenu();
-	langMenu = new wxMenu();
+	this->langMenu = new wxMenu();
-	helpMenu = new wxMenu();
+	this->helpMenu = new wxMenu();
 
-	lang_config = new LangConfig();
+	this->lang_config = new LangConfig();
 
 	this->SetupMainMenus();
 	this->SetupLangMenu();
@@ -33,7 +30,7 @@ TyroMenu::TyroMenu()
 TyroMenu::~TyroMenu()
 {
 	wxLogDebug("TyroMenu Destructor Called.");
-	delete lang_config;
+	delete this->lang_config;
 }
 
 /**
@@ -44,41 +41,38 @@ TyroMenu::~TyroMenu()
 void TyroMenu::SetupMainMenus()
 {
 	// Add items to top-level menus
-	fileMenu->Append(wxID_NEW, "&New\tCtrl+N", "Create a new file");
+	this->fileMenu->Append(wxID_NEW, "&New\tCtrl+N", "Create a new file");
-	fileMenu->AppendSeparator();
+	this->fileMenu->AppendSeparator();
-	fileMenu->Append(wxID_OPEN, "&Open\tCtrl+O", "Opens an existing file");
+	this->fileMenu->Append(wxID_OPEN, "&Open\tCtrl+O", "Opens an existing file");
-	fileMenu->Append(myID_OPEN_DIR, "&Open Dir\tShift+Ctrl+O", "Opens the selected folder in the sidebar");
+	this->fileMenu->Append(myID_OPEN_DIR, "&Open Dir\tShift+Ctrl+O", "Opens the selected folder in the sidebar");
+	this->fileMenu->AppendSeparator();
+	this->fileMenu->Append(wxID_SAVE, "&Save\tCtrl+S", "Save the content");
+	this->fileMenu->Append(wxID_SAVEAS, "Save &As...\tShift+Ctrl+S", "Save current file as...");
+	this->fileMenu->AppendSeparator();
+	this->fileMenu->Append(wxID_CLOSE, "&Close\tCtrl+W", "Close the current document");
+	this->fileMenu->Append(myID_CLOSE_ALL, "C&lose All\tShift+Ctrl+W", "Close all open documents.");
+	this->fileMenu->Append(wxID_EXIT, "&Quit\tCtrl+Q", "Quit the application");
 	
-	fileMenu->Append(wxID_SAVE, "&Save\tCtrl+S", "Save the content");
+	this->editMenu->Append(wxID_UNDO, "&Undo\tCtrl+Z", "Undo last action");
-	fileMenu->Append(wxID_SAVEAS, "Save &As...\tShift+Ctrl+S", "Save current file as...");
+	this->editMenu->Append(wxID_REDO, "&Redo\tCtrl+Y", "Redo last action");
-	fileMenu->AppendSeparator();
+	this->editMenu->AppendSeparator();
-	fileMenu->Append(wxID_CLOSE, "&Close\tCtrl+W", "Close the current document");
+	this->editMenu->Append(wxID_CUT, "Cu&t\tCtrl+X", "Cut selected text");
-	fileMenu->Append(myID_CLOSE_ALL, "C&lose All\tShift+Ctrl+W", "Close all open documents.");
+	this->editMenu->Append(wxID_COPY, "&Copy\tCtrl+C", "Copy selected text");
-	fileMenu->Append(wxID_EXIT, "&Quit\tCtrl+Q", "Quit the application");
+	this->editMenu->Append(wxID_PASTE, "&Paste\tCtrl+V", "Paste contents of clipboard");
-	
-	editMenu->Append(wxID_UNDO, "&Undo\tCtrl+Z", "Undo last action");
-	editMenu->Append(wxID_REDO, "&Redo\tCtrl+Y", "Redo last action");
-	editMenu->AppendSeparator();
-	editMenu->Append(wxID_CUT, "Cu&t\tCtrl+X", "Cut selected text");
-	editMenu->Append(wxID_COPY, "&Copy\tCtrl+C", "Copy selected text");
-	editMenu->Append(wxID_PASTE, "&Paste\tCtrl+V", "Paste contents of clipboard");
 	//editMenu->Append(wxID_DELETE, "&Delete\tDel");
-	editMenu->AppendSeparator();
+	this->editMenu->AppendSeparator();
-	editMenu->Append(wxID_FIND, "&Find\tCtrl+F");
+	this->editMenu->Append(wxID_FIND, "&Find\tCtrl+F");
-	editMenu->Append(wxID_REPLACE, "&Replace\tCtrl+R");
+	this->editMenu->Append(wxID_REPLACE, "&Replace\tCtrl+R");
+	this->editMenu->AppendSeparator();
+	this->editMenu->Append(wxID_SELECTALL, "Select All\tCtrl+A", "Select all the text in the current document");
+	this->editMenu->AppendSeparator();
+	this->editMenu->Append(wxID_PREFERENCES, "&Preferences\tCtrl+P");
 
-	editMenu->AppendSeparator();
+	this->viewMenu->AppendCheckItem(myID_VIEW_WHITESPACE, "Show Invisible Characters\tCtrl+Shift+I", "Toggle visibility of white space characters");
-	editMenu->Append(wxID_SELECTALL, "Select All\tCtrl+A", "Select all the text in the current document");
+	this->viewMenu->AppendCheckItem(myID_VIEW_LINE_ENDINGS, "Show line endings", "Toggle visibility of line ending characters");
-#ifndef TRAVIS
+	this->viewMenu->AppendCheckItem(myID_LINE_WRAP, "Word Wrap", "Toggle wrapping of long lines");
-	editMenu->AppendSeparator();
-	editMenu->Append(wxID_PREFERENCES, "&Preferences\tCtrl+P");
-#endif
 	
-	viewMenu->AppendCheckItem(myID_VIEW_WHITESPACE, "Show Invisible Characters\tCtrl+Shift+I", "Toggle visibility of white space characters");
+	this->helpMenu->Append(wxID_ABOUT, "&About...\tF1", "Show info about this application");
-	viewMenu->AppendCheckItem(myID_VIEW_LINE_ENDINGS, "Show line endings", "Toggle visibility of line ending characters");
-	viewMenu->AppendCheckItem(myID_LINE_WRAP, "Word Wrap", "Toggle wrapping of long lines");
-	
-	helpMenu->Append(wxID_ABOUT, "&About...\tF1", "Show info about this application");
 }
 
 /**

src/widgets/TyroMenu.h

@@ -4,6 +4,7 @@
 #pragma once
 
 #include "src/widgets/widget.h"
+#include "src/settings/LangConfig.h"
 
 class TyroMenu : public wxMenuBar {
 public:
@@ -13,6 +14,7 @@ public:
 	void SetIdChecked(int id, bool checked);
 	void SetCurrentLanguage(string lang);
 private:
+	LangConfig *lang_config = nullptr;
 	wxMenu *fileMenu = nullptr;
 	wxMenu *editMenu = nullptr;
 	wxMenu *viewMenu = nullptr;