From 7654add5e5acffcfc9d9eff04b1494a9b6622513 Mon Sep 17 00:00:00 2001
From: Wilson Snyder <wsnyder@wsnyder.org>
Date: Mon, 31 Oct 2011 21:39:15 -0400
Subject: [PATCH] Indicate 'exiting due to errors' if errors, not warnings.

---
 Changes         |  2 ++
 bin/verilator   |  4 ++--
 src/V3Error.cpp | 11 ++++++++---
 3 files changed, 12 insertions(+), 5 deletions(-)

diff --git a/Changes b/Changes
index 94add6886..e2ac21f46 100644
--- a/Changes
+++ b/Changes
@@ -10,6 +10,8 @@ indicates the contributor was also the author of the fix; Thanks!
 
 **** Search for user -y paths before default current directory.  [Ruben Diez]
 
+**** Indicate 'exiting due to errors' if errors, not warnings. [Ruben Diez]
+
 **** Fix reporting not found modules if generate-off, bug403. [Jeremy Bennett]
 
 
diff --git a/bin/verilator b/bin/verilator
index 3e1c42628..cc2ccde45 100755
--- a/bin/verilator
+++ b/bin/verilator
@@ -973,8 +973,8 @@ is an alias for GCC compatibility.
 
 Verilator defaults to the current directory ("-y .") and any specified
 --Mdir, though these default paths are used after any user specified
-directories.  This allows "-y `pwd`" to be used if absolute filenames are
-desired for error messages instead of relative filenames.
+directories.  This allows '-y "$(pwd)"' to be used if absolute filenames
+are desired for error messages instead of relative filenames.
 
 =back
 
diff --git a/src/V3Error.cpp b/src/V3Error.cpp
index 1b5fa2f30..c3bc4cc8e 100644
--- a/src/V3Error.cpp
+++ b/src/V3Error.cpp
@@ -332,13 +332,18 @@ void V3Error::incErrors() {
 
 void V3Error::abortIfErrors() {
     if (errorCount()) {
-	v3fatal ("Exiting due to "<<dec<<errorOrWarnCount()<<" warning(s)\n");
+	abortIfWarnings();
     }
 }
 
 void V3Error::abortIfWarnings() {
-    if (v3Global.opt.warnFatal() ? errorOrWarnCount() : errorCount()) {
-	v3fatal ("Exiting due to "<<dec<<errorOrWarnCount()<<" warning(s)\n");
+    bool exwarn = v3Global.opt.warnFatal() && warnCount();
+    if (errorCount() && exwarn) {
+	v3fatal ("Exiting due to "<<dec<<errorCount()<<" error(s), "<<warnCount()<<" warning(s)\n");
+    } else if (errorCount()) {
+	v3fatal ("Exiting due to "<<dec<<errorCount()<<" error(s)\n");
+    } else if (exwarn) {
+	v3fatal ("Exiting due to "<<dec<<warnCount()<<" warning(s)\n");
     }
 }