about summary refs log tree commit diff stats
diff options
context:
space:
mode:
authorKartik K. Agaram <vc@akkartik.com>2015-06-27 18:06:28 -0700
committerKartik K. Agaram <vc@akkartik.com>2015-06-27 18:06:28 -0700
commiteba30983f79e27dc3aa02a785423a96dea68d8b1 (patch)
tree2d7755e006730ac11ab38e322b4dd5c4634e30a6
parent5877ef32469e488e5634fd2374b2ae9638d36642 (diff)
downloadmu-eba30983f79e27dc3aa02a785423a96dea68d8b1.tar.gz
1669 - now it fails in the right place
..when building until layer 41
-rw-r--r--003trace.cc7
-rw-r--r--003trace.test.cc5
2 files changed, 10 insertions, 2 deletions
diff --git a/003trace.cc b/003trace.cc
index b8c1c737..97a00b1c 100644
--- a/003trace.cc
+++ b/003trace.cc
@@ -265,9 +265,12 @@ int trace_count(string layer, string line) {
   Trace_stream->newline();
   long result = 0;
   for (vector<trace_line>::iterator p = Trace_stream->past_lines.begin(); p != Trace_stream->past_lines.end(); ++p) {
-    if (layer == p->label)
-      if (line == "" || line == p->contents)
+    if (layer == p->label) {
+//?       cerr << "a: " << line << "$\n"; //? 1
+//?       cerr << "b: " << trim(p->contents) << "$\n"; //? 1
+      if (line == "" || line == trim(p->contents))
         ++result;
+    }
   }
   return result;
 }
diff --git a/003trace.test.cc b/003trace.test.cc
index 91b495a1..146f9dc2 100644
--- a/003trace.test.cc
+++ b/003trace.test.cc
@@ -51,6 +51,11 @@ void test_trace_supports_count2() {
   CHECK_EQ(trace_count("test layer 1"), 2);
 }
 
+void test_trace_count_ignores_trailing_whitespace() {
+  trace("test layer 1") << "foo\n";
+  CHECK(trace_count("test layer 1", "foo") == 1);
+}
+
 // pending: DUMP tests
 // pending: readable_contents() adds newline if necessary.
 // pending: raise also prints to stderr.