Skip to content

Commit 9d20633

Browse files
skirpichevvstinner
andauthored
Apply suggestions from code review
Co-authored-by: Victor Stinner <[email protected]>
1 parent bcf54f6 commit 9d20633

File tree

1 file changed

+7
-6
lines changed

1 file changed

+7
-6
lines changed

Lib/test/test_capi/test_float.py

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -189,14 +189,14 @@ def test_pack_unpack_roundtrip_for_nans(self):
189189
signaling = random.randint(0, 1)
190190
quiet = int(not signaling)
191191
if size == 8:
192-
payload = random.randint(signaling, 1<<50)
193-
i = (sign<<63) + (0x7ff<<52) + (quiet<<51) + payload
192+
payload = random.randint(signaling, 1 << 50)
193+
i = (sign << 63) + (0x7ff << 52) + (quiet << 51) + payload
194194
elif size == 4:
195-
payload = random.randint(signaling, 1<<21)
196-
i = (sign<<31) + (0xff<<23) + (quiet<<22) + payload
195+
payload = random.randint(signaling, 1 << 21)
196+
i = (sign << 31) + (0xff << 23) + (quiet << 22) + payload
197197
elif size == 2:
198-
payload = random.randint(signaling, 1<<8)
199-
i = (sign<<15) + (0x1f<<10) + (quiet<<9) + payload
198+
payload = random.randint(signaling, 1 << 8)
199+
i = (sign << 15) + (0x1f << 10) + (quiet << 9) + payload
200200
data = bytes.fromhex(f'{i:x}')
201201
for endian in (BIG_ENDIAN, LITTLE_ENDIAN):
202202
with self.subTest(data=data, size=size, endian=endian):
@@ -206,6 +206,7 @@ def test_pack_unpack_roundtrip_for_nans(self):
206206
# On this platform sNaN becomes qNaN when returned
207207
# from function. That's a known bug, e.g.
208208
# https://developercommunity.visualstudio.com/t/155064
209+
# (see also gh-130317).
209210
value = _testcapi.float_set_snan(value)
210211
data2 = pack(size, value, endian)
211212
self.assertTrue(math.isnan(value))

0 commit comments

Comments
 (0)