Skip to content
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

Fix wording of bit/byte in ex. 16 #311

Closed
wants to merge 1 commit into from

Conversation

pdietl
Copy link
Contributor

@pdietl pdietl commented May 26, 2023

Exercise 16 talks about storing the bits of a byte in a little-endian fashion, but then explains that little-endian means least significant byte first. In this context, it should say least significant bit.

Exercise 16 talks about storing the bits of a byte in a
little-endian fashion, but then explains that little-endian
means least significant _byte_ first. In this context, it
should say least significant _bit_.
@chrboesch
Copy link
Collaborator

'little-endian' order (least significant byte first)

Describes what is meant by 'little endian', see also: wikipedia. And in this 'way' the bits are then arranged. Perhaps this is a bit confusing, but your change would defeat the very purpose.

@pdietl
Copy link
Contributor Author

pdietl commented May 28, 2023

I’m not sure what you are saying. In that same Wikipedia article, it talks about bit order:

Bit order is a concept similar to endianness, but on a level of bits, not bytes. In serial communication, bits in a byte may be transmitted least-significant bit (lsb) or most significant bit (msb) first. RS-232, HDLC, Ethernet, and USB transmit lsb first. Teletext, I2C, SMBus, PMBus, and SONET and SDH[43] transmit msb first.ARINC 429 uses one ordering for its label field and the other ordering for the remainder of the frame.

@chrboesch chrboesch closed this in 531e9b9 Jun 9, 2023
vamega pushed a commit to vamega/ziglings that referenced this pull request Jul 25, 2023
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.

None yet

2 participants