Hacker News
new
|
past
|
comments
|
ask
|
show
|
jobs
|
submit
login
nsteel
on Aug 11, 2023
|
parent
|
context
|
favorite
| on:
Source code for Quake 2 rerelease
Pretty sure your line 3 and the original lines 3-5 are not equivalent. Check it with `i` entering the loop with value `maxclients->value`.
The best reason not to re-write for style reasons is you break it.
augustk
on Aug 11, 2023
[–]
Thanks, now corrected.
nsteel
on Aug 11, 2023
|
parent
[–]
If `i` can enter with value greater than `maxclients->value` (why not), it's still wrong.
Guidelines
|
FAQ
|
Lists
|
API
|
Security
|
Legal
|
Apply to YC
|
Contact
Search:
The best reason not to re-write for style reasons is you break it.