Details

Reviewers

PiRK

Group Reviewers

Restricted Project

Commits

rABC90bf9c092471: [lint] Fix variables shadowing python builtins in linearize-data.py

Summary

Shadowing builtins can lead to unexpected behavior. Continuing work from D13740

Test Plan

cp example-linearize.cfg linearize.cfg  # Make changes as necessary
./linearize-hashes.py linearize.cfg > hashlist.txt
./linearize-data.py linearize.cfg

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

sdulfari created this revision.Apr 20 2023, 18:35

Herald added a reviewer: Restricted Project. · View Herald TranscriptApr 20 2023, 18:35

sdulfari requested review of this revision.Apr 20 2023, 18:35

Harbormaster completed remote builds in B23386: Diff 39837.Apr 20 2023, 18:35

sdulfari added a child revision: D13751: [lint] Fix remaining variables shadowing python builtins and enable detection of shadowed variables going forward.Apr 20 2023, 19:52

PiRK accepted this revision.Apr 21 2023, 06:23

This revision is now accepted and ready to land.Apr 21 2023, 06:23

Closed by commit rABC90bf9c092471: [lint] Fix variables shadowing python builtins in linearize-data.py (authored by sdulfari). · Explain WhyApr 21 2023, 15:30

This revision was automatically updated to reflect the committed changes.

sdulfari added a commit: rABC90bf9c092471: [lint] Fix variables shadowing python builtins in linearize-data.py.

[lint] Fix variables shadowing python builtins in linearize-data.py
ClosedPublic
Actions

Details

Diff Detail

Event Timeline

Revision Contents
Changeset List

Diff 39854

contrib/linearize/linearize-data.py

[lint] Fix variables shadowing python builtins in linearize-data.pyClosedPublicActions

Details

Diff Detail

Event Timeline

Revision ContentsChangeset List

Diff 39854

contrib/linearize/linearize-data.py

[lint] Fix variables shadowing python builtins in linearize-data.py
ClosedPublic
Actions

Revision Contents
Changeset List