Fix compiler warining non-virtual destructor

ref:
https://stackoverflow.com/questions/38407723/how-to-delete-an-object-of-derived-class-that-has-no-dstructor

elements/10_electric/91_en_60617/en_60617_11/qet_directory

@@ -3,5 +3,7 @@
         <name lang="fr">11- Schémas et plans d'installation, architecturaux et topographiques</name>
         <name lang="es">11- Dibujos y planos arquitectónicos y topográficos</name>
         <name lang="da">11- Bygningsmæssige og topografiske installationstegninger og -skemaer</name>
+        <name lang="en">11- Installation, architectural and topographical diagrams and plans</name>
+        <name lang="be">11- Installatie, bouwkundig en topografische schema's en plannen</name>
     </names>
 </qet-directory>

sources/borderproperties.cpp

@@ -27,7 +27,7 @@
 	- 17 columns of 60.0 px wide by 20.0px high
 	- 8    lines of 80.0 px high by 20.0px wide
 
-	Initialise un objet BorderProperties avec les proprietes par
+	\~French Initialise un objet BorderProperties avec les proprietes par
 	defaut suivantes :
 	- 17 colonnes affichees de 60.0 px de large pour 20.0px de haut
 	- 8    lignes affichees de 80.0 px de haut pour 20.0px de large
@@ -53,12 +53,11 @@ BorderProperties::~BorderProperties() {
 
 /**
 	@brief BorderProperties::operator ==
-	@param bp :
+
-	- other BorderProperties container/class
+	\~ @param bp : Other BorderProperties container/class.
-	- autre conteneur BorderProperties
+	\~French Autre conteneur BorderProperties
-	@return
+	\~ @return True if it and this container are identical, false otherwise.
-	- true if it and this container are identical, false otherwise
+	\~French True si ip et ce conteneur sont identiques, false sinon
-	- true si ip et ce conteneur sont identiques, false sinon
 */
 bool BorderProperties::operator==(const BorderProperties &bp) {
 	return(
@@ -75,12 +74,13 @@ bool BorderProperties::operator==(const BorderProperties &bp) {
 
 /**
 	@brief BorderProperties::operator !=
-	@param bp :
+
-	- other BorderProperties container/class
+	\~ @param bp :
-	- autre conteneur BorderProperties
+	Other BorderProperties container/class.
-	@return
+	\~French Autre conteneur BorderProperties
-	- false if it and this container are identical, true otherwise
+	\~ @return
-	- false si bp et ce conteneur sont identiques, true sinon
+	False if it and this container are identical, true otherwise.
+	\~French False si bp et ce conteneur sont identiques, true sinon
 */
 bool BorderProperties::operator!=(const BorderProperties &bp) {
 	return(!(*this == bp));
@@ -88,11 +88,12 @@ bool BorderProperties::operator!=(const BorderProperties &bp) {
 
 /**
 	@brief BorderProperties::toXml
-	- Exports dimensions as XML attributes added to element e.
+	Exports dimensions as XML attributes added to element e.
-	- Exporte les dimensions sous formes d'attributs XML ajoutes a l'element e.
+	\~French Exporte les dimensions sous formes d'attributs XML ajoutes a l'element e.
-	@param e :
+
-	- XML element to which attributes will be added
+	\~ @param e :
-	- Element XML auquel seront ajoutes des attributs
+	XML element to which attributes will be added
+	\~French Element XML auquel seront ajoutes des attributs
 */
 void BorderProperties::toXml(QDomElement &e) const {
 	e.setAttribute("cols",        columns_count);
@@ -105,11 +106,12 @@ void BorderProperties::toXml(QDomElement &e) const {
 
 /**
 	@brief BorderProperties::fromXml
-	- Import dimensions from XML attributes of element e
+	Import dimensions from XML attributes of element e
-	- Importe les dimensions a partir des attributs XML de l'element e
+	\~French Importe les dimensions a partir des attributs XML de l'element e
-	@param e :
+
-	- XML element whose attributes will be read
+	\~ @param e :
-	- Element XML dont les attributs seront lus
+	XML element whose attributes will be read
+	\~French Element XML dont les attributs seront lus
 */
 void BorderProperties::fromXml(QDomElement &e) {
 	if (e.hasAttribute("cols"))        columns_count   = e.attribute("cols").toInt();
@@ -122,14 +124,15 @@ void BorderProperties::fromXml(QDomElement &e) {
 
 /**
 	@brief BorderProperties::toSettings
-	- Export dimensions in a QSettings object.
+	Export dimensions in a QSettings object.
-	- Exporte les dimensions dans une configuration.
+	\~French Exporte les dimensions dans une configuration.
-	@param settings :
+
-	- QSettings object to write
+	\~ @param settings :
-	- Parametres a ecrire
+	QSettings object to write
-	@param prefix :
+	\~French Parametres a ecrire
-	- prefix to be added before the names of the parameters
+	\~ @param prefix :
-	- prefixe a ajouter devant les noms des parametres
+	prefix to be added before the names of the parameters
+	\~French prefixe a ajouter devant les noms des parametres
 */
 void BorderProperties::toSettings(QSettings &settings, const QString &prefix) const {
 	settings.setValue(prefix + "cols",        columns_count);
@@ -142,14 +145,12 @@ void BorderProperties::toSettings(QSettings &settings, const QString &prefix) co
 
 /**
 	@brief BorderProperties::fromSettings
-	- Import dimensions from a QSettings object.
+	Import dimensions from a QSettings object.
-	- Importe les dimensions depuis une configuration.
+	\~French Importe les dimensions depuis une configuration.
-	@param settings :
+	\~ @param settings : QSettings object to read
-	- QSettings object to read
+	\~French Parametres a lire
-	- Parametres a lire
+	\~ @param prefix : prefix to be added before the names of the parameters
-	@param prefix :
+	\~French prefixe a ajouter devant les noms des parametres
-	- prefix to be added before the names of the parameters
-	- prefixe a ajouter devant les noms des parametres
 */
 void BorderProperties::fromSettings(QSettings &settings, const QString &prefix) {
 	columns_count   = settings.value(prefix + "cols",            columns_count).toInt();

sources/bordertitleblock.cpp

@@ -32,9 +32,16 @@
 #define MIN_ROW_HEIGHT 5.0
 
 /**
-	Constructeur simple : construit une bordure en recuperant les dimensions
+	@brief BorderTitleBlock::BorderTitleBlock
+	Simple constructor:
+	build a border by recovering the dimensions
+	in the application configuration.
+
+	\~French Constructeur simple :
+	construit une bordure en recuperant les dimensions
 	dans la configuration de l'application.
-	@param parent QObject parent de ce BorderTitleBlock
+	\~ @param parent :
+	\~French QObject parent de ce BorderTitleBlock
 */
 BorderTitleBlock::BorderTitleBlock(QObject *parent) :
 	QObject(parent)
@@ -61,95 +68,119 @@ BorderTitleBlock::BorderTitleBlock(QObject *parent) :
 }
 
 /**
-	Destructeur - ne fait rien
+	@brief BorderTitleBlock::~BorderTitleBlock
+	\~French Destructeur - ne fait rien
 */
 BorderTitleBlock::~BorderTitleBlock() {
 }
 
 /**
- * @brief BorderTitleBlock::titleBlockRect
+	@brief BorderTitleBlock::titleBlockRect
- * @return the rectangle of the titleblock in scene coordinate.
+	@return the rectangle of the titleblock in scene coordinate.
 */
 QRectF BorderTitleBlock::titleBlockRect() const
 {
 	if (m_edge == Qt::BottomEdge)
-		return QRectF(diagram_rect_.bottomLeft(), QSize(diagram_rect_.width(), m_titleblock_template_renderer -> height()));
+		return QRectF(diagram_rect_.bottomLeft(),
+			      QSize(diagram_rect_.width(),
+				    m_titleblock_template_renderer -> height()
+				    ));
 	else
-		return QRectF(diagram_rect_.topRight(), QSize(m_titleblock_template_renderer -> height(), diagram_rect_.height()));
+		return QRectF(diagram_rect_.topRight(),
+			      QSize(m_titleblock_template_renderer -> height(),
+				    diagram_rect_.height()
+				    ));
 }
 
+/**
+	@brief BorderTitleBlock::titleblockInformation
+	@return
+*/
 DiagramContext BorderTitleBlock::titleblockInformation() const {
 	return m_titleblock_template_renderer->context();
 }
 
 /**
- * @brief BorderTitleBlock::titleBlockRectForQPainter
+	@brief BorderTitleBlock::titleBlockRectForQPainter
- * @return The title block rect to use with the QPainter in the method draw.
+	@return The title block rect to use with the QPainter in the method draw.
- * The returned rect is alway horizontal (like displayed at the bottom of rect) only the top left change of pos
+	The returned rect is alway horizontal
- * according to the edge where the title block need to be displayed.
+	(like displayed at the bottom of rect) only the top left change of pos
- * Rect according to edge:
+	according to the edge where the title block need to be displayed.
- * Bottom : top left is at the bottom left edge of the diagram rect.
+	Rect according to edge:
- * Right : top left is at the bottom right of diagram rect. Befor use this rect you need to rotate the QPainter by -90°
+	Bottom : top left is at the bottom left edge of the diagram rect.
- * for snap the rect at the right edge of diagram.
+	Right : top left is at the bottom right of diagram rect.
+	Befor use this rect you need to rotate the QPainter by -90°
+	for snap the rect at the right edge of diagram.
 */
 QRectF BorderTitleBlock::titleBlockRectForQPainter() const
 {
-	if (m_edge == Qt::BottomEdge) //Rect at bottom have same position and dimension of displayed rect
+	//Rect at bottom have same position and dimension of displayed rect
+	if (m_edge == Qt::BottomEdge)
 		return titleBlockRect();
 	else
-		return QRectF (diagram_rect_.bottomRight(), QSize(diagram_rect_.height(), m_titleblock_template_renderer -> height()));
+		return QRectF (diagram_rect_.bottomRight(),
+			       QSize(diagram_rect_.height(),
+				     m_titleblock_template_renderer -> height()
+				     ));
 
 }
 
 /**
- * @brief BorderTitleBlock::borderAndTitleBlockRect
+	@brief BorderTitleBlock::borderAndTitleBlockRect
- * @return the bounding rectangle of diagram and titleblock.
+	@return the bounding rectangle of diagram and titleblock.
- * It's like unite outsideBorderRect and titleBlockRect.
+	It's like unite outsideBorderRect and titleBlockRect.
- * The rect is in scene coordinate
+	The rect is in scene coordinate
 */
 QRectF BorderTitleBlock::borderAndTitleBlockRect() const {
 	return diagram_rect_ | titleBlockRect();
 }
 
 /**
- * @brief BorderTitleBlock::columnsRect
+	@brief BorderTitleBlock::columnsRect
- * @return The columns rect in scene coordinate.
+	@return The columns rect in scene coordinate.
- * If column is not displayed, return a null QRectF
+	If column is not displayed, return a null QRectF
 */
 QRectF BorderTitleBlock::columnsRect() const
 {
 	if (!display_columns_) return QRectF();
-	return QRectF (Diagram::margin, Diagram::margin, (columns_count_*columns_width_) + rows_header_width_, columns_header_height_);
+	return QRectF (Diagram::margin,
+		       Diagram::margin,
+		       (columns_count_*columns_width_) + rows_header_width_,
+		       columns_header_height_);
 }
 
 /**
- * @brief BorderTitleBlock::rowsRect
+	@brief BorderTitleBlock::rowsRect
- * @return The rows rect in scene coordinate.
+	@return The rows rect in scene coordinate.
- * If row is not displayed, return a null QRectF
+	If row is not displayed, return a null QRectF
 */
 QRectF BorderTitleBlock::rowsRect() const
 {
 	if (!display_rows_) return QRectF();
-	return QRectF (Diagram::margin, Diagram::margin, rows_header_width_, (rows_count_*rows_height_) + columns_header_height_);
+	return QRectF (Diagram::margin,
+		       Diagram::margin,
+		       rows_header_width_,
+		       (rows_count_*rows_height_) + columns_header_height_);
 }
 
 /**
- * @brief BorderTitleBlock::outsideBorderRect
+	@brief BorderTitleBlock::outsideBorderRect
- * @return The rect of outside border (diagram with columns and rows)
+	@return The rect of outside border (diagram with columns and rows)
- * The rect is in scene coordinate
+	The rect is in scene coordinate
 */
 QRectF BorderTitleBlock::outsideBorderRect() const
 {
-	return QRectF (Diagram::margin, Diagram::margin,
+	return QRectF (Diagram::margin,
+		       Diagram::margin,
 		       (columns_width_*columns_count_) + rows_header_width_,
 		       (rows_height_*rows_count_) + columns_header_height_);
 }
 
 /**
- * @brief BorderTitleBlock::insideBorderRect
+	@brief BorderTitleBlock::insideBorderRect
- * @return The rect of the inside border, in other word, the drawing area.
+	@return The rect of the inside border, in other word, the drawing area.
- * This method take care about if rows or columns are displayed or not.
+	This method take care about if rows or columns are displayed or not.
- * The rect is in scene coordinate
+	The rect is in scene coordinate
 */
 QRectF BorderTitleBlock::insideBorderRect() const
 {
@@ -165,6 +196,7 @@ QRectF BorderTitleBlock::insideBorderRect() const
 }
 
 /**
+	@brief BorderTitleBlock::titleBlockToXml
 	Exports the title block current values to XML.
 	@param xml_elmt the XML element attributes will be added to
 */
@@ -173,6 +205,7 @@ void BorderTitleBlock::titleBlockToXml(QDomElement &xml_elmt) {
 }
 
 /**
+	@brief BorderTitleBlock::titleBlockFromXml
 	Reads the title block values from XML.
 	@param xml_elmt the XML element values will be read from
 */
@@ -183,6 +216,7 @@ void BorderTitleBlock::titleBlockFromXml(const QDomElement &xml_elmt) {
 }
 
 /**
+	@brief BorderTitleBlock::borderToXml
 	Exports the border current settings to XML.
 	@param xml_elmt the XML element attributes will be added to
 */
@@ -200,6 +234,7 @@ void BorderTitleBlock::borderToXml(QDomElement &xml_elmt) {
 }
 
 /**
+	@brief BorderTitleBlock::borderFromXml
 	Reads the border settings from XML.
 	@param xml_elmt the XML element values will be read from
 */
@@ -213,7 +248,8 @@ void BorderTitleBlock::borderFromXml(const QDomElement &xml_elmt) {
 	double cols_width = xml_elmt.attribute("colsize").toDouble(&ok);
 	if (ok) setColumnsWidth(cols_width);
 	
-	// backward compatibility: diagrams saved with 0.1 version have a "height" attribute
+	// backward compatibility:
+	//	diagrams saved with 0.1 version have a "height" attribute
 	if (xml_elmt.hasAttribute("rows") && xml_elmt.hasAttribute("rowsize")) {
 		// rows counts
 		int rows_count = xml_elmt.attribute("rows").toInt(&ok);
@@ -236,7 +272,9 @@ void BorderTitleBlock::borderFromXml(const QDomElement &xml_elmt) {
 }
 
 /**
-	@return les proprietes du cartouches
+	@brief BorderTitleBlock::exportTitleBlock
+	@return the properties of the titleblock
+	\~French les proprietes du cartouches
 */
 TitleBlockProperties BorderTitleBlock::exportTitleBlock() {
 	TitleBlockProperties ip;
@@ -260,8 +298,8 @@ TitleBlockProperties BorderTitleBlock::exportTitleBlock() {
 }
 
 /**
- * @brief BorderTitleBlock::importTitleBlock
+	@brief BorderTitleBlock::importTitleBlock
- * @param ip the new properties of titleblock
+	@param ip the new properties of titleblock
 */
 void BorderTitleBlock::importTitleBlock(const TitleBlockProperties &ip) {
 	setAuthor(ip.author);
@@ -282,12 +320,15 @@ void BorderTitleBlock::importTitleBlock(const TitleBlockProperties &ip) {
 	additional_fields_ = ip.context;
 	
 	emit(needFolioData()); // Note: we expect additional data to be provided
-	// through setFolioData(), which in turn calls updateDiagramContextForTitleBlock().
+	// through setFolioData(),
+	// which in turn calls updateDiagramContextForTitleBlock().
 	emit(needTitleBlockTemplate(ip.template_name));
 }
 
 /**
-	@return les proprietes de la bordure
+	@brief BorderTitleBlock::exportBorder
+	@return border properties
+	\~French les proprietes de la bordure
 */
 BorderProperties BorderTitleBlock::exportBorder() {
 	BorderProperties bp;
@@ -303,7 +344,9 @@ BorderProperties BorderTitleBlock::exportBorder() {
 }
 
 /**
-	@param bp les nouvelles proprietes de la bordure
+	@brief BorderTitleBlock::importBorder
+	@param bp : the new properties of the border
+	\~French les nouvelles proprietes de la bordure
 */
 void BorderTitleBlock::importBorder(const BorderProperties &bp) {
 	setColumnsHeaderHeight(bp.columns_header_height);
@@ -317,6 +360,7 @@ void BorderTitleBlock::importBorder(const BorderProperties &bp) {
 }
 
 /**
+	@brief BorderTitleBlock::titleBlockTemplate
 	@return the titleblock template used to render the titleblock
 	@see TitleBlockTemplateRenderer::titleBlockTemplate()
 */
@@ -325,14 +369,19 @@ const TitleBlockTemplate *BorderTitleBlock::titleBlockTemplate() {
 }
 
 /**
-	@param titleblock_template The new titleblock template to use to render the titleblock
+	@brief BorderTitleBlock::setTitleBlockTemplate
+	@param titleblock_template:
+	The new titleblock template to use to render the titleblock
 	@see TitleBlockTemplateRenderer::setTitleBlockTemplate()
 */
-void BorderTitleBlock::setTitleBlockTemplate(const TitleBlockTemplate *titleblock_template) {
+void BorderTitleBlock::setTitleBlockTemplate(
-	m_titleblock_template_renderer -> setTitleBlockTemplate(titleblock_template);
+		const TitleBlockTemplate *titleblock_template) {
+	m_titleblock_template_renderer -> setTitleBlockTemplate(
+				titleblock_template);
 }
 
 /**
+	@brief BorderTitleBlock::titleBlockTemplateName
 	@return The name of the template used to render the titleblock.
 */
 QString BorderTitleBlock::titleBlockTemplateName() const {
@@ -341,10 +390,11 @@ QString BorderTitleBlock::titleBlockTemplateName() const {
 }
 
 /**
+	@brief BorderTitleBlock::titleBlockTemplateChanged
 	This slot may be used to inform this class that the given title block
-	template has changed. The title block-dedicated rendering cache will thus be
+	template has changed.
-	flushed.
+	The title block-dedicated rendering cache will thus be flushed.
-	@param template_name Name of the title block template that has changed
+	@param template_name : Name of the title block template that has changed
 */
 void BorderTitleBlock::titleBlockTemplateChanged(const QString &template_name) {
 	if (titleBlockTemplateName() != template_name) return;
@@ -352,14 +402,18 @@ void BorderTitleBlock::titleBlockTemplateChanged(const QString &template_name) {
 }
 
 /**
+	@brief BorderTitleBlock::titleBlockTemplateRemoved
 	This slot has to be used to inform this class that the given title block
 	template is about to be removed and is no longer accessible. This class
 	will either use the provided optional TitleBlockTemplate or the default
 	title block provided by QETApp::defaultTitleBlockTemplate()
-	@param template_name Name of the title block template that has changed
+	@param removed_template_name :
+	Name of the title block template that has changed
 	@param new_template (Optional) title block template to use instead
 */
-void BorderTitleBlock::titleBlockTemplateRemoved(const QString &removed_template_name, const TitleBlockTemplate *new_template) {
+void BorderTitleBlock::titleBlockTemplateRemoved(
+		const QString &removed_template_name,
+		const TitleBlockTemplate *new_template) {
 	if (titleBlockTemplateName() != removed_template_name) return;
 	
 	if (new_template) {
@@ -370,7 +424,9 @@ void BorderTitleBlock::titleBlockTemplateRemoved(const QString &removed_template
 }
 
 /**
-	@param di true pour afficher le cartouche, false sinon
+	@brief BorderTitleBlock::displayTitleBlock
+	@param di : true to display the title block, false otherwise
+	\~French true pour afficher le cartouche, false sinon
 */
 void BorderTitleBlock::displayTitleBlock(bool di) {
 	bool change = (di != display_titleblock_);
@@ -379,7 +435,9 @@ void BorderTitleBlock::displayTitleBlock(bool di) {
 }
 
 /**
-	@param dc true pour afficher les entetes des colonnes, false sinon
+	@brief BorderTitleBlock::displayColumns
+	@param dc : true to display the column headers, false otherwise
+	\~French true pour afficher les entetes des colonnes, false sinon
 */
 void BorderTitleBlock::displayColumns(bool dc) {
 	bool change = (dc != display_columns_);
@@ -388,7 +446,9 @@ void BorderTitleBlock::displayColumns(bool dc) {
 }
 
 /**
-	@param dr true pour afficher les entetes des lignes, false sinon
+	@brief BorderTitleBlock::displayRows
+	@param dr : true to display line headers, false otherwise
+	\~French true pour afficher les entetes des lignes, false sinon
 */
 void BorderTitleBlock::displayRows(bool dr) {
 	bool change = (dr != display_rows_);
@@ -397,9 +457,13 @@ void BorderTitleBlock::displayRows(bool dr) {
 }
 
 /**
-	@param db true pour afficher la bordure du schema, false sinon
+	@brief BorderTitleBlock::displayBorder
-	Note : si l'affichage de la bordure est ainsi desactivee, les lignes et
+	@param db : true to display the border of the diagram, false otherwise
-	colonnes ne seront pas dessinees.
+	\~French true pour afficher la bordure du schema, false sinon
+	\~ @note : if the border display is deactivated,
+	the rows and columns will not be drawn.
+	\~French si l'affichage de la bordure est ainsi desactivee,
+	les lignes et colonnes ne seront pas dessinees.
 */
 void BorderTitleBlock::displayBorder(bool db) {
 	bool change = (db != display_border_);
@@ -408,29 +472,34 @@ void BorderTitleBlock::displayBorder(bool db) {
 }
 
 /**
- * @brief BorderTitleBlock::slot_setAutoPageNum
+	@brief BorderTitleBlock::slot_setAutoPageNum
- * @param pageAutoNum
+	@param pageAutoNum :
- * Set Page (Folio) Auto Num
+	Set Page (Folio) Auto Num
 */
 void BorderTitleBlock::slot_setAutoPageNum(QString pageAutoNum) {
 	btb_auto_page_num_=std::move(pageAutoNum);
 }
 
 /**
- * @brief BorderTitleBlock::updateRectangles
+	@brief BorderTitleBlock::updateRectangles
- * This method update the diagram rect according to the value of rows and columns (number and size)
+	This method update the diagram rect according
+	to the value of rows and columns (number and size)
 */
 void BorderTitleBlock::updateRectangles()
 {
 	QRectF previous_diagram = diagram_rect_;
-	diagram_rect_ = QRectF(Diagram::margin, Diagram::margin, diagramWidth(), diagramHeight());
+	diagram_rect_ = QRectF(Diagram::margin,
-	if (diagram_rect_ != previous_diagram) emit(borderChanged(previous_diagram, diagram_rect_));
+			       Diagram::margin,
+			       diagramWidth(),
+			       diagramHeight());
+	if (diagram_rect_ != previous_diagram)
+		emit(borderChanged(previous_diagram, diagram_rect_));
 }
 
 /**
- * @brief BorderTitleBlock::draw
+	@brief BorderTitleBlock::draw
- * Draw the border and the titleblock.
+	Draw the border and the titleblock.
- * @param painter, QPainter to use for draw this.
+	@param painter, QPainter to use for draw this.
 */
 void BorderTitleBlock::draw(QPainter *painter)
 {
@@ -464,16 +533,24 @@ void BorderTitleBlock::draw(QPainter *painter)
 	if (display_border_ && display_columns_) {
 		for (int i = 1 ; i <= columns_count_ ; ++ i) {
 			QRectF numbered_rectangle = QRectF(
-				diagram_rect_.topLeft().x() + (rows_header_width_ + ((i - 1) * columns_width_)),
+				diagram_rect_.topLeft().x()
+					+ (rows_header_width_
+					   + ((i - 1) * columns_width_)),
 				diagram_rect_.topLeft().y(),
 				columns_width_,
 				columns_header_height_
 			);
 			painter -> drawRect(numbered_rectangle);
 			if (settings.value("border-columns_0", true).toBool()){
-			painter -> drawText(numbered_rectangle, Qt::AlignVCenter | Qt::AlignCenter, QString("%1").arg(i - 1));
+			painter -> drawText(numbered_rectangle,
+					    Qt::AlignVCenter
+					    | Qt::AlignCenter,
+					    QString("%1").arg(i - 1));
 			}else{
-			painter -> drawText(numbered_rectangle, Qt::AlignVCenter | Qt::AlignCenter, QString("%1").arg(i));
+			painter -> drawText(numbered_rectangle,
+					    Qt::AlignVCenter
+					    | Qt::AlignCenter,
+					    QString("%1").arg(i));
 			}
 		}
 	}
@@ -484,12 +561,19 @@ void BorderTitleBlock::draw(QPainter *painter)
 		for (int i = 1 ; i <= rows_count_ ; ++ i) {
 			QRectF lettered_rectangle = QRectF(
 				diagram_rect_.topLeft().x(),
-				diagram_rect_.topLeft().y() + (columns_header_height_ + ((i - 1) * rows_height_)),
+				diagram_rect_.topLeft().y()
+					+ (
+						columns_header_height_
+						+ ((i - 1)* rows_height_)
+						),
 				rows_header_width_,
 				rows_height_
 			);
 			painter -> drawRect(lettered_rectangle);
-			painter -> drawText(lettered_rectangle, Qt::AlignVCenter | Qt::AlignCenter, row_string);
+			painter -> drawText(lettered_rectangle,
+					    Qt::AlignVCenter
+					    | Qt::AlignCenter,
+					    row_string);
 			row_string = incrementLetters(row_string);
 		}
 	}
@@ -500,14 +584,18 @@ void BorderTitleBlock::draw(QPainter *painter)
 		if (m_edge == Qt::BottomEdge)
 		{
 			painter -> translate(tbt_rect.topLeft());
-			m_titleblock_template_renderer -> render(painter, tbt_rect.width());
+			m_titleblock_template_renderer -> render(
+						painter,
+						tbt_rect.width());
 			painter -> translate(-tbt_rect.topLeft());
 		}
 		else
 		{
 			painter->translate(tbt_rect.topLeft());
 			painter->rotate(-90);
-			m_titleblock_template_renderer -> render(painter, tbt_rect.width());
+			m_titleblock_template_renderer -> render(
+						painter,
+						tbt_rect.width());
 			painter->rotate(90);
 			painter -> translate(-tbt_rect.topLeft());
 		}
@@ -516,7 +604,20 @@ void BorderTitleBlock::draw(QPainter *painter)
 	painter -> restore();
 }
 
-void BorderTitleBlock::drawDxf(int width, int height, bool keep_aspect_ratio, QString &file_path, int color) {
+/**
+ * @brief BorderTitleBlock::drawDxf
+ * @param width
+ * @param height
+ * @param keep_aspect_ratio
+ * @param file_path
+ * @param color
+ */
+void BorderTitleBlock::drawDxf(
+		int width,
+		int height,
+		bool keep_aspect_ratio,
+		QString &file_path,
+		int color) {
 	Q_UNUSED (width); Q_UNUSED (height); Q_UNUSED (keep_aspect_ratio);
 
 	// Transform to DXF scale.
@@ -525,6 +626,7 @@ void BorderTitleBlock::drawDxf(int width, int height, bool keep_aspect_ratio, QS
 	rows_header_width_     *= Createdxf::xScale;
 	columns_width_         *= Createdxf::xScale;
 
+	// draw the empty box that appears as soon as there is a header
 	// dessine la case vide qui apparait des qu'il y a un entete
 	if (display_border_ &&
 		(display_columns_ ||
@@ -533,7 +635,10 @@ void BorderTitleBlock::drawDxf(int width, int height, bool keep_aspect_ratio, QS
 		Createdxf::drawRectangle(
 			file_path,
 			double(diagram_rect_.topLeft().x()) * Createdxf::xScale,
-			Createdxf::sheetHeight - double(diagram_rect_.topLeft().y()) * Createdxf::yScale - columns_header_height_,
+			Createdxf::sheetHeight
+					- double(diagram_rect_.topLeft().y())
+					* Createdxf::yScale
+					- columns_header_height_,
 			rows_header_width_,
 			columns_header_height_,
 			color
@@ -542,6 +647,7 @@ void BorderTitleBlock::drawDxf(int width, int height, bool keep_aspect_ratio, QS
 
 	QSettings settings;
 	
+	// draw the numbering of the columns
 	// dessine la numerotation des colonnes
 	if (display_border_ &&
 		display_columns_) {
@@ -549,34 +655,74 @@ void BorderTitleBlock::drawDxf(int width, int height, bool keep_aspect_ratio, QS
 			double xCoord = diagram_rect_.topLeft().x() +
 					(rows_header_width_ + ((i - 1) *
 					 columns_width_));
-			double yCoord = Createdxf::sheetHeight - diagram_rect_.topLeft().y() - columns_header_height_;
+			double yCoord = Createdxf::sheetHeight
+					- diagram_rect_.topLeft().y()
+					- columns_header_height_;
 			double recWidth = columns_width_;
 			double recHeight = columns_header_height_;
-			Createdxf::drawRectangle(file_path, xCoord, yCoord, recWidth, recHeight, color);
+			Createdxf::drawRectangle(file_path, xCoord, yCoord,
+						 recWidth, recHeight, color);
 			if (settings.value("border-columns_0", true).toBool()){
-			Createdxf::drawTextAligned(file_path, QString::number(i - 1), xCoord,
+			Createdxf::drawTextAligned(file_path,
-									   yCoord + recHeight*0.5, recHeight*0.7, 0, 0, 1, 2, xCoord+recWidth/2, color, 0);
+						   QString::number(i - 1),
+						   xCoord,
+						   yCoord + recHeight*0.5,
+						   recHeight*0.7,
+						   0,
+						   0,
+						   1,
+						   2,
+						   xCoord+recWidth/2,
+						   color,
+						   0);
 			}else{
-			Createdxf::drawTextAligned(file_path, QString::number(i), xCoord,
+			Createdxf::drawTextAligned(file_path,
-									   yCoord + recHeight*0.5, recHeight*0.7, 0, 0, 1, 2, xCoord+recWidth/2, color, 0);
+						   QString::number(i),
+						   xCoord,
+						   yCoord + recHeight*0.5,
+						   recHeight*0.7,
+						   0,
+						   0,
+						   1,
+						   2,
+						   xCoord+recWidth/2,
+						   color,
+						   0);
 			}
 		}
 	}
 
+	// draw line numbering
 	// dessine la numerotation des lignes
-
 	if (display_border_ && display_rows_) {
 		QString row_string("A");
 		for (int i = 1 ; i <= rows_count_ ; ++ i) {
-			double xCoord = diagram_rect_.topLeft().x() * Createdxf::xScale;
+			double xCoord = diagram_rect_.topLeft().x()
-			double yCoord = Createdxf::sheetHeight - diagram_rect_.topLeft().y()*Createdxf::yScale
+					* Createdxf::xScale;
-							- (columns_header_height_ + ((i - 1) * rows_height_))
+			double yCoord = Createdxf::sheetHeight
+					- diagram_rect_.topLeft().y()
+					*Createdxf::yScale
+					- (
+						columns_header_height_
+						+ ((i - 1)
+						   * rows_height_) )
 					- rows_height_;
 			double recWidth = rows_header_width_;
 			double recHeight = rows_height_;
-			Createdxf::drawRectangle(file_path, xCoord, yCoord, recWidth, recHeight, color);
+			Createdxf::drawRectangle(file_path, xCoord, yCoord,
-			Createdxf::drawTextAligned(file_path, row_string, xCoord,
+						 recWidth, recHeight, color);
-									   yCoord + recHeight*0.5, recWidth*0.7, 0, 0, 1, 2, xCoord+recWidth/2, color, 0);
+			Createdxf::drawTextAligned(file_path,
+						   row_string,
+						   xCoord,
+						   yCoord + recHeight*0.5,
+						   recWidth*0.7,
+						   0,
+						   0,
+						   1,
+						   2,
+						   xCoord+recWidth/2,
+						   color,
+						   0);
 			row_string = incrementLetters(row_string);
 		}
 	}
@@ -585,7 +731,10 @@ void BorderTitleBlock::drawDxf(int width, int height, bool keep_aspect_ratio, QS
 	if (display_titleblock_) {
 		//qp -> translate(titleblock_rect_.topLeft());
 		QRectF rect = titleBlockRect();
-		m_titleblock_template_renderer -> renderDxf(rect, rect.width(), file_path, color);
+		m_titleblock_template_renderer -> renderDxf(rect,
+							    rect.width(),
+							    file_path,
+							    color);
 		//qp -> translate(-titleblock_rect_.topLeft());
 	}
 
@@ -598,11 +747,17 @@ void BorderTitleBlock::drawDxf(int width, int height, bool keep_aspect_ratio, QS
 }
 
 /**
-	Permet de changer le nombre de colonnes.
+	@brief BorderTitleBlock::setColumnsCount
-	Si ce nombre de colonnes est inferieur au minimum requis, c'est ce minimum
+	Allows you to change the number of columns.
-	qui est utilise.
+	If this number of columns is less than the minimum required,
-	@param nb_c nouveau nombre de colonnes
+	it is this minimum which is used.
-	@see minNbColumns()
+
+	\~French Permet de changer le nombre de colonnes.
+	Si ce nombre de colonnes est inferieur au minimum requis,
+	c'est ce minimum qui est utilise.
+	\~ @param nb_c : new number of columns
+	\~French nouveau nombre de colonnes
+	\~ @see minNbColumns()
 */
 void BorderTitleBlock::setColumnsCount(int nb_c) {
 	if (nb_c == columnsCount()) return;
@@ -611,11 +766,17 @@ void BorderTitleBlock::setColumnsCount(int nb_c) {
 }
 
 /**
-	Change la largeur des colonnes.
+	@brief BorderTitleBlock::setColumnsWidth
-	Si la largeur indiquee est inferieure au minimum requis, c'est ce minimum
+	Change the width of the columns.
-	qui est utilise.
+	If the width indicated is less than the minimum required,
-	@param new_cw nouvelle largeur des colonnes
+	it is this minimum which is used.
-	@see minColumnsWidth()
+
+	\~French Change la largeur des colonnes.
+	Si la largeur indiquee est inferieure au minimum requis,
+	c'est ce minimum qui est utilise.
+	\~ @param new_cw : new column width
+	\~French nouvelle largeur des colonnes
+	\~ @see minColumnsWidth()
 */
 void BorderTitleBlock::setColumnsWidth(const qreal &new_cw) {
 	if (new_cw == columnsWidth()) return;
@@ -624,9 +785,14 @@ void BorderTitleBlock::setColumnsWidth(const qreal &new_cw) {
 }
 
 /**
-	Change la hauteur des en-tetes contenant les numeros de colonnes. Celle-ci
+	@brief BorderTitleBlock::setColumnsHeaderHeight
-	doit rester comprise entre 5 et 50 px.
+	Change the height of the headers containing the column numbers.
-	@param new_chh nouvelle hauteur des en-tetes de colonnes
+	This must remain between 5 and 50 px.
+
+	\~French Change la hauteur des en-tetes contenant les numeros de colonnes.
+	Celle-ci doit rester comprise entre 5 et 50 px.
+	\~ @param new_chh : new height of column headers
+	\~French nouvelle hauteur des en-tetes de colonnes
 */
 void BorderTitleBlock::setColumnsHeaderHeight(const qreal &new_chh) {
 	columns_header_height_ = qBound(qreal(5.0), new_chh, qreal(50.0));
@@ -634,11 +800,17 @@ void BorderTitleBlock::setColumnsHeaderHeight(const qreal &new_chh) {
 }
 
 /**
-	Permet de changer le nombre de lignes.
+	@brief BorderTitleBlock::setRowsCount
-	Si ce nombre de lignes est inferieur au minimum requis, cette fonction ne
+	Allows you to change the number of lines.
-	fait rien
+	If this number of lines is lower than the minimum required,
-	@param nb_r nouveau nombre de lignes
+	this function does nothing
-	@see minNbRows()
+
+	\~French Permet de changer le nombre de lignes.
+	Si ce nombre de lignes est inferieur au minimum requis,
+	cette fonction ne fait rien
+	\~ @param nb_r : new number of lines
+	\~French nouveau nombre de lignes
+	\~ @see minNbRows()
 */
 void BorderTitleBlock::setRowsCount(int nb_r) {
 	if (nb_r == rowsCount()) return;
@@ -647,11 +819,17 @@ void BorderTitleBlock::setRowsCount(int nb_r) {
 }
 
 /**
-	Change la hauteur des lignes.
+	@brief BorderTitleBlock::setRowsHeight
-	Si la hauteur indiquee est inferieure au minimum requis, c'est ce minimum
+	Change the height of the lines.
-	qui est utilise.
+	If the indicated height is lower than the minimum required,
-	@param new_rh nouvelle hauteur des lignes
+	it is this minimum which is used.
-	@see minRowsHeight()
+
+	\~French Change la hauteur des lignes.
+	Si la hauteur indiquee est inferieure au minimum requis,
+	c'est ce minimum qui est utilise.
+	\~ @param new_rh : new row height
+	\~French nouvelle hauteur des lignes
+	\~ @see minRowsHeight()
 */
 void BorderTitleBlock::setRowsHeight(const qreal &new_rh) {
 	if (new_rh == rowsHeight()) return;
@@ -660,9 +838,14 @@ void BorderTitleBlock::setRowsHeight(const qreal &new_rh) {
 }
 
 /**
-	Change la largeur des en-tetes contenant les numeros de lignes. Celle-ci
+	@brief BorderTitleBlock::setRowsHeaderWidth
-	doit rester comprise entre 5 et 50 px.
+	Change the width of the headers containing the line numbers.
-	@param new_rhw nouvelle largeur des en-tetes des lignes
+	This must remain between 5 and 50 px.
+
+	\~French Change la largeur des en-tetes contenant les numeros de lignes.
+	Celle-ci doit rester comprise entre 5 et 50 px.
+	\~ @param new_rhw : new width of line headers
+	\~French nouvelle largeur des en-tetes des lignes
 */
 void BorderTitleBlock::setRowsHeaderWidth(const qreal &new_rhw) {
 	rows_header_width_ = qBound(qreal(5.0), new_rhw, qreal(50.0));
@@ -670,19 +853,26 @@ void BorderTitleBlock::setRowsHeaderWidth(const qreal &new_rhw) {
 }
 
 /**
-	Cette methode essaye de se rapprocher le plus possible de la hauteur donnee
+	@brief BorderTitleBlock::setDiagramHeight
-	en parametre en modifiant le nombre de lignes en cours.
+	This method tries to get as close as possible to the given height
+	as a parameter by modifying the number of rows in progress.
+
+	\~French Cette methode essaye de se rapprocher le plus possible de la
+	hauteur donnee en parametre en modifiant le nombre de lignes en cours.
+	\~ @param height :
 */
 void BorderTitleBlock::setDiagramHeight(const qreal &height) {
+	//          size of rows to use = rows_height
 	// taille des lignes a utiliser = rows_height
 	setRowsCount(qRound(ceil(height / rows_height_)));
 }
 
 /**
- * @brief BorderTitleBlock::convertPosition
+	@brief BorderTitleBlock::convertPosition
- * Convert a Point in cartesian coordinate (x : 12.5, 56.9) to a point in grid coordinate (ex : B2)
+	Convert a Point in cartesian coordinate (x : 12.5, 56.9)
- * @param pos : position to convert
+	to a point in grid coordinate (ex : B2)
- * @return the converted point in grid coordinate.
+	@param pos : position to convert
+	@return the converted point in grid coordinate.
 */
 DiagramPosition BorderTitleBlock::convertPosition(const QPointF &pos)
 {
@@ -701,24 +891,24 @@ DiagramPosition BorderTitleBlock::convertPosition(const QPointF &pos)
 }
 
 /**
- * @brief BorderTitleBlock::setAuthor
+	@brief BorderTitleBlock::setAuthor
- * @param author the new value of the "Author" field
+	@param author the new value of the "Author" field
 */
 void BorderTitleBlock::setAuthor(const QString &author) {
 	btb_author_ = author;
 }
 
 /**
- * @brief BorderTitleBlock::setDate
+	@brief BorderTitleBlock::setDate
- * @param date the new value of the "Date" field
+	@param date the new value of the "Date" field
 */
 void BorderTitleBlock::setDate(const QDate &date) {
 	btb_date_ = date;
 }
 
 /**
- * @brief BorderTitleBlock::setTitle
+	@brief BorderTitleBlock::setTitle
- * @param title the new value of the "Title" field
+	@param title the new value of the "Title" field
 */
 void BorderTitleBlock::setTitle(const QString &title)
 {
@@ -730,8 +920,8 @@ void BorderTitleBlock::setTitle(const QString &title)
 }
 
 /**
- * @brief BorderTitleBlock::setFolio
+	@brief BorderTitleBlock::setFolio
- * @param folio the new value of the "Folio" field
+	@param folio the new value of the "Folio" field
 */
 void BorderTitleBlock::setFolio(const QString &folio)
 {
@@ -740,12 +930,14 @@ void BorderTitleBlock::setFolio(const QString &folio)
 }
 
 /**
-	Update the informations given to the titleblock template by regenerating a
+	@brief BorderTitleBlock::updateDiagramContextForTitleBlock
-	DiagramContext object.
+	Update the informations given to the titleblock template
-	@param initial_context Base diagram context that will be overridden by
+	by regenerating a DiagramContext object.
-	diagram-wide values
+	@param initial_context :
+	Base diagram context that will be overridden by diagram-wide values
 */
-void BorderTitleBlock::updateDiagramContextForTitleBlock(const DiagramContext &initial_context) {
+void BorderTitleBlock::updateDiagramContextForTitleBlock(
+		const DiagramContext &initial_context) {
 	// Our final DiagramContext is the initial one (which is supposed to bring
 	// project-wide properties), overridden by the "additional fields" one...
 	DiagramContext context = initial_context;
@@ -755,7 +947,8 @@ void BorderTitleBlock::updateDiagramContextForTitleBlock(const DiagramContext &i
 	
 	// ... overridden by the historical and/or dynamically generated fields
 	context.addValue("author",      btb_author_);
-	context.addValue("date",        btb_date_.toString(Qt::SystemLocaleShortDate));
+	context.addValue("date",        btb_date_.toString(
+				 Qt::SystemLocaleShortDate));
 	context.addValue("title",       btb_title_);
 	context.addValue("filename",    btb_filename_);
 	context.addValue("plant",     btb_plant_);
@@ -772,14 +965,25 @@ void BorderTitleBlock::updateDiagramContextForTitleBlock(const DiagramContext &i
 	m_titleblock_template_renderer -> setContext(context);
 }
 
+/**
+	@brief BorderTitleBlock::incrementLetters
+	increments string with Letters A to Z
+	@param string
+	@return string ++Letters
+	eg:
+	- A-> B
+	- Z -> AA
+*/
 QString BorderTitleBlock::incrementLetters(const QString &string) {
 	if (string.isEmpty()) {
 		return("A");
 	} else {
+		// separate previous digits from last digit
 		// separe les digits precedents du dernier digit
 		QString first_digits(string.left(string.count() - 1));
 		QChar last_digit(string.at(string.count() - 1));
 		if (last_digit != 'Z') {
+			// increments the last digit
 			// incremente le dernier digit
 			last_digit = last_digit.toLatin1() + 1;
 			return(first_digits + QString(last_digit));
@@ -790,17 +994,32 @@ QString BorderTitleBlock::incrementLetters(const QString &string) {
 }
 
 /**
-	@param index numero du schema (de 1 a total)
+	@brief BorderTitleBlock::setFolioData
-	@param total nombre total de schemas dans le projet
+
-	@param project_properties Project-wide properties, to be merged with diagram-wide ones.
+	\~ @param index : schema number (from 1 to total)
+	\~French numero du schema (de 1 a total)
+
+	\~ @param total : total number of diagrams in the project
+	\~French nombre total de schemas dans le projet
+
+	\~ @param autonum :
+
+	\~ @param project_properties : Project-wide properties,
+	to be merged with diagram-wide ones.
 */
-void BorderTitleBlock::setFolioData(int index, int total, const QString& autonum, const DiagramContext &project_properties) {
+void BorderTitleBlock::setFolioData(
+		int index,
+		int total,
+		const QString& autonum,
+		const DiagramContext &project_properties) {
 	if (index < 1 || total < 1 || index > total) return;
 	
+	// memorize information
 	// memorise les informations
 	folio_index_ = index;
 	folio_total_ = total;
 	
+	// regenerate the content of the folio field
 	// regenere le contenu du champ folio
 	btb_final_folio_ = btb_folio_;
 
@@ -811,61 +1030,60 @@ void BorderTitleBlock::setFolioData(int index, int total, const QString& autonum
 	btb_final_folio_.replace("%id",    QString::number(folio_index_));
 	btb_final_folio_.replace("%total", QString::number(folio_total_));
 
-
 	updateDiagramContextForTitleBlock(project_properties);
 }
 
 /**
- * @brief BorderTitleBlock::setPlant
+	@brief BorderTitleBlock::setPlant
- * @param plant the new value of the "plant" field
+	@param plant the new value of the "plant" field
 */
 void BorderTitleBlock::setPlant(const QString &plant) {
 	btb_plant_ = plant;
 }
 
 /**
- * @brief BorderTitleBlock::setLocMach
+	@brief BorderTitleBlock::setLocMach
- * @param locmach the new value of the "locmach" field
+	@param locmach the new value of the "locmach" field
 */
 void BorderTitleBlock::setLocMach(const QString &locmach) {
 	btb_locmach_ = locmach;
 }
 
 /**
- * @brief BorderTitleBlock::setIndicerev
+	@brief BorderTitleBlock::setIndicerev
- * @param indexrev the new value of the "indexrev" field
+	@param indexrev the new value of the "indexrev" field
 */
 void BorderTitleBlock::setIndicerev(const QString &indexrev) {
 	btb_indexrev_ = indexrev;
 }
 
 /**
- * @brief BorderTitleBlock::setFileName
+	@brief BorderTitleBlock::setFileName
- * @param filename the new value of the "filename" field
+	@param filename the new value of the "filename" field
 */
 void BorderTitleBlock::setFileName(const QString &filename) {
 	btb_filename_ = filename;
 }
 
 /**
- * @brief BorderTitleBlock::setVersion
+	@brief BorderTitleBlock::setVersion
- * @param version the new value of the "version" field
+	@param version the new value of the "version" field
 */
 void BorderTitleBlock::setVersion(const QString &version) {
 	btb_version_ = version;
 }
 
 /**
- * @brief BorderTitleBlock::setAutoPageNum
+	@brief BorderTitleBlock::setAutoPageNum
- * @param auto_page_num the new value of the "auto_page_num" field
+	@param auto_page_num the new value of the "auto_page_num" field
 */
 void BorderTitleBlock::setAutoPageNum(const QString &auto_page_num) {
 	btb_auto_page_num_ = auto_page_num;
 }
 
 /**
- * @brief BorderTitleBlock::setPreviousFolioNum
+	@brief BorderTitleBlock::setPreviousFolioNum
- * @param previous the new value of the "previous-folio-num" field
+	@param previous the new value of the "previous-folio-num" field
 */
 void BorderTitleBlock::setPreviousFolioNum(const QString &previous)
 {
@@ -876,8 +1094,8 @@ void BorderTitleBlock::setPreviousFolioNum(const QString &previous)
 }
 
 /**
- * @brief BorderTitleBlock::setNextFolioNum
+	@brief BorderTitleBlock::setNextFolioNum
- * @param next the new value of the "next-folio-num" field
+	@param next the new value of the "next-folio-num" field
 */
 void BorderTitleBlock::setNextFolioNum(const QString &next)
 {

sources/bordertitleblock.h

@@ -29,6 +29,7 @@ class DiagramPosition;
 class TitleBlockTemplate;
 class TitleBlockTemplateRenderer;
 /**
+	@brief The BorderTitleBlock class
 	This class represents the border and the titleblock which frame a
 	particular electric diagram.
 */
@@ -55,9 +56,11 @@ class BorderTitleBlock : public QObject
 		/// @return the columns width, in pixels
 		qreal columnsWidth() const { return(columns_width_); }
 		/// @return the total width of all columns, headers excluded
-		qreal columnsTotalWidth() const { return(columns_count_ * columns_width_); }
+		qreal columnsTotalWidth() const {
+			return(columns_count_ * columns_width_); }
 		/// @return the column headers height, in pixels
-		qreal columnsHeaderHeight() const { return(columns_header_height_); }
+		qreal columnsHeaderHeight() const {
+			return(columns_header_height_); }
 	
 		//ROWS
 		/// @return the number of rows
@@ -65,15 +68,26 @@ class BorderTitleBlock : public QObject
 		/// @return the rows height, in pixels
 		qreal rowsHeight() const { return(rows_height_); }
 		/// @return the total height of all rows, headers excluded
-		qreal rowsTotalHeight() const { return(rows_count_ * rows_height_); }
+		qreal rowsTotalHeight() const {
+			return(rows_count_ * rows_height_); }
 		/// @return la rows header width, in pixels
 		qreal rowsHeaderWidth() const { return(rows_header_width_); }
 	
 		// border - title block = diagram
-			/// @return the diagram width, i.e. the width of the border without title block
+		/**
-		qreal diagramWidth() const { return(columnsTotalWidth() + rowsHeaderWidth()); }
+			@brief diagramWidth
-			/// @return the diagram height, i.e. the height of the border without title block
+			@return the diagram width,
-		qreal diagramHeight() const { return(rowsTotalHeight() + columnsHeaderHeight()); }
+			i.e. the width of the border without title block
+		*/
+		qreal diagramWidth() const {
+			return(columnsTotalWidth() + rowsHeaderWidth()); }
+		/**
+			@brief diagramHeight
+			@return the diagram height,
+			i.e. the height of the border without title block
+		*/
+		qreal diagramHeight() const {
+			return(rowsTotalHeight() + columnsHeaderHeight()); }
 
 		QRectF titleBlockRect () const;
 
@@ -110,7 +124,8 @@ class BorderTitleBlock : public QObject
 		/// @return the value of the title block Additional Fields
 		QString version() const { return(btb_version_); }
 		/// @return the value of the title block Additional Fields
-		DiagramContext additionalFields() const { return (additional_fields_); }
+		DiagramContext additionalFields() const {
+			return (additional_fields_); }
 		/// @return the value of the title block
 		QString autoPageNum() const { return(btb_auto_page_num_); }
 		/// @return the value of the total number of folios
@@ -118,10 +133,13 @@ class BorderTitleBlock : public QObject
 	
 		// methods to get display options
 		/// @return true si le cartouche est affiche, false sinon
-		bool titleBlockIsDisplayed() const { return(display_titleblock_); }
+		bool titleBlockIsDisplayed() const {
-			/// @return true si les entetes des colonnes sont affiches, false sinon
+			return(display_titleblock_); }
+		/// @return true si les entetes des colonnes sont affiches,
+		/// false sinon
 		bool columnsAreDisplayed() const { return(display_columns_); }
-			/// @return true si les entetes des lignes sont affiches, false sinon
+		/// @return true si les entetes des lignes sont affiches,
+		/// false sinon
 		bool rowsAreDisplayed() const { return(display_rows_); }
 		/// @return true si la bordure est affichee, false sinon
 		bool borderIsDisplayed() const { return(display_border_); }
@@ -142,7 +160,8 @@ class BorderTitleBlock : public QObject
 		void setDate(const QDate &date);
 		void setTitle(const QString &title);
 		void setFolio(const QString &folio);
-		void setFolioData(int, int, const QString& = nullptr, const DiagramContext & = DiagramContext());
+		void setFolioData(int, int, const QString& = nullptr,
+				  const DiagramContext & = DiagramContext());
 		void setPlant(const QString &plant);
 		void setLocMach(const QString &locmach);
 		void setIndicerev(const QString &indexrev);
@@ -168,7 +187,9 @@ class BorderTitleBlock : public QObject
 		
 	public slots:
 		void titleBlockTemplateChanged(const QString &);
-		void titleBlockTemplateRemoved(const QString &, const TitleBlockTemplate * = nullptr);
+		void titleBlockTemplateRemoved(
+				const QString &,
+				const TitleBlockTemplate * = nullptr);
 		
 		// methods to set display options
 		void displayTitleBlock(bool);
@@ -179,21 +200,25 @@ class BorderTitleBlock : public QObject
 	
 	private:
 		void updateRectangles();
-		void updateDiagramContextForTitleBlock(const DiagramContext & = DiagramContext());
+		void updateDiagramContextForTitleBlock(
+				const DiagramContext & = DiagramContext());
 		QString incrementLetters(const QString &);
 	
 		signals:
 		/**
+			@brief borderChanged
 			Signal emitted after the border has changed
 			@param old_border Former border
 			@param new_border New border
 		*/
 		void borderChanged(QRectF old_border, QRectF new_border);
 		/**
+			@brief displayChanged
 			Signal emitted after display options have changed
 		*/
 		void displayChanged();
 		/**
+			@brief diagramTitleChanged
 			Signal emitted after the title has changed
 		*/
 		void diagramTitleChanged(const QString &);
@@ -203,48 +228,53 @@ class BorderTitleBlock : public QObject
 		*/
 		void titleBlockFolioChanged(const QString &);
 		/**
-				Signal emitted when the title block requires its data to be updated in order
+			@brief needFolioData
-				to generate the folio field.
+			Signal emitted when the title block
+			requires its data to be updated
+			in order to generate the folio field.
 		*/
 		void needFolioData();
 		/**
-				Signal emitted when this object needs to set a specific title block
+			@brief needTitleBlockTemplate
-				template. This object cannot handle the job since it does not know of
+			Signal emitted when this object needs to
-				its parent project.
+			set a specific title block template.
+			This object cannot handle the job
+			since it does not know of its parent project.
 		*/
 		void needTitleBlockTemplate(const QString &);
 	
 	// attributes
 	private:
 		// titleblock basic data
-		QString btb_author_;
+		QString btb_author_;        ///< titleblock author
-		QDate   btb_date_;
+		QDate   btb_date_;          ///< titleblock date
-		QString btb_title_;
+		QString btb_title_;         ///< titleblock title
-		QString btb_folio_;
+		QString btb_folio_;         ///< titleblock folio
-		QString btb_plant_;
+		QString btb_plant_;         ///< titleblock plant
-		QString btb_locmach_;
+		QString btb_locmach_;       ///< titleblock locmach
-		QString btb_indexrev_;
+		QString btb_indexrev_;      ///< titleblock index rev
-		QString btb_final_folio_;
+		QString btb_final_folio_;   ///< titleblock final folio
-		QString btb_auto_page_num_;
+		QString btb_auto_page_num_; ///< titleblock auto page num
-		int folio_index_;
+		int folio_index_;           ///< titleblock index
-		int folio_total_;
+		int folio_total_;           ///< titleblock total
-		QString btb_filename_;
+		QString btb_filename_;      ///< titleblock filename
-		QString btb_version_;
+		QString btb_version_;       ///< titleblock version
+		/// titleblock additional fields
 		DiagramContext additional_fields_;
-		Qt::Edge m_edge;
+		Qt::Edge m_edge;              ///< titleblock edge
-		QString m_next_folio_num,
+		QString m_next_folio_num;     ///< titleblock next folio num
-				m_previous_folio_num;
+		QString m_previous_folio_num; ///< titleblock previous folio num
 	
 		// border dimensions (rows and columns)
 		// columns: number and dimensions
-		int columns_count_;
+		int columns_count_;           ///< columns count
-		qreal columns_width_;
+		qreal columns_width_;         ///< columns width
-		qreal columns_header_height_;
+		qreal columns_header_height_; ///< columns header height
 	
 		// rows: number and dimensions
-		int rows_count_;
+		int rows_count_;          ///< rows count
-		qreal rows_height_;
+		qreal rows_height_;       ///< rows height
-		qreal rows_header_width_;
+		qreal rows_header_width_; ///< rows header width
 	
 		// title block dimensions
 		qreal titleblock_height_;

sources/conductorautonumerotation.cpp

@@ -1,5 +1,5 @@
 /*
-	Copyright 2006-2014 The QElectroTech team
+	Copyright 2006-2020 The QElectroTech team
 	This file is part of QElectroTech.
 
 	QElectroTech is free software: you can redistribute it and/or modify
@@ -26,16 +26,23 @@
 #include "assignvariables.h"
 
 /**
- * @brief ConductorAutoNumerotation::ConductorAutoNumerotation
+	@brief ConductorAutoNumerotation::ConductorAutoNumerotation
- * Constructor of autonum, after create a class, call numerate to apply the autonum.
+	Constructor of autonum, after create a class,
- * When autonum is applyed, they do with an undo command added to the stack of diagram.
+	call numerate to apply the autonum.
- * If you give a parent_undo at constructor, the undo command create in this class have parent_undo for parent,
+	When autonum is applyed,
- * and wasn't added to the stack of diagram (it's the responsabillty of the parent_undo)
+	they do with an undo command added to the stack of diagram.
- * @param conductor   : the conductor to apply automatic numerotation
+	If you give a parent_undo at constructor,
- * @param diagram     : the diagram of conductor
+	the undo command create in this class have parent_undo for parent,
- * @param parent_undo : parent undo command
+	and wasn't added to the stack of diagram
+	(it's the responsabillty of the parent_undo)
+	@param conductor   : the conductor to apply automatic numerotation
+	@param diagram     : the diagram of conductor
+	@param parent_undo : parent undo command
 */
-ConductorAutoNumerotation::ConductorAutoNumerotation(Conductor *conductor, Diagram *diagram, QUndoCommand *parent_undo) :
+ConductorAutoNumerotation::ConductorAutoNumerotation(
+		Conductor *conductor,
+		Diagram *diagram,
+		QUndoCommand *parent_undo) :
 	m_diagram      (diagram),
 	m_conductor     (conductor),
 	conductor_list (conductor -> relatedPotentialConductors().values()),
@@ -43,18 +50,21 @@ ConductorAutoNumerotation::ConductorAutoNumerotation(Conductor *conductor, Diagr
 {}
 
 /**
- * @brief ConductorAutoNumerotation::numerate
+	@brief ConductorAutoNumerotation::numerate
- * execute the automatic numerotation
+	execute the automatic numerotation
 */
 void ConductorAutoNumerotation::numerate() {
 	if (!m_conductor) return;
 	if (conductor_list.size() >= 1 ) numeratePotential();
-	else if (m_conductor -> properties().type == ConductorProperties::Multi) numerateNewConductor();
+	else if (m_conductor -> properties().type == ConductorProperties::Multi)
+		numerateNewConductor();
 }
 
 /**
- * @brief ConductorAutoNumerotation::applyText
+	@brief ConductorAutoNumerotation::applyText
- * apply the text @t to @conductor_ and all conductors at the same potential
+	apply the text @t to @conductor_
+	and all conductors at the same potential
+	@param t : Conductor text
 */
 void ConductorAutoNumerotation::applyText(const QString& t)
 {
@@ -70,20 +80,36 @@ void ConductorAutoNumerotation::applyText(const QString& t)
 
 	if (m_parent_undo)
 	{
-		new QPropertyUndoCommand(m_conductor, "properties", old_value, new_value, m_parent_undo);
+		new QPropertyUndoCommand(
+					m_conductor,
+					"properties",
+					old_value,
+					new_value,
+					m_parent_undo);
 		undo = m_parent_undo;
 	}
 	else
 	{
 		undo = new QUndoCommand();
-		new QPropertyUndoCommand(m_conductor, "properties", old_value, new_value, undo);
+		new QPropertyUndoCommand(
-		undo->setText(QObject::tr("Modifier les propriétés d'un conducteur", "undo caption"));
+					m_conductor,
+					"properties",
+					old_value,
+					new_value,
+					undo);
+		undo->setText(
+			QObject::tr(
+				"Modifier les propriétés d'un conducteur",
+				"undo caption"));
 	}
 
 	if (!conductor_list.isEmpty())
 	{
 		if (!m_parent_undo)
-			undo->setText(QObject::tr("Modifier les propriétés de plusieurs conducteurs", "undo caption"));
+			undo->setText(
+				QObject::tr(
+					"Modifier les propriétés de plusieurs conducteurs",
+					"undo caption"));
 
 		foreach (Conductor *cond, conductor_list)
 		{
@@ -91,7 +117,12 @@ void ConductorAutoNumerotation::applyText(const QString& t)
 			old_value.setValue(cp2);
 			cp2.text = t;
 			new_value.setValue(cp2);
-			new QPropertyUndoCommand(cond, "properties", old_value, new_value, undo);
+			new QPropertyUndoCommand(
+						cond,
+						"properties",
+						old_value,
+						new_value,
+						undo);
 		}
 	}
 
@@ -100,15 +131,19 @@ void ConductorAutoNumerotation::applyText(const QString& t)
 }
 
 /**
- * @brief ConductorAutoNumerotation::newProperties
+	@brief ConductorAutoNumerotation::newProperties
- * Create a new properties according to the current autonum rule of diagram
+	Create a new properties according to the current autonum rule of diagram
- * @param d
+	@param diagram : Diagram class
- * @param cp
+	@param cp : ConductorProperties
- * @param seq
+	@param seq : sequentialNumbers
 */
-void ConductorAutoNumerotation::newProperties(Diagram *diagram, ConductorProperties &cp, autonum::sequentialNumbers &seq)
+void ConductorAutoNumerotation::newProperties(
+		Diagram *diagram,
+		ConductorProperties &cp,
+		autonum::sequentialNumbers &seq)
 {	
-	NumerotationContext context = diagram->project()->conductorAutoNum(diagram->conductorsAutonumName());
+	NumerotationContext context = diagram->project()->conductorAutoNum(
+				diagram->conductorsAutonumName());
 	if (context.isEmpty()) {
 		return;
 	}
@@ -124,8 +159,8 @@ void ConductorAutoNumerotation::newProperties(Diagram *diagram, ConductorPropert
 }
 
 /**
- * @brief ConductorAutoNumerotation::numeratePotential
+	@brief ConductorAutoNumerotation::numeratePotential
- * Numerate a conductor on an existing potential
+	Numerate a conductor on an existing potential
 */
 void ConductorAutoNumerotation::numeratePotential()
 {
@@ -136,7 +171,8 @@ void ConductorAutoNumerotation::numeratePotential()
 		if (conductor->properties() != cp)
 			properties_equal = false;
 	}
-		//Every properties of the potential is equal, so we apply it to m_conductor
+	// Every properties of the potential is equal,
+	// so we apply it to m_conductor
 	if (properties_equal)
 	{
 		m_conductor->setProperties(cp);
@@ -169,21 +205,25 @@ void ConductorAutoNumerotation::numeratePotential()
 		//the texts isn't identicals
 	else
 	{
-		PotentialSelectorDialog psd(m_conductor, m_parent_undo, m_conductor->diagramEditor());
+		PotentialSelectorDialog psd(
+					m_conductor,
+					m_parent_undo,
+					m_conductor->diagramEditor());
 		psd.exec();
 	}
 }
 
 /**
- * @brief ConductorAutoNumerotation::numerateNewConductor
+	@brief ConductorAutoNumerotation::numerateNewConductor
- * create and apply a new numerotation to @m_conductor
+	create and apply a new numerotation to @m_conductor
 */
 void ConductorAutoNumerotation::numerateNewConductor()
 {
 	if (!m_conductor || m_diagram->conductorsAutonumName().isEmpty())
 		return;
 
-	NumerotationContext context = m_diagram->project()->conductorAutoNum(m_diagram -> conductorsAutonumName());
+	NumerotationContext context = m_diagram->project()->conductorAutoNum(
+				m_diagram -> conductorsAutonumName());
 	if (context.isEmpty())
 		return;
 
@@ -194,10 +234,17 @@ void ConductorAutoNumerotation::numerateNewConductor()
 	cp.m_formula = formula;
 	m_conductor->setProperties(cp);
 
-	autonum::setSequential(formula, m_conductor->rSequenceNum(), context, m_diagram, autoNum_name);
+	autonum::setSequential(formula,
+			       m_conductor->rSequenceNum(),
+			       context,
+			       m_diagram,
+			       autoNum_name);
 
 	NumerotationContextCommands ncc (context, m_diagram);
 	m_diagram->project()->addConductorAutoNum(autoNum_name, ncc.next());
 
-	applyText(autonum::AssignVariables::formulaToLabel(formula, m_conductor->rSequenceNum(), m_diagram));
+	applyText(autonum::AssignVariables::formulaToLabel(
+			  formula,
+			  m_conductor->rSequenceNum(),
+			  m_diagram));
 }

sources/conductorautonumerotation.h

@@ -1,5 +1,5 @@
 /*
-	Copyright 2006-2014 The QElectroTech team
+	Copyright 2006-2020 The QElectroTech team
 	This file is part of QElectroTech.
 
 	QElectroTech is free software: you can redistribute it and/or modify
@@ -26,16 +26,23 @@ class Conductor;
 class QUndoCommand;
 class ConductorProperties;
 
+/**
+	@brief The ConductorAutoNumerotation class
+*/
 class ConductorAutoNumerotation
 {
 	public:
 		//constructors & destructor
-		ConductorAutoNumerotation (Conductor *conductor, Diagram *diagram, QUndoCommand *undo_parent = nullptr);
+		ConductorAutoNumerotation (Conductor *conductor,
+					   Diagram *diagram,
+					   QUndoCommand *undo_parent = nullptr);
 
 		//methods
 		void        numerate       ();
 		void        applyText      (const QString&);
-		static void newProperties(Diagram *diagram, ConductorProperties &cp, autonum::sequentialNumbers &seq);
+		static void newProperties(Diagram *diagram,
+					  ConductorProperties &cp,
+					  autonum::sequentialNumbers &seq);
 
 	private:
 		//methods

sources/properties/propertiesinterface.cpp

@@ -20,3 +20,8 @@
 PropertiesInterface::PropertiesInterface()
 {
 }
+
+PropertiesInterface::~PropertiesInterface()
+{
+
+}

sources/properties/propertiesinterface.h

@@ -30,6 +30,7 @@ class PropertiesInterface
 {
 	public:
 	PropertiesInterface();
+	virtual ~PropertiesInterface();
 	// Save/load properties to setting file. QString is use for prefix a word befor the name of each paramètre
 	virtual void toSettings	  (QSettings &settings, const QString = QString()) const =0;
 	virtual void fromSettings (const QSettings &settings, const QString = QString()) =0;