| commit | 2151138b0f679d00f0328bd743d6b3548ac28fbc | |
| author | Haran Talmon <haranp@meta.com> | |
| Tue, 15 Nov 2022 06:32:53 +0000 (14 22:32 -0800) | ||
| committer | Facebook GitHub Bot <facebook-github-bot@users.noreply.github.com> | |
| Tue, 15 Nov 2022 06:32:53 +0000 (14 22:32 -0800) | ||
| tree | 3f7a6484dbcad66390ff8a57cc69bc4b8e96dde8 | treesnapshot (tar.gz zip) |
| parent | f7e13a45e45ec0855af01a93b6b6dd7925570588 | commitdiff |
Fix a signed/unsigned compare warning.
Summary:
MSVC2017 with /W4 complains about a signed/unsigned mismatch when destroying
a `ConcurrentHashMap`, due to the `releaseLink` invocation which compares
`count` (an unsigned value) with 1 (signed). Fix this by using `1u` explicitly
instead. Also fix `acquireLink` while I'm at it.
Reviewed By: Orvid
Differential Revision: D41260748
fbshipit-source-id: 0b66bc1ea5622c8841a237cf7ee84a6835a775b4
Summary:
MSVC2017 with /W4 complains about a signed/unsigned mismatch when destroying
a `ConcurrentHashMap`, due to the `releaseLink` invocation which compares
`count` (an unsigned value) with 1 (signed). Fix this by using `1u` explicitly
instead. Also fix `acquireLink` while I'm at it.
Reviewed By: Orvid
Differential Revision: D41260748
fbshipit-source-id: 0b66bc1ea5622c8841a237cf7ee84a6835a775b4
| third-party/folly/src/folly/concurrency/detail/ConcurrentHashMap-detail.h | diffblobblamehistory |