GUI change:

- methods OnInitDialog, OnApply & OnClose of CDlgBase now return true if successful. return of false prevents a dialog from being loaded or left respectively;
- massive code cleaning considering the 'virtual' attribute of overridden methods;
- also fixes #1476 (Don't close "Create new account" window if user not set account name)

1 files changed, 3 insertions, 3 deletions

diff --git a/protocols/Steam/src/steam_options.h b/protocols/Steam/src/steam_options.h
index d0aaf03bd9..a71e6ae2fa 100644
--- a/protocols/Steam/src/steam_options.h
+++ b/protocols/Steam/src/steam_options.h
@@ -16,8 +16,8 @@ private:
 protected:

 	CSteamOptionsMain(CSteamProto *proto, int idDialog, HWND hwndParent = NULL);

 

-	void OnInitDialog();

-	void OnApply();

+	bool OnInitDialog() override;

+	bool OnApply() override;

 

 public:

 	static CDlgBase *CreateAccountManagerPage(void *param, HWND owner)

@@ -42,7 +42,7 @@ private:
 	CCtrlButton m_add;

 

 protected:

-	void OnInitDialog();

+	bool OnInitDialog() override;

 	void OnBlock(CCtrlButton*);

 

 public: