aiorepl: Optimize sys.stdout.write calls. - #1148
Conversation
a6dace9 to
a5484e3
Compare
|
Thanks for the PR! You can maybe pick a more readable name than |
dcec65b to
0c256a9
Compare
|
@agatti - done. I had tried to follow what was here:
... with Regardless, confirmed - yes, the output size remains the same at 2,955 bytes. |
|
Thanks! That naming scheme can work for global entities (eg. classes, functions, variables), arguments in function declarations, and class members. Those names are retained in the MPY file once compiled, but local variable names get discarded when compiled. |
Understood, but everything we're talking about here is all at the same scope. :-) So maybe those should be renamed as well... |
|
Point taken, then how about Oh, and feel free to squash things into one single commit while you're at it. Still, that's a nice change you've made there - I wonder where else in |
0c256a9 to
cca9257
Compare
|
@agatti - agreed, updated, and squashed. |
|
Great, thank you! |
- aiorepl.mpy: 3,152 -> 2,955 bytes (-197 bytes, -6.3%) - Bump patch version to 0.2.4. Signed-off-by: Mark A. Ziesemer <online@mark.ziesemer.com>
cca9257 to
df05604
Compare
Summary
aiorepl.mpy: 3,152 -> 2,955 bytes (-197 bytes, -6.3%)Testing
Tested on RP2.
Generative AI
I did not use generative AI tools when creating this PR.