Some style cleanup of LabelModel.h

app/LabelModel.h

@@ -31,14 +31,23 @@
 namespace glabels
 {
 
+
 	class LabelModel : public QObject
 	{
 		Q_OBJECT
 
+
+		/////////////////////////////////
+		// Lifecycle
+		/////////////////////////////////
 	public:
 		LabelModel();
 		virtual ~LabelModel() {}
 
+
+		/////////////////////////////////
+		// Signals
+		/////////////////////////////////
 	signals:
 		void changed();
 		void nameChanged();
@@ -51,83 +60,75 @@ namespace glabels
 		void objectToTop( LabelModelObject* object );
 		void objectToBottom( LabelModelObject* object );
 
+
+		/////////////////////////////////
+		// Properties
+		/////////////////////////////////
 	public:
-		bool isModified() const { return mModified; }
+		inline bool isModified() const;
-		void clearModified() { mModified = false; }
+		inline void clearModified();
 
-		const QString &filename() const { return mFilename; }
+		inline const QString& filename() const;
-		void setFilename( const QString &filename )
+		inline void setFilename( const QString &filename );
-		{
-			if ( mFilename != filename )
-			{
-				mFilename = filename;
-				emit nameChanged();
-			}
-		}
 
-		int compressionLevel() const { return mCompressionLevel; }
+		inline int compressionLevel() const;
-		void setCompressionLevel( int compressionLevel ) { mCompressionLevel = compressionLevel; }
+		inline void setCompressionLevel( int compressionLevel );
 
-		const libglabels::Template* tmplate() const { return mTmplate; }
+		inline const libglabels::Template* tmplate() const;
-		void setTmplate( const libglabels::Template* tmplate )
+		inline void setTmplate( const libglabels::Template* tmplate );
-		{
-			if (mTmplate != tmplate)
-			{
-				mTmplate = tmplate;
-				mFrame = tmplate->frames().first();
-				mModified = true;
-				emit changed();
-				emit sizeChanged();
-			}
-		}
 
-		bool rotate() const { return mRotate; }
+		inline bool rotate() const;
-		void setRotate( bool rotate )
+		inline void setRotate( bool rotate );
-		{
-			if (mRotate != rotate)
-			{
-				mRotate = rotate;
-				mModified = true;
-				emit changed();
-				emit sizeChanged();
-			}
-		}
 
-		double w() const { return mRotate ? mFrame->h() : mFrame->w(); }
+		inline double w() const;
-		double h() const { return mRotate ? mFrame->w() : mFrame->h(); }
+		inline double h() const;
+
+		inline const QList<LabelModelObject*>& objectList() const;
 
 		
+		/////////////////////////////////
+		// Manage objects
+		/////////////////////////////////
+	public:
 		void addObject( LabelModelObject* object );
-
 		void deleteObject( LabelModelObject* object );
 
 
+		/////////////////////////////////
+		// Manipulate selection
+		/////////////////////////////////
+	public:
 		void selectObject( LabelModelObject* object );
-
 		void unselectObject( LabelModelObject* object );
-
 		void selectAll();
-
 		void unselectAll();
-
 		void selectRegion( const LabelRegion& region );
-
 		bool isSelectionEmpty();
-
 		bool isSelectionAtomic();
 
 
+		/////////////////////////////////
+		// Get selected objects
+		/////////////////////////////////
+	public:
 		QList<LabelModelObject*> getSelection();
-
 		LabelModelObject* getFirstSelectedObject();
 
 
+		/////////////////////////////////
+		// Query selection capabilities
+		/////////////////////////////////
+	public:
 		bool canSelectionText();
 		bool canSelectionFill();
 		bool canSelectionLineColor();
 		bool canSelectionLineWidth();
 
 
+		/////////////////////////////////
+		// Operations on selections
+		/////////////////////////////////
+	public:
 		void deleteSelection();
 		void raiseSelectionToTop();
 		void lowerSelectionToBottom();
@@ -157,24 +158,130 @@ namespace glabels
 		void setSelectionLineColorNode( ColorNode lineColorNode );
 		void setSelectionFillColorNode( ColorNode fillColorNode );
 
-
+		/////////////////////////////////
+		// Slots
+		/////////////////////////////////
 	private slots:
 		void onObjectChanged( LabelModelObject* object );
 		void onObjectMoved( LabelModelObject* object );
 
 
+		/////////////////////////////////
+		// Private data
+		/////////////////////////////////
 	private:
-
-		QList<LabelModelObject*> mObjectList;
-
 		bool                         mModified;
 		QString                      mFilename;
 		int                          mCompressionLevel;
 		const libglabels::Template*  mTmplate;
 		const libglabels::Frame*     mFrame;
 		bool                         mRotate;
+
+		QList<LabelModelObject*>     mObjectList;
+
 	};
 
+
+
+	/////////////////////////////////
+	// INLINE METHODS
+	/////////////////////////////////
+
+	inline bool LabelModel::isModified() const
+	{
+		return mModified;
+	}
+
+
+	inline void LabelModel::clearModified()
+	{
+		mModified = false;
+	}
+
+
+	inline const QString& LabelModel::filename() const
+	{
+		return mFilename;
+	}
+
+
+	inline void LabelModel::setFilename( const QString &filename )
+	{
+		if ( mFilename != filename )
+		{
+			mFilename = filename;
+			emit nameChanged();
+		}
+	}
+
+
+	inline int LabelModel::compressionLevel() const
+	{
+		return mCompressionLevel;
+	}
+
+
+	inline void LabelModel::setCompressionLevel( int compressionLevel )
+	{
+		mCompressionLevel = compressionLevel;
+	}
+
+
+	inline const libglabels::Template* LabelModel::tmplate() const
+	{
+		return mTmplate;
+	}
+
+
+	inline void LabelModel::setTmplate( const libglabels::Template* tmplate )
+	{
+		if (mTmplate != tmplate)
+		{
+			mTmplate = tmplate;
+			mFrame = tmplate->frames().first();
+			mModified = true;
+			emit changed();
+			emit sizeChanged();
+		}
+	}
+
+
+	inline bool LabelModel::rotate() const
+	{
+		return mRotate;
+	}
+
+
+	inline void LabelModel::setRotate( bool rotate )
+	{
+		if (mRotate != rotate)
+		{
+			mRotate = rotate;
+			mModified = true;
+			emit changed();
+			emit sizeChanged();
+		}
+	}
+
+
+	inline double LabelModel::w() const
+	{
+		return mRotate ? mFrame->h() : mFrame->w();
+	}
+
+
+	inline double LabelModel::h() const
+	{
+		return mRotate ? mFrame->w() : mFrame->h();
+	}
+
+
+	inline const QList<LabelModelObject*>& LabelModel::objectList() const
+	{
+		return mObjectList;
+	}
+
+
 }
 
 #endif // glabels_LabelModel_h