Skip to content

Commit 4589bf9

Browse files
authored
[clang-tidy] Fix modernize-use-std-format lit test signature (#102759)
My fix for my original fix of issue #92896 in 666d224 modified the function signature for fmt::sprintf to more accurately match the real implementation in libfmt but failed to do the same for absl::StrFormat. The latter fix applied equally well to absl::StrFormat so it's important that its test verifies that the bug is fixed too.
1 parent f070f61 commit 4589bf9

File tree

1 file changed

+2
-3
lines changed

1 file changed

+2
-3
lines changed

clang-tools-extra/test/clang-tidy/checkers/modernize/use-std-format.cpp

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -11,9 +11,8 @@
1111

1212
namespace absl
1313
{
14-
// Use const char * for the format since the real type is hard to mock up.
15-
template <typename... Args>
16-
std::string StrFormat(const char *format, const Args&... args);
14+
template <typename S, typename... Args>
15+
std::string StrFormat(const S &format, const Args&... args);
1716
} // namespace absl
1817

1918
template <typename T>

0 commit comments

Comments
 (0)