-
Notifications
You must be signed in to change notification settings - Fork 23
New issue
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
Add suport for unsorted normalization of addresses #128
Conversation
Signed-off-by: Daniel Müller <[email protected]>
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## master #128 +/- ##
==========================================
- Coverage 69.71% 69.11% -0.61%
==========================================
Files 23 23
Lines 4346 4397 +51
==========================================
+ Hits 3030 3039 +9
- Misses 1316 1358 +42
☔ View full report in Codecov by Sentry. |
e8c629c
to
820fef8
Compare
Rebased. |
820fef8
to
71bf445
Compare
Please prioritize reviewing this logic over #129, as it will have to land first. |
71bf445
to
2e04517
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
heh, another leetcode-like solution comes to the rescue :)
Introduce normalize_user_addrs_sorted_impl() as an implementation detail of normalize_user_addrs_sorted() that works on an iterator. We will later need this function to implement normalization of unsorted addresses. Signed-off-by: Daniel Müller <[email protected]>
This change introduces another normalization function variant that works on unsorted addresses. Signed-off-by: Daniel Müller <[email protected]>
2e04517
to
56fd233
Compare
Please see individual commits for descriptions.