[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]

[vidalia-svn] r1977: Add a virtual method to ConfigPage that ConfigDialog can cal (in trunk: . src/gui/config)



Author: edmanm
Date: 2007-10-09 23:37:04 -0400 (Tue, 09 Oct 2007)
New Revision: 1977

Modified:
   trunk/
   trunk/src/gui/config/configpage.h
Log:
 r2009@lysithea:  edmanm | 2007-10-09 23:00:09 -0400
 Add a virtual method to ConfigPage that ConfigDialog can call for each
 subclassed page to revert any failed settings on that page. Also tweak a
 couple comments.



Property changes on: trunk
___________________________________________________________________
 svk:merge ticket from /local/vidalia/trunk [r2009] on dc66be73-d13e-47ba-a267-8dc7cda68c65

Modified: trunk/src/gui/config/configpage.h
===================================================================
--- trunk/src/gui/config/configpage.h	2007-10-10 03:36:57 UTC (rev 1976)
+++ trunk/src/gui/config/configpage.h	2007-10-10 03:37:04 UTC (rev 1977)
@@ -47,13 +47,13 @@
    * and return true if everything was saved successfully. */
   virtual bool save(QString &errmsg) = 0;
 
-  /** Subclassed pages should overload this method to return true if they
+  /** Subclassed pages can overload this method to return true if they
    * contain settings that have been modified since they were last applied to
    * Tor. The default implementation always returns false. */
   virtual bool changedSinceLastApply() {
     return false;
   }
-  /** Subclassed pages should overload this method to apply any settings to
+  /** Subclassed pages can overload this method to apply any settings to
    * Tor that have been modified since they were last applied (e.g., the
    * changes were made while Tor was not running). Returns true if the changes
    * were applied successfully. */
@@ -61,6 +61,9 @@
     Q_UNUSED(errmsg);
     return true;
   }
+  /** Subclassed pages can overload this method to revert any cancelled
+   * settings. */
+  virtual void revert() {}
 
 private:
   QString _title; /**< Title of this configuration page. */