- make oApp not return a const app object

- homogenize oconfig documentation
- remove debug output

3 files changed, 64 insertions, 87 deletions

diff --git a/libopie2/opiecore/oapplication.h b/libopie2/opiecore/oapplication.h
index 1bd34e2..6bc2b1c 100644
--- a/libopie2/opiecore/oapplication.h
+++ b/libopie2/opiecore/oapplication.h
@@ -63,9 +63,9 @@ class OApplication: public QPEApplication
      * more than one cannot be created in the same application. It
      * saves you the trouble of having to pass the pointer explicitly
      * to every function that may require it.
      */
-    static const OApplication* oApplication() { return _instance; };
+    static OApplication* oApplication() { return _instance; };
     /**
      * Returns the application name as given during creation.
      *
      * @returns a reference to the application name
diff --git a/libopie2/opiecore/oconfig.h b/libopie2/opiecore/oconfig.h
index afe14b1..444d280 100644
--- a/libopie2/opiecore/oconfig.h
+++ b/libopie2/opiecore/oconfig.h
@@ -32,8 +32,9 @@
 #ifndef OCONFIG_H
 #define OCONFIG_H
 
 //FIXME: Implement for X11 or reuse libqpe/Config there also?
+//FIXME: Or rather use QSettings also for libqpe?
 
 #include <qpe/config.h>
 
 class QColor;
@@ -46,113 +47,85 @@ class QFont;
 
 class OConfig : public Config
 {
   public:
-
-  /**
-    * Constructs a OConfig object.
-    *
-    * @param name A file to parse.
-    */
+    /**
+     * Constructs a OConfig object with a @a name.
+     */
     OConfig( const QString &name, Domain domain = User );
-
-  /**
-    * Destructs the OConfig object.
-    *
-    * Writes back any dirty configuration entries, and destroys
-    * dynamically created objects.
-    */
+    /**
+     * Destructs the OConfig object.
+     *
+     * Writes back any dirty configuration entries, and destroys
+     * dynamically created objects.
+     */
     virtual ~OConfig();
-
-   /**
-    * Returns the name of the group in which we are
-    * searching for keys and from which we are retrieving entries.
-    *
-    * @return The current group.
-    */
+    /**
+     * @returns the name of the current group.
+     * The current group is used for searching keys and accessing entries.
+     */
     const QString& group() { return git.key(); };
-
-   /**
-    * Reads a @ref QColor entry.
-    *
-    * Read the value of an entry specified by @p pKey in the current group
-    * and interpret it as a color.
-    *
-    * @param pKey The key to search for.
-    * @param pDefault A default value (null QColor by default) returned if the
-    * key was not found or if the value cannot be interpreted.
-    * @return The value for this key.
-    */
+    /**
+     * @returns a @ref QColor entry or a @a default value if the key is not found.
+     */
     QColor readColorEntry( const QString& key, const QColor* pDefault ) const;
-
-   /**
-    * Reads a @ref QFont value.
-    *
-    * Read the value of an entry specified by @p pKey in the current group
-    * and interpret it as a font object.
-    *
-    * @param pKey The key to search for.
-    * @param pDefault A default value (null QFont by default) returned if the
-    * key was not found or if the read value cannot be interpreted.
-    * @return The value for this key.
-    */
+    /**
+     * @returns a @ref QFont value or a @a default value if the key is not found.
+     */
     QFont readFontEntry( const QString& key, const QFont* pDefault ) const;
-
 };
 
 /**
-  * Helper class to facilitate working with @ref OConfig / @ref OSimpleConfig
-  * groups.
-  *
-  * Careful programmers always set the group of a
-  * @ref OConfig object to the group they want to read from
-  * and set it back to the old one of afterwards. This is usually
-  * written as:
-  * <pre>
-  *
-  * QString oldgroup config()->group();
-  * config()->setGroup( "TheGroupThatIWant" );
-  * ...
-  * config()->writeEntry( "Blah", "Blubb" );
-  *
-  * config()->setGroup( oldgroup );
-  * </pre>
-  *
-  * In order to facilitate this task, you can use
-  * OConfigGroupSaver. Simply construct such an object ON THE STACK
-  * when you want to switch to a new group. Then, when the object goes
-  * out of scope, the group will automatically be restored. If you
-  * want to use several different groups within a function or method,
-  * you can still use OConfigGroupSaver: Simply enclose all work with
-  * one group (including the creation of the OConfigGroupSaver object)
-  * in one block.
-  *
-  * @author Matthias Kalle Dalheimer <Kalle@kde.org>
-  * @version $Id$
-  * @see OConfig
-  * @short Helper class for easier use of OConfig groups
-  */
+ * @brief Helper class for easier use of OConfig groups.
+ *
+ * Careful programmers always set the group of a
+ * @ref OConfig object to the group they want to read from
+ * and set it back to the old one of afterwards. This is usually
+ * written as:
+ * <pre>
+ *
+ * QString oldgroup config()->group();
+ * config()->setGroup( "TheGroupThatIWant" );
+ * ...
+ * config()->writeEntry( "Blah", "Blubb" );
+ *
+ * config()->setGroup( oldgroup );
+ * </pre>
+ *
+ * In order to facilitate this task, you can use
+ * OConfigGroupSaver. Simply construct such an object ON THE STACK
+ * when you want to switch to a new group. Then, when the object goes
+ * out of scope, the group will automatically be restored. If you
+ * want to use several different groups within a function or method,
+ * you can still use OConfigGroupSaver: Simply enclose all work with
+ * one group (including the creation of the OConfigGroupSaver object)
+ * in one block.
+ *
+ * @author Matthias Kalle Dalheimer <Kalle@kde.org>
+ * @version $Id$
+ * @see OConfig
+ */
 
 class OConfigGroupSaver
 {
   public:
-   /**
-    * Constructor. You pass a pointer to the OConfigBase-derived
-    * object you want to work with and a string indicating the _new_
-    * group.
-    * @param config The OConfig-derived object this
-    *               OConfigGroupSaver works on.
-    * @param group  The new group that the config object should switch to.
-    */
+    /**
+     * Constructor.
+     * Create the object giving a @config object and a @a group to become
+     * the current group.
+     */
     OConfigGroupSaver( OConfig* config, QString group ) :_config(config), _oldgroup(config->group() )
         { _config->setGroup( group ); }
 
     OConfigGroupSaver( OConfig* config, const char *group ) :_config(config), _oldgroup(config->group())
         { _config->setGroup( group ); }
 
     OConfigGroupSaver( OConfig* config, const QCString &group ) : _config(config), _oldgroup(config->group())
         { _config->setGroup( group ); }
-
+    /**
+     * Destructor.
+     * Restores the last current group.
+     */
     ~OConfigGroupSaver() { _config->setGroup( _oldgroup ); }
 
     OConfig* config() { return _config; };
 
diff --git a/libopie2/opienet/omanufacturerdb.cpp b/libopie2/opienet/omanufacturerdb.cpp
index c185fc5..c3c213c 100644
--- a/libopie2/opienet/omanufacturerdb.cpp
+++ b/libopie2/opienet/omanufacturerdb.cpp
@@ -14,9 +14,9 @@
 **********************************************************************/
 
 #include "omanufacturerdb.h"
 
-// Qt
+/* QT */
 #include <qstring.h>
 #include <qfile.h>
 #include <qtextstream.h>
 
@@ -80,14 +80,18 @@ OManufacturerDB::OManufacturerDB()
             if ( extManu[0] == '#' ) // we have an extended manufacturer
             {
                 s.skipWhiteSpace();
                 extManu = s.readLine();
+                #ifdef DEBUG
                 qDebug( "OManufacturerDB: read '%s' as extended manufacturer string", (const char*) extManu );
+                #endif
                 manufacturersExt.insert( addr, extManu );
             }
             else
                 s.readLine();
+            #ifdef DEBUG
             qDebug( "ManufacturerDB: read tuple %s, %s", (const char*) addr, (const char*) manu );
+            #endif
             manufacturers.insert( addr, manu );
 
         }
     }