doctest: Do not use unnamed class

kraj · kraj · commit 2189c3ada692 · 2023-04-17T16:52:31.000-07:00
When compiling with clang this results in emitting absolute path into
debug info, especially .debug_str section has

~(unnamed class at /mnt/b/yoe/master/build/tmp/work/core2-64-yoe-linux-musl/libcereal/1.3.2+gitAUTOINC+ebef1e9298-r0/git/unittests/doctest.h:6428:5)
(unnamed class at /mnt/b/yoe/master/build/tmp/work/core2-64-yoe-linux-musl/libcereal/1.3.2+gitAUTOINC+ebef1e9298-r0/git/unittests/doctest.h:6428:5)

Therefore use a name for the class instead and help get rid of this
absolute path in debug info. This fixes

File /usr/lib/libcereal/ptest/tests/.debug/test_unordered_map in package libcereal-dbg contains reference to TMPDIR

Signed-off-by: Khem Raj &lt;raj.khem@gmail.com&gt;
diff --git a/unittests/doctest.h b/unittests/doctest.h
@@ -6424,7 +6424,7 @@ void Context::setCout(std::ostream* out) { p->cout = out; }
 static class DiscardOStream : public std::ostream
 {
 private:
-    class : public std::streambuf
+    class discardBufStream: public std::streambuf
     {
     private:
         // allowing some buffering decreases the amount of calls to overflow

Original file line number	Diff line number	Diff line change
`@@ -6424,7 +6424,7 @@ void Context::setCout(std::ostream* out) { p->cout = out; }`
`6424`	`6424`	`static class DiscardOStream : public std::ostream`
`6425`	`6425`	`{`
`6426`	`6426`	`private:`
`6427`		`- class : public std::streambuf`
	`6427`	`+ class discardBufStream: public std::streambuf`
`6428`	`6428`	`{`
`6429`	`6429`	`private:`
`6430`	`6430`	`// allowing some buffering decreases the amount of calls to overflow`