We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fold
in fold we have heuristic:
partsCount := bits.OnesCount16(hph.afterMap[row]) switch partsCount { case 0: // Everything deleted case 1: // Leaf or extension node default: // Branch node }
need replace it by something more readable:
switch partsCount, t := hph.updateType(); t { case UpdateTypeEverythingDeleted: case NewExtension: case NewBranch: default: }
and such enum must have comments - what is it about, why only 3 values, what is "everything", etc...
The text was updated successfully, but these errors were encountered:
afterMap
unfold
e2563f8
awskii
Successfully merging a pull request may close this issue.
in
fold
we have heuristic:need replace it by something more readable:
and such enum must have comments - what is it about, why only 3 values, what is "everything", etc...
The text was updated successfully, but these errors were encountered: