Page MenuHomePhabricator

Correct an issue when winning "arc anoid" with certain terminal dimensions
ClosedPublic

Authored by epriestley on Jun 1 2021, 6:07 AM.
Tags
None
Referenced Files
F14405287: D21667.diff
Mon, Dec 23, 7:23 AM
Unknown Object (File)
Fri, Dec 20, 8:03 PM
Unknown Object (File)
Wed, Dec 18, 11:23 PM
Unknown Object (File)
Wed, Dec 18, 3:14 PM
Unknown Object (File)
Fri, Dec 13, 7:45 AM
Unknown Object (File)
Sat, Dec 7, 3:25 PM
Unknown Object (File)
Fri, Dec 6, 1:01 PM
Unknown Object (File)
Mon, Dec 2, 2:47 AM
Subscribers
None

Details

Summary

See PHI2085. Python 3 is stricter about integers and floats than Python 2 was, and we can end up passing a float where an integer was expected if the player wins "arc anoid" using a terminal with certain (most?) character dimensions.

Test Plan
  • Modified "arcanoid.py" to win instantly.
  • Adjusted terminal window to 80x24, ran "arc anoid", reproduced crash.
  • Ran "python2 arcanoid.py" and observed old victory animation behavior.
  • Applied patch.
  • Ran "arc anoid" and observed identical victory animation behavior.

Diff Detail

Repository
rARC Arcanist
Branch
anoid1
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 25370
Build 35028: Run Core Tests
Build 35027: arc lint + arc unit

Event Timeline

epriestley retitled this revision from Correct an issue when winning "arc anoid" with an odd terminal width to Correct an issue when winning "arc anoid" with certain terminal widths.Jun 1 2021, 6:10 AM
epriestley edited the summary of this revision. (Show Details)
epriestley edited the test plan for this revision. (Show Details)
epriestley retitled this revision from Correct an issue when winning "arc anoid" with certain terminal widths to Correct an issue when winning "arc anoid" with certain terminal dimensions.
This revision was not accepted when it landed; it landed in state Needs Review.Jun 1 2021, 6:10 AM
This revision was automatically updated to reflect the committed changes.