about summary refs log tree commit diff stats
diff options
context:
space:
mode:
authorKartik K. Agaram <vc@akkartik.com>2015-05-28 15:08:36 -0700
committerKartik K. Agaram <vc@akkartik.com>2015-05-28 15:08:36 -0700
commitf7b31a14a20e54229f00ff1e31bbc94dce285f4b (patch)
tree34683eb999e6f8d7283cfea57dc42a206ee8e925
parent192e52c1fce730381aaa5603ae7668796c26120d (diff)
downloadmu-f7b31a14a20e54229f00ff1e31bbc94dce285f4b.tar.gz
1505
Fix messages on failed screen tests.
-rw-r--r--072scenario_screen.cc6
1 files changed, 3 insertions, 3 deletions
diff --git a/072scenario_screen.cc b/072scenario_screen.cc
index a37318c0..7bcf46bf 100644
--- a/072scenario_screen.cc
+++ b/072scenario_screen.cc
@@ -229,19 +229,19 @@ void check_screen(const string& expected_contents, const int color) {
       // really a mismatch
       // can't print multi-byte unicode characters in warnings just yet. not very useful for debugging anyway.
       char expected_pretty[10] = {0};
-      if (curr < 256) {
+      if (curr < 256 && !iscntrl(curr)) {
         // " ('<curr>')"
         expected_pretty[0] = ' ', expected_pretty[1] = '(', expected_pretty[2] = '\'', expected_pretty[3] = static_cast<unsigned char>(curr), expected_pretty[4] = '\'', expected_pretty[5] = ')', expected_pretty[6] = '\0';
       }
       char actual_pretty[10] = {0};
-      if (Memory[addr] < 256) {
+      if (Memory[addr] < 256 && !iscntrl(Memory[addr])) {
         // " ('<curr>')"
         actual_pretty[0] = ' ', actual_pretty[1] = '(', actual_pretty[2] = '\'', actual_pretty[3] = static_cast<unsigned char>(Memory[addr]), actual_pretty[4] = '\'', actual_pretty[5] = ')', actual_pretty[6] = '\0';
       }
 
       if (Current_scenario && !Hide_warnings) {
         // genuine test in a mu file
-        raise << "\nF - " << Current_scenario->name << ": expected screen location (" << row << ", " << column << ") to contain " << curr << expected_pretty << " instead of " << Memory[addr] << actual_pretty << "'\n";
+        raise << "\nF - " << Current_scenario->name << ": expected screen location (" << row << ", " << column << ") to contain " << curr << expected_pretty << " instead of " << Memory[addr] << actual_pretty << '\n';
         dump_screen();
       }
       else {