[haiku-commits] r37094 - haiku/trunk/src/servers/net

  • From: axeld@xxxxxxxxxxxxxxxx
  • To: haiku-commits@xxxxxxxxxxxxx
  • Date: Fri, 11 Jun 2010 09:29:16 +0200 (CEST)

Author: axeld
Date: 2010-06-11 09:29:16 +0200 (Fri, 11 Jun 2010)
New Revision: 37094
Changeset: http://dev.haiku-os.org/changeset/37094/haiku

Modified:
   haiku/trunk/src/servers/net/Settings.cpp
   haiku/trunk/src/servers/net/Settings.h
Log:
* Cleanup.


Modified: haiku/trunk/src/servers/net/Settings.cpp
===================================================================
--- haiku/trunk/src/servers/net/Settings.cpp    2010-06-11 02:36:51 UTC (rev 
37093)
+++ haiku/trunk/src/servers/net/Settings.cpp    2010-06-11 07:29:16 UTC (rev 
37094)
@@ -1,5 +1,5 @@
 /*
- * Copyright 2006-2009, Haiku, Inc. All Rights Reserved.
+ * Copyright 2006-2010, Haiku, Inc. All Rights Reserved.
  * Distributed under the terms of the MIT License.
  *
  * Authors:
@@ -42,7 +42,7 @@
 
 const static settings_template kInterfaceTemplate[] = {
        {B_STRING_TYPE, "device", NULL, true},
-       {B_BOOL_TYPE,   "disabled", NULL},
+       {B_BOOL_TYPE, "disabled", NULL},
        {B_MESSAGE_TYPE, "address", kInterfaceAddressTemplate},
        {B_INT32_TYPE, "flags", NULL},
        {B_INT32_TYPE, "metric", NULL},
@@ -87,8 +87,6 @@
 
 
 Settings::Settings()
-       :
-       fUpdated(false)
 {
        _Load();
 }
@@ -96,15 +94,6 @@
 
 Settings::~Settings()
 {
-       // only save the settings if something has changed
-       if (!fUpdated)
-               return;
-
-#if 0
-       BFile file;
-       if (_Open(&file, B_CREATE_FILE | B_WRITE_ONLY) != B_OK)
-               return;
-#endif
 }
 
 
@@ -260,8 +249,10 @@
                return B_ENTRY_NOT_FOUND;
 
        const driver_settings* settings = get_driver_settings(handle);
-       if (settings != NULL)
-               status = _ConvertFromDriverSettings(*settings, 
settingsTemplate, message);
+       if (settings != NULL) {
+               status = _ConvertFromDriverSettings(*settings, settingsTemplate,
+                       message);
+       }
 
        unload_driver_settings(handle);
        return status;

Modified: haiku/trunk/src/servers/net/Settings.h
===================================================================
--- haiku/trunk/src/servers/net/Settings.h      2010-06-11 02:36:51 UTC (rev 
37093)
+++ haiku/trunk/src/servers/net/Settings.h      2010-06-11 07:29:16 UTC (rev 
37094)
@@ -1,5 +1,5 @@
 /*
- * Copyright 2006-2007, Haiku, Inc. All Rights Reserved.
+ * Copyright 2006-2010, Haiku, Inc. All Rights Reserved.
  * Distributed under the terms of the MIT License.
  *
  * Authors:
@@ -19,48 +19,63 @@
 
 
 class Settings {
-       public:
-               Settings();
-               ~Settings();
+public:
+                                                               Settings();
+                                                               ~Settings();
 
-               status_t GetNextInterface(uint32& cookie, BMessage& interface);
-               status_t GetNextService(uint32& cookie, BMessage& service);
-               const BMessage& Services() const;
+                       status_t                        
GetNextInterface(uint32& cookie,
+                                                                       
BMessage& interface);
+                       status_t                        GetNextService(uint32& 
cookie,
+                                                                       
BMessage& service);
+                       const BMessage&         Services() const;
 
-               status_t StartMonitoring(const BMessenger& target);
-               status_t StopMonitoring(const BMessenger& target);
+                       status_t                        StartMonitoring(const 
BMessenger& target);
+                       status_t                        StopMonitoring(const 
BMessenger& target);
 
-               status_t Update(BMessage* message);
+                       status_t                        Update(BMessage* 
message);
 
-       private:
-               status_t _Load(const char* name = NULL, uint32* _type = NULL);
-               status_t _GetPath(const char* name, BPath& path);
+private:
+                       status_t                        _Load(const char* name 
= NULL,
+                                                                       uint32* 
_type = NULL);
+                       status_t                        _GetPath(const char* 
name, BPath& path);
 
-               status_t _StartWatching(const char* name, const BMessenger& 
target);
-               const settings_template* _FindSettingsTemplate(
-                       const settings_template* settingsTemplate, const char* 
name);
-               const settings_template* _FindParentValueTemplate(
-                       const settings_template* settingsTemplate);
-               status_t _AddParameter(const driver_parameter& parameter,
-                       const char* name, uint32 type, BMessage& message);
+                       status_t                        _StartWatching(const 
char* name,
+                                                                       const 
BMessenger& target);
+                       const settings_template* _FindSettingsTemplate(
+                                                                       const 
settings_template* settingsTemplate,
+                                                                       const 
char* name);
+                       const settings_template* _FindParentValueTemplate(
+                                                                       const 
settings_template* settingsTemplate);
+                       status_t                        _AddParameter(const 
driver_parameter& parameter,
+                                                                       const 
char* name, uint32 type,
+                                                                       
BMessage& message);
 
-               status_t _ConvertFromDriverParameter(const driver_parameter& 
parameter,
-                       const settings_template* settingsTemplate, BMessage& 
message);
-               status_t _ConvertFromDriverSettings(const driver_settings& 
settings,
-                       const settings_template* settingsTemplate, BMessage& 
message);
-               status_t _ConvertFromDriverSettings(const char* path,
-                       const settings_template* settingsTemplate, BMessage& 
message);
+                       status_t                        
_ConvertFromDriverParameter(
+                                                                       const 
driver_parameter& parameter,
+                                                                       const 
settings_template* settingsTemplate,
+                                                                       
BMessage& message);
+                       status_t                        
_ConvertFromDriverSettings(
+                                                                       const 
driver_settings& settings,
+                                                                       const 
settings_template* settingsTemplate,
+                                                                       
BMessage& message);
+                       status_t                        
_ConvertFromDriverSettings(const char* path,
+                                                                       const 
settings_template* settingsTemplate,
+                                                                       
BMessage& message);
 
-               bool _IsWatching(const BMessenger& target) const { return 
fListener == target; }
-               bool _IsWatching() const { return fListener.IsValid(); }
+                       bool                            _IsWatching(const 
BMessenger& target) const
+                                                                       { 
return fListener == target; }
+                       bool                            _IsWatching() const
+                                                                       { 
return fListener.IsValid(); }
 
-               BMessenger      fListener;
-               BMessage        fInterfaces;
-               BMessage        fServices;
-               bool            fUpdated;
+private:
+                       BMessenger                      fListener;
+                       BMessage                        fInterfaces;
+                       BMessage                        fServices;
 };
 
+
 static const uint32 kMsgInterfaceSettingsUpdated = 'SUif';
 static const uint32 kMsgServiceSettingsUpdated = 'SUsv';
 
+
 #endif // SETTINGS_H


Other related posts: