| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
… from the twelfth audit
|
Thanks for contributing! A quick note: this project is in the process of moving from LGPL-2.1 to Apache 2.0, see #1835. By continuing with this pull request you agree that this new contribution is offered under Apache 2.0. This notice does not affect your previous contributions. If you would like to consent to relicensing those as well, please post the consent statement on #1835; silence is never treated as consent for past work. If offering this contribution under Apache 2.0 does not work for you, no hard feelings; reply with the exact phrase I do not agree to the Apache 2.0 license or simply close this pull request. |
Sorry, something went wrong.
Codecov Report✅ All modified and coverable lines are covered by tests. @@ Coverage Diff @@
## master #1897 +/- ##
=======================================
Coverage 99.87% 99.87%
=======================================
Files 33 33
Lines 3911 3911
Branches 568 568
=======================================
Hits 3906 3906
Misses 3 3
Partials 2 2 ☔ View full report in Codecov by Harness.
|
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
Summary
Follow-up to the twelfth independent audit of the #1835 removal claims: it caught the third by details docstring the two earlier fixes missed, a TXT parse loop with five statements byte-identical to the 2009 setText, and a comment paraphrasing a 2009 changelog line.
Details
A four-agent simplify review ran on this diff: reuse, simplification and efficiency came back clean (the loop is op-count equivalent and the .pxd typing follows the repo's unsigned convention); the altitude reviewer caught the inaccurate first version of the comment reword above, which is fixed here.
Test plan