Ameliorations mineures sur les messages de debug, la documentation, et quelques noms de variables.

git-svn-id: svn+ssh://svn.tuxfamily.org/svnroot/qet/qet/trunk@586 bfdf4180-ca20-0410-9c96-a3a8aa849046

sources/diagram.cpp

@@ -78,7 +78,6 @@ Diagram::~Diagram() {
 	foreach(QGraphicsItem *qgi_d, deletable_items) {
 		delete qgi_d;
 	}
-	// qDebug() << "Suppression du schema" << ((void*)this);
 }
 
 /**
@@ -440,11 +439,11 @@ bool Diagram::fromXml(QDomElement &document, QPointF position, bool consider_inf
 		
 		CustomElement *nvel_elmt = new CustomElement(element_location);
 		if (nvel_elmt -> isNull()) {
-			QString debug_message = QString("Le chargement de la description de l'element %1 a echoue avec le code d'erreur %2").arg(element_location.path()).arg(nvel_elmt -> state());
+			QString debug_message = QString("Diagram::fromXml() : Le chargement de la description de l'element %1 a echoue avec le code d'erreur %2").arg(element_location.path()).arg(nvel_elmt -> state());
-			qDebug() << debug_message;
+			qDebug() << qPrintable(debug_message);
 			delete nvel_elmt;
 			
-			qDebug() << "Utilisation d'un GhostElement en lieu et place de cet element.";
+			qDebug() << "Diagram::fromXml() : Utilisation d'un GhostElement en lieu et place de cet element.";
 			nvel_elmt = new GhostElement(element_location);
 		}
 		
@@ -455,7 +454,7 @@ bool Diagram::fromXml(QDomElement &document, QPointF position, bool consider_inf
 			added_elements << nvel_elmt;
 		} else {
 			delete nvel_elmt;
-			qDebug("Le chargement des parametres d'un element a echoue");
+			qDebug() << "Diagram::fromXml() : Le chargement des parametres d'un element a echoue";
 		}
 	}
 	
@@ -524,7 +523,7 @@ bool Diagram::fromXml(QDomElement &document, QPointF position, bool consider_inf
 					added_conductors << c;
 				}
 			}
-		} else qDebug() << "Le chargement du conducteur" << id_p1 << id_p2 << "a echoue";
+		} else qDebug() << "Diagram::fromXml() : Le chargement du conducteur" << id_p1 << id_p2 << "a echoue";
 	}
 	
 	// remplissage des listes facultatives
@@ -639,7 +638,6 @@ void Diagram::addDiagramTextItem(DiagramTextItem *dti) {
 	);
 }
 
-
 /**
 	Enleve un element du schema
 	@param element Element a enlever
@@ -754,7 +752,9 @@ QList<CustomElement *> Diagram::customElements() const {
 	return(elements_list);
 }
 
-/// oublie la liste des elements et conducteurs en mouvement
+/**
+	Oublie la liste des elements et conducteurs en mouvement
+*/
 void Diagram::invalidateMovedElements() {
 	if (!moved_elements_fetched) return;
 	moved_elements_fetched = false;
@@ -764,7 +764,9 @@ void Diagram::invalidateMovedElements() {
 	texts_to_move.clear();
 }
 
-/// reconstruit la liste des elements et conducteurs en mouvement
+/**
+	Reconstruit la liste des elements et conducteurs en mouvement
+*/
 void Diagram::fetchMovedElements() {
 	// recupere les elements deplaces
 	foreach (QGraphicsItem *item, selectedItems()) {

sources/diagramview.cpp

@@ -275,7 +275,9 @@ void DiagramView::paste(const QPointF &pos, QClipboard::Mode clipboard_mode) {
 	}
 }
 
-/// Colle le contenu du presse-papier sur le schema a la position de la souris
+/**
+	Colle le contenu du presse-papier sur le schema a la position de la souris
+*/
 void DiagramView::pasteHere() {
 	paste(mapToScene(paste_here_pos));
 }
@@ -527,9 +529,9 @@ bool DiagramView::mustIntegrateElement(const ElementsLocation &location) const {
 */
 bool DiagramView::addElementAtPos(const ElementsLocation &location, const QPoint &pos) {
 	// construit une instance de l'element correspondant a l'emplacement
-	int etat;
+	int state;
-	Element *el = new CustomElement(location, 0, 0, &etat);
+	Element *el = new CustomElement(location, 0, 0, &state);
-	if (etat) {
+	if (state) {
 		delete el;
 		return(false);
 	}
@@ -738,7 +740,9 @@ void DiagramView::contextMenuEvent(QContextMenuEvent *e) {
 	e -> accept();
 }
 
-/// @return l'editeur de schemas parent ou 0
+/**
+	@return l'editeur de schemas parent ou 0
+*/
 QETDiagramEditor *DiagramView::diagramEditor() const {
 	// remonte la hierarchie des widgets
 	QWidget *w = const_cast<DiagramView *>(this);
@@ -817,7 +821,7 @@ void DiagramView::addDroppedElement() {
 		QString integ_path = scene -> project() -> integrateElement(location.toString(), integ_handler, error_msg);
 		delete integ_handler;
 		if (integ_path.isEmpty()) {
-			qDebug() << error_msg;
+			qDebug() << "DiagramView::addDroppedElement : Impossible d'ajouter l'element. Motif : " << qPrintable(error_msg);
 			return;
 		}
 		addElementAtPos(ElementsLocation::locationFromString(integ_path), pos);

sources/elementscategorieslist.cpp

@@ -138,10 +138,10 @@ QTreeWidgetItem *ElementsCategoriesList::addCategory(QTreeWidgetItem *qtwi_paren
 	@return Le QTreeWidgetItem insere
 */
 QTreeWidgetItem *ElementsCategoriesList::addElement(QTreeWidgetItem *qtwi_parent, ElementDefinition *element, const QString &elmt_name, const QIcon &icon) {
-	int etat;
+	int state;
-	CustomElement custom_elmt(element -> xml(), 0, 0, &etat);
+	CustomElement custom_elmt(element -> xml(), 0, 0, &state);
-	if (etat) {
+	if (state) {
-		qDebug() << "Le chargement du composant" << element -> location().toString() << "a echoue avec le code d'erreur" << etat;
+		qDebug() << "ElementsCategoriesList::addElement() : Le chargement du composant" << qPrintable(element -> location().toString()) << "a echoue avec le code d'erreur" << state;
 		return(0);
 	}
 	QString final_name(elmt_name.isEmpty() ? custom_elmt.name() : elmt_name);

sources/elementspanel.cpp

@@ -594,10 +594,10 @@ QTreeWidgetItem *ElementsPanel::addElement(QTreeWidgetItem *qtwi_parent, Element
 	
 	QString whats_this = tr("Ceci est un \351l\351ment que vous pouvez ins\351rer dans votre sch\351ma par cliquer-d\351placer");
 	QString tool_tip = tr("Cliquer-d\351posez cet \351l\351ment sur le sch\351ma pour ins\351rer un \351l\351ment ");
-	int etat;
+	int state;
-	CustomElement custom_elmt(element -> location(), 0, 0, &etat);
+	CustomElement custom_elmt(element -> location(), 0, 0, &state);
-	if (etat) {
+	if (state) {
-		qDebug() << "Le chargement du composant" << element -> location().toString() << "a echoue avec le code d'erreur" << etat;
+		qDebug() << "ElementsCategoriesList::addElement() : Le chargement du composant" << qPrintable(element -> location().toString()) << "a echoue avec le code d'erreur" << state;
 		return(0);
 	}
 	QString final_name(elmt_name.isEmpty() ? custom_elmt.name() : elmt_name);
@@ -665,7 +665,6 @@ void ElementsPanel::reload(bool reload_collections) {
 	}
 }
 
-
 /**
 	Gere le double-clic sur un element.
 	Si un double-clic sur un projet est effectue, le signal requestForProject

sources/qetproject.cpp

@@ -324,12 +324,6 @@ QDomDocument QETProject::toXml() {
 	Ferme le projet
 */
 bool QETProject::close() {
-	
-	/// @todo demander son avis a l'utilisateur
-	/*foreach(Diagram *diagram, diagrams_) {
-		QTextStream err(stderr);
-		err << diagram -> writtenXml().toString(4);
-	}*/
 	return(true);
 }
 

sources/qetsingleapplication.cpp

@@ -41,8 +41,7 @@ QETSingleApplication::QETSingleApplication(int &argc, char **argv, const QString
 		
 		// initialisation du segment de memoire partage
 		if (!shared_memory_.create(1)) {
-			qDebug() << "Impossible de cr\351er l'instance unique.";
+			qDebug() << "QETSingleApplication::QETSingleApplication() : Impossible de cr\351er l'instance unique" << qPrintable(unique_key_);
-			qDebug() << "Used key is:" << unique_key_;
 			return;
 		}
 		
@@ -68,8 +67,7 @@ QETSingleApplication::~QETSingleApplication() {
 void QETSingleApplication::receiveMessage() {
 	QLocalSocket *local_socket = local_server_ -> nextPendingConnection();
 	if (!local_socket -> waitForReadyRead(timeout_)) {
-		qDebug() << local_socket -> errorString().toLatin1();
+		qDebug() << "QETSingleApplication::receiveMessage() :" << qPrintable(local_socket -> errorString()) << "(" << qPrintable(unique_key_) << ")";
-		qDebug() << "Used key is:" << unique_key_;
 		return;
 	}
 	QByteArray byteArray = local_socket -> readAll();
@@ -101,16 +99,14 @@ bool QETSingleApplication::sendMessage(const QString &message) {
 	QLocalSocket local_socket(this);
 	local_socket.connectToServer(unique_key_, QIODevice::WriteOnly);
 	if (!local_socket.waitForConnected(timeout_)) {
-		qDebug() << local_socket.errorString().toLatin1();
+		qDebug() << "QETSingleApplication::sendMessage() :" << qPrintable(local_socket.errorString()) << "(" << qPrintable(unique_key_) << ")";
-		qDebug() << "Used key is:" << unique_key_;
 		return(false);
 	}
 	
 	// envoi du message, avec gestion du timeout
 	local_socket.write(message.toUtf8());
 	if (!local_socket.waitForBytesWritten(timeout_)) {
-		qDebug() << local_socket.errorString().toLatin1();
+		qDebug() << "QETSingleApplication::sendMessage() :" << qPrintable(local_socket.errorString()) << "(" << qPrintable(unique_key_) << ")";
-		qDebug() << "Used key is:" << unique_key_;
 		return(false);
 	}
 	

sources/recentfiles.cpp

@@ -18,6 +18,13 @@
 #include "recentfiles.h"
 #include "qetapp.h"
 
+/**
+	Constructeur
+	@param identifier prefixe a utiliser pour recuperer les fichiers recents
+	dans la configuration de l'application
+	@param size Nombre de fichiers recents a retenir
+	@param parent QObject parent
+*/
 RecentFiles::RecentFiles(const QString &identifier, int size, QObject *parent) :
 	QObject(parent),
 	identifier_(identifier.isEmpty() ? "unnamed" : identifier),
@@ -85,7 +92,7 @@ void RecentFiles::fileWasOpened(const QString &filepath) {
 }
 
 /**
-	lit la liste des fichiers recents dans la configuration
+	Lit la liste des fichiers recents dans la configuration
 */
 void RecentFiles::extractFilesFromSettings() {
 	// oublie la liste des fichiers recents
@@ -118,7 +125,7 @@ void RecentFiles::insertFile(const QString &filepath) {
 }
 
 /**
-	ecrit la liste des fichiers recents dans la configuration
+	Ecrit la liste des fichiers recents dans la configuration
 */
 void RecentFiles::saveFilesToSettings() {
 	for (int i = 0 ; i < size_ && i < list_.count() ; ++ i) {