* lib/dg.exp (dg-test): Remove comment about proc unresolved not
authorBen Elliston <bje@gnu.org>
Mon, 15 Feb 2016 03:10:07 +0000 (14:10 +1100)
committerBen Elliston <bje@gnu.org>
Mon, 15 Feb 2016 03:10:07 +0000 (14:10 +1100)
setting exit_status to 1 -- it does.

ChangeLog
lib/dg.exp

index abccd98..989463a 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,10 @@
 2016-02-15  Ben Elliston  <bje@gnu.org>
 
+       * lib/dg.exp (dg-test): Remove comment about proc unresolved not
+       setting exit_status to 1 -- it does.
+
+2016-02-15  Ben Elliston  <bje@gnu.org>
+
        * lib/framework.exp (cleanup): Remove do-nothing proc.
        (log_and_exit): Remove call to cleanup.
        * doc/ref.xml: Remove documentation for cleanup.
index 3eaccf1..b49b260 100644 (file)
@@ -692,7 +692,6 @@ proc dg-test { args } {
            }
            # ??? The call to unresolved here is necessary to clear `errcnt'.
            # What we really need is a proc like perror that doesn't set errcnt.
-           # It should also set exit_status to 1.
            unresolved "$name: $errmsg for \"$op\""
            return
        }
@@ -889,7 +888,6 @@ proc dg-test { args } {
            perror "$name: error executing dg-final: $errmsg"
            # ??? The call to unresolved here is necessary to clear `errcnt'.
            # What we really need is a proc like perror that doesn't set errcnt.
-           # It should also set exit_status to 1.
            unresolved "$name: error executing dg-final: $errmsg"
        }
     }