Remove more per-file suppressions in tests.
Bug: 42271175
Change-Id: I97ac62a74a72e1985c8b604c421838a6e01e0bfd
Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/120179
Reviewed-by: Thomas Sepez <tsepez@google.com>
Commit-Queue: Tom Sepez <tsepez@chromium.org>
Reviewed-by: Lei Zhang <thestig@chromium.org>
diff --git a/xfa/fxfa/parser/xfa_utils_unittest.cpp b/xfa/fxfa/parser/xfa_utils_unittest.cpp
index fff0144..48f448d 100644
--- a/xfa/fxfa/parser/xfa_utils_unittest.cpp
+++ b/xfa/fxfa/parser/xfa_utils_unittest.cpp
@@ -2,29 +2,28 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
-#if defined(UNSAFE_BUFFERS_BUILD)
-// TODO(crbug.com/pdfium/2154): resolve buffer safety issues.
-#pragma allow_unsafe_buffers
-#endif
-
#include "xfa/fxfa/parser/xfa_utils.h"
#include <iterator>
#include "testing/gtest/include/gtest/gtest.h"
-TEST(XfaUtilsImpTest, XFA_MapRotation) {
- struct TestCase {
- int input;
- int expected_output;
- } TestCases[] = {{-1000000, 80}, {-361, 359}, {-360, 0}, {-359, 1},
- {-91, 269}, {-90, 270}, {-89, 271}, {-1, 359},
- {0, 0}, {1, 1}, {89, 89}, {90, 90},
- {91, 91}, {359, 359}, {360, 0}, {361, 1},
- {100000, 280}};
+namespace {
- for (size_t i = 0; i < std::size(TestCases); ++i) {
- EXPECT_EQ(TestCases[i].expected_output,
- XFA_MapRotation(TestCases[i].input));
+struct TestCase {
+ int input;
+ int expected_output;
+};
+
+} // namespace
+
+TEST(XfaUtilsImpTest, XFA_MapRotation) {
+ static const TestCase kTestCases[] = {
+ {-1000000, 80}, {-361, 359}, {-360, 0}, {-359, 1}, {-91, 269},
+ {-90, 270}, {-89, 271}, {-1, 359}, {0, 0}, {1, 1},
+ {89, 89}, {90, 90}, {91, 91}, {359, 359}, {360, 0},
+ {361, 1}, {100000, 280}};
+ for (const TestCase& item : kTestCases) {
+ EXPECT_EQ(item.expected_output, XFA_MapRotation(item.input));
}
}
