Merge bitcoin#25888: refactor: use strprintf for creating unknown-service-flag string

MacroFake · MacroFake · commit 027b6729bce7 · 2022-08-22T08:26:16.000+02:00
706c8e0 refactor: use `strprintf` for creating unknown-service-flag string (Sebastian Falbesoner) Pull request description: No need to use a stringstream here. The trivial change can be verified by running the functional test `rpc_net.py`: https://github.com/bitcoin/bitcoin/blob/c73c8d53fe27956faacb3e8ca0e94adf071de6ec/test/functional/rpc_net.py#L181-L184 As far as I could tell, this is the only instace left where we used `std::ostringstream` for the creation of simple strings (in `FormatSubVersion` using a stream makes sense since the number of placeholders is not constant). ACKs for top commit: kristapsk: ACK 706c8e0 Tree-SHA512: 069cea29aef03996ae16a0dc3ed87b1b2cf2ab0bf5987c225b10da12d0f4b62b7c3faf3a169c0b912eb2ad60c6ea0a09a622be7eaadad78cee0463ef4ffc0e19
diff --git a/src/protocol.cpp b/src/protocol.cpp
@@ -199,12 +199,7 @@ static std::string serviceFlagToStr(size_t bit)
     // Not using default, so we get warned when a case is missing
     }
 
-    std::ostringstream stream;
-    stream.imbue(std::locale::classic());
-    stream << "UNKNOWN[";
-    stream << "2^" << bit;
-    stream << "]";
-    return stream.str();
+    return strprintf("UNKNOWN[2^%u]", bit);
 }
 
 std::vector<std::string> serviceFlagsToStr(uint64_t flags)

Original file line number	Diff line number	Diff line change
`@@ -199,12 +199,7 @@ static std::string serviceFlagToStr(size_t bit)`
`199`	`199`	`// Not using default, so we get warned when a case is missing`
`200`	`200`	`}`
`201`	`201`
`202`		`- std::ostringstream stream;`
`203`		`- stream.imbue(std::locale::classic());`
`204`		`- stream << "UNKNOWN[";`
`205`		`- stream << "2^" << bit;`
`206`		`- stream << "]";`
`207`		`- return stream.str();`
	`202`	`+ return strprintf("UNKNOWN[2^%u]", bit);`
`208`	`203`	`}`
`209`	`204`
`210`	`205`	`std::vector<std::string> serviceFlagsToStr(uint64_t flags)`