From 3052d2c85412fba525efcc3e3cda2cac4426ff4b Mon Sep 17 00:00:00 2001
From: Clinton Stimpson <clinton@elemtech.com>
Date: Mon, 14 Apr 2008 16:15:28 -0400
Subject: [PATCH] BUG:  Fix issue when non-error messages were incorrectly
 colored red.

---
 Source/QtDialog/CMakeSetupDialog.cxx | 26 ++++++--------------------
 Source/QtDialog/CMakeSetupDialog.h   |  3 +++
 2 files changed, 9 insertions(+), 20 deletions(-)

diff --git a/Source/QtDialog/CMakeSetupDialog.cxx b/Source/QtDialog/CMakeSetupDialog.cxx
index 1c5e25d47..56ee7ac73 100644
--- a/Source/QtDialog/CMakeSetupDialog.cxx
+++ b/Source/QtDialog/CMakeSetupDialog.cxx
@@ -138,6 +138,7 @@ CMakeSetupDialog::CMakeSetupDialog()
   // fixed pitch font in output window
   QFont outputFont("Courier");
   this->Output->setFont(outputFont);
+  this->ErrorFormat.setForeground(QBrush(Qt::red));
 
   // start the cmake worker thread
   this->CMakeThread = new QCMakeThread(this);
@@ -512,32 +513,17 @@ void CMakeSetupDialog::showProgress(const QString& /*msg*/, float percent)
 {
   this->ProgressBar->setValue(qRound(percent * 100));
 }
-  
+
 void CMakeSetupDialog::error(const QString& message)
 {
-  QStringList messages = message.split('\n');
-  foreach(QString m, messages)
-    {
-    // make sure we escape html tags in the cmake messages
-    m.replace(QString("&"), QString("&amp;"));
-    m.replace(QString("<"), QString("&lt;"));
-    m.replace(QString(">"), QString("&gt;"));
-    m.replace(QString(" "), QString("&nbsp;"));
-    this->Output->append(QString("<b><font color=red>%1</font></b>").arg(m));
-    }
+  this->Output->setCurrentCharFormat(this->ErrorFormat);
+  this->Output->append(message);
 }
 
 void CMakeSetupDialog::message(const QString& message)
 {
-  QStringList messages = message.split('\n');
-  foreach(QString m, messages)
-    {
-    // make sure we escape html tags in the cmake messages
-    m.replace(QString("&"), QString("&amp;"));
-    m.replace(QString("<"), QString("&lt;"));
-    m.replace(QString(">"), QString("&gt;"));
-    this->Output->append(m);
-    }
+  this->Output->setCurrentCharFormat(this->MessageFormat);
+  this->Output->append(message);
 }
 
 void CMakeSetupDialog::setEnabledState(bool enabled)
diff --git a/Source/QtDialog/CMakeSetupDialog.h b/Source/QtDialog/CMakeSetupDialog.h
index 211dfcb02..c942d154e 100644
--- a/Source/QtDialog/CMakeSetupDialog.h
+++ b/Source/QtDialog/CMakeSetupDialog.h
@@ -99,6 +99,9 @@ protected:
   QAction* InstallForCommandLineAction;
   State CurrentState;
 
+  QTextCharFormat ErrorFormat;
+  QTextCharFormat MessageFormat;
+
 };
 
 // QCMake instance on a thread