Skip to content

Fix conclusion in Hamming approach#3096

Merged
jagdish-15 merged 2 commits intoexercism:mainfrom
kahgoh:fix/hamming-approach
Mar 3, 2026
Merged

Fix conclusion in Hamming approach#3096
jagdish-15 merged 2 commits intoexercism:mainfrom
kahgoh:fix/hamming-approach

Conversation

@kahgoh
Copy link
Member

@kahgoh kahgoh commented Mar 2, 2026

The information about one of the IntStream being more efficient just because it has less steps is incorrect because the extra steps does not result in additional iterations. In regards to the for loop, while the one shown does not show a final field, it does not mean that the code can be written in such a way that it is final.

Fixes #3090

pull request


Reviewer Resources:

Track Policies

The information about one of the IntStream being more efficient
just because it has less steps is incorrect because the extra steps
does not result in additional iterations. In regards to the `for` loop,
while the one shown does not show a `final` field, it does not mean
that the code can be written in such a way that it is final.

Fixes exercism#3090
@kahgoh kahgoh requested a review from jagdish-15 March 2, 2026 13:07
@kahgoh kahgoh added the x:rep/tiny Tiny amount of reputation label Mar 2, 2026
Link no longer used since previous commit.
@jagdish-15 jagdish-15 merged commit 03786c1 into exercism:main Mar 3, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

x:rep/tiny Tiny amount of reputation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

hamming dig deeper: wrong statement at the end

2 participants