Skip to content

Add the word "set" to avoid misinterpretation in description#286

Merged
trekhleb merged 3 commits into
trekhleb:mergefrom
sagidM:master
Dec 20, 2020
Merged

Add the word "set" to avoid misinterpretation in description#286
trekhleb merged 3 commits into
trekhleb:mergefrom
sagidM:master

Conversation

@sagidM

@sagidM sagidM commented Jan 6, 2019

Copy link
Copy Markdown
Contributor

The sentence Powers of two in binary form always have just one bit. is a little bit confusing.
People are used to say that the number 1001 0001 is 8-bit number and it has 3 bits set.

Here is an example of use from wikipedia:
The second method uses the fact that powers of two have one and only one bit set in their binary representation:
https://en.wikipedia.org/wiki/Bit_manipulation#Example_of_bit_manipulation

@codecov

codecov Bot commented Jan 6, 2019

Copy link
Copy Markdown

Codecov Report

Merging #286 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff          @@
##           master   #286   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files         147    147           
  Lines        2590   2590           
  Branches      432    432           
=====================================
  Hits         2590   2590

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f08fc37...40cba84. Read the comment docs.

@sagidM sagidM changed the title Add word "set" to avoid misinterpretation in description Add the word "set" to avoid misinterpretation in description Jan 6, 2019
@trekhleb trekhleb changed the base branch from master to merge December 20, 2020 18:56
@trekhleb trekhleb merged commit 2794445 into trekhleb:merge Dec 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants