Code Review.

General clean up
let member variables have better names, starting with _ and the needed get functions
signals/slots --> Q_SIGNALS/Q_SLOTS
moc headers

1 files changed, 14 insertions, 13 deletions

diff --git a/src/bookmarks/bookmarkowner.h b/src/bookmarks/bookmarkowner.h
index e0c2a8ad..795ccbf0 100644
--- a/src/bookmarks/bookmarkowner.h
+++ b/src/bookmarks/bookmarkowner.h
@@ -40,6 +40,7 @@
 // Forward Declarations
 class KAction;
 
+
 /**
  * Reimplementation of KBookmarkOwner, this class allows to manage
  * bookmarks as actions.
@@ -111,14 +112,7 @@ public:
     */
     virtual void openFolderinTabs(const KBookmarkGroup &bookmark);
 
-signals:
-    /**
-     * This signal is emitted when an url has to be loaded
-     * @param url the URL to load
-     */
-    void openUrl(const KUrl &, const Rekonq::OpenType &);
-
-public slots:
+public Q_SLOTS:
     void setCurrentBookmark(const KBookmark &bookmark);
 
     void openBookmark(const KBookmark &bookmark = KBookmark());
@@ -132,16 +126,23 @@ public slots:
     void editBookmark(KBookmark bookmark = KBookmark());
     bool deleteBookmark(KBookmark bookmark = KBookmark());
 
-private:
-    KBookmarkManager *m_manager;
-
-    QVector<KAction*> m_actions;
-    KBookmark m_currentBookmark;
+Q_SIGNALS:
+    /**
+     * This signal is emitted when an url has to be loaded
+     * @param url the URL to load
+     */
+    void openUrl(const KUrl &, const Rekonq::OpenType &);
 
+private:
     void setupActions();
     void createAction(const BookmarkAction &action,
                       const QString &text, const QString &icon,
                       const QString &help, const char *slot);
+
+    
+    KBookmarkManager *m_manager;
+    QVector<KAction*> m_actions;
+    KBookmark m_currentBookmark;
 };
 
 #endif // BOOKMARKOWNER_H