tests: Fix compilation of cprinftest.
authorGabe Black <gabeblack@google.com>
Sun, 19 Nov 2017 04:53:51 +0000 (20:53 -0800)
committerGabe Black <gabeblack@google.com>
Sun, 19 Nov 2017 23:31:52 +0000 (23:31 +0000)
This test has been broken since 70176fecd1ff04 in 2014. The problem was
that the array size in the test was technically not constant because it
was based on an int variable that wasn't declared as const. That
prevented g++ from resolving it as a template parameter. Before the
change mentioned above, the implementation wasn't based on templates.

Change-Id: I6819cf522f9ba4636ac661da368b9bcbae0a813f
Reviewed-on: https://gem5-review.googlesource.com/5821
Reviewed-by: Jason Lowe-Power <jason@lowepower.com>
Maintainer: Jason Lowe-Power <jason@lowepower.com>

src/unittest/cprintftest.cc

index 7e6d2f2d46a8c0ede8ea2d6a001ea083a971850c..091d292bc6f3f94c57ce4a48a3dedcd86307d5f7 100644 (file)
@@ -45,7 +45,7 @@ main()
     cprintf("%s\n", foo);
 
     string _bar = "asdfkhasdlkfjhasdlkfhjalksdjfhalksdjhfalksdjfhalksdjhf";
-    int length = 11;
+    const int length = 11;
     char bar[length + 1];
     bar[length] = 0;