Commit 09f1561
authored
### Rationale for this change
This updates GoogleTest to version 1.17.0 to prevent potential undefined behavior in `ASSERT_DOUBLE_EQ` and `ASSERT_FLOAT_EQ` macros.
Prior to version 1.17, GoogleTest used union-based type punning which is undefined behavior in C++ (reading from a union member that wasn't most recently written).
See: google/googletest@59c924b
### What changes are included in this PR?
Updated `cpp/thirdparty/versions.txt`:
- `ARROW_GTEST_BUILD_VERSION`: 1.16.0 -> 1.17.0
- `ARROW_GTEST_BUILD_SHA256_CHECKSUM`: Updated to match new version
### Are these changes tested?
This is a dependency version update. Existing tests will validate compatibility.
### Are there any user-facing changes?
No. This updates GoogleTest to version 1.17.0 to prevent potential undefined behavior in ASSERT_DOUBLE_EQ and ASSERT_FLOAT_EQ macros.
Prior to version 1.17, GoogleTest used union-based type punning which is undefined behavior in C++ (reading from a union member that wasn't most recently written). This was fixed in GoogleTest commit 59c924bc.
See: google/googletest@59c924b
* GitHub Issue: #48376
Authored-by: Andrew Leng <[email protected]>
Signed-off-by: Sutou Kouhei <[email protected]>
1 parent 203437b commit 09f1561
1 file changed
+2
-2
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
76 | 76 | | |
77 | 77 | | |
78 | 78 | | |
79 | | - | |
80 | | - | |
| 79 | + | |
| 80 | + | |
81 | 81 | | |
82 | 82 | | |
83 | 83 | | |
| |||
0 commit comments