Daniel Carvalho has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/47819 )

Change subject: base: Remove DPRINTF_UNCONDITIONAL test
......................................................................

base: Remove DPRINTF_UNCONDITIONAL test

This macro has been marked as deprecated [1]. To avoid
having warnings popping up on every run of the unit tests,
assume that this macro will no longer be modified, and
remove the respective tests.

[1] https://gem5-review.googlesource.com/c/public/gem5/+/44987

Change-Id: I1800440ab545b0acb83ac9c1cf20c19400242d3f
Signed-off-by: Daniel R. Carvalho <oda...@yahoo.com.br>
---
M src/base/trace.test.cc
1 file changed, 0 insertions(+), 28 deletions(-)



diff --git a/src/base/trace.test.cc b/src/base/trace.test.cc
index 526e8dd..491a826 100644
--- a/src/base/trace.test.cc
+++ b/src/base/trace.test.cc
@@ -556,34 +556,6 @@
 #endif
 }

-/** Test DPRINTF_UNCONDITIONAL with tracing on. */
-TEST(TraceTest, MacroDPRINTF_UNCONDITIONAL)
-{
-    StringWrap name("Foo");
-
-    // Flag enabled
-    Trace::enable();
-    EXPECT_TRUE(debug::changeFlag("TraceTestDebugFlag", true));
-    EXPECT_TRUE(debug::changeFlag("FmtFlag", true));
-    DPRINTF_UNCONDITIONAL(TraceTestDebugFlag, "Test message");
-#if TRACING_ON
-    ASSERT_EQ(getString(Trace::output()),
-        "      0: TraceTestDebugFlag: Foo: Test message");
-#else
-    ASSERT_EQ(getString(Trace::output()), "");
-#endif
-
-    // Flag disabled
-    Trace::disable();
-    EXPECT_TRUE(debug::changeFlag("TraceTestDebugFlag", false));
-    DPRINTF_UNCONDITIONAL(TraceTestDebugFlag, "Test message");
-#if TRACING_ON
-    ASSERT_EQ(getString(Trace::output()), "      0: Foo: Test message");
-#else
-    ASSERT_EQ(getString(Trace::output()), "");
-#endif
-}
-
 /**
  * Test that there is a global name() to fall through when a locally scoped
  * name() is not defined.

--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/47819
To unsubscribe, or for help writing mail filters, visit https://gem5-review.googlesource.com/settings

Gerrit-Project: public/gem5
Gerrit-Branch: develop
Gerrit-Change-Id: I1800440ab545b0acb83ac9c1cf20c19400242d3f
Gerrit-Change-Number: 47819
Gerrit-PatchSet: 1
Gerrit-Owner: Daniel Carvalho <oda...@yahoo.com.br>
Gerrit-MessageType: newchange
_______________________________________________
gem5-dev mailing list -- gem5-dev@gem5.org
To unsubscribe send an email to gem5-dev-le...@gem5.org
%(web_page_url)slistinfo%(cgiext)s/%(_internal_name)s

Reply via email to