From 56ffa78ce34095bae941f3cb95e25bdc3dc309ee Mon Sep 17 00:00:00 2001 From: Rasesh Patel Date: Mon, 10 Jun 2019 15:07:41 -0700 Subject: [PATCH] The order of the operators in the struct should match the order of the test cases This makes it a bit easier to follow the logic for anyone reading the tests as docs. --- absl/types/variant_test.cc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/absl/types/variant_test.cc b/absl/types/variant_test.cc index d70248276..2efaf21e4 100644 --- a/absl/types/variant_test.cc +++ b/absl/types/variant_test.cc @@ -1793,8 +1793,8 @@ TEST(VariantTest, VisitSimple) { EXPECT_EQ("B", piece); struct StrLen { - int operator()(const std::string& s) const { return s.size(); } int operator()(const char* s) const { return strlen(s); } + int operator()(const std::string& s) const { return s.size(); } }; v = "SomeStr";