Docs: Add explanation about little/big endian#109841
Merged
Merged
Conversation
|
All commit authors signed the Contributor License Agreement. |
bedevere-app
Bot
added
awaiting review
docs
AA-Turner
reviewed
Sep 25, 2023
Member
AA-Turner
left a comment
AA-Turner
left a comment
There was a problem hiding this comment.
You could maybe write the byte pairs as Python byte strings. But that has cons. Also consider extracting this note into an admonition like tip or hint or info.
A
Comment on lines
+163
to
+166
| The number ``1023``, or ``0x3ff`` in hex, is represented as follows in bytes: | ||
|
|
||
| * ``0x03 0xff`` in big-endian (``>``) | ||
| * ``0xff 0x03`` in little-endian (``<``) |
Member
There was a problem hiding this comment.
You could make this a .. tip or hint?
AA-Turner
added
needs backport to 3.11
Co-authored-by: Adam Turner <9087854+AA-Turner@users.noreply.github.com>
Granjow
commented
Oct 31, 2023
Contributor
Author
There was a problem hiding this comment.
Sorry, this MR drowned in my email inbox. Better now? @AA-Turner
Contributor
hauntsaninja
left a comment
hauntsaninja
left a comment
There was a problem hiding this comment.
Thanks for the PR! It looks like the doctest is failing, maybe add the import?
|
Thanks @Granjow for the PR, and @hauntsaninja for merging it 🌮🎉.. I'm working now to backport this PR to: 3.11, 3.12. |
miss-islington
pushed a commit
to miss-islington/cpython
that referenced
this pull request
Feb 19, 2024
(cherry picked from commit 177b9cb) Co-authored-by: Simon A. Eugster <simon.eu@gmail.com> Co-authored-by: Adam Turner <9087854+AA-Turner@users.noreply.github.com> Co-authored-by: Shantanu <12621235+hauntsaninja@users.noreply.github.com>
|
GH-115646 is a backport of this pull request to the 3.12 branch. |
miss-islington
pushed a commit
to miss-islington/cpython
that referenced
this pull request
Feb 19, 2024
(cherry picked from commit 177b9cb) Co-authored-by: Simon A. Eugster <simon.eu@gmail.com> Co-authored-by: Adam Turner <9087854+AA-Turner@users.noreply.github.com> Co-authored-by: Shantanu <12621235+hauntsaninja@users.noreply.github.com>
|
GH-115647 is a backport of this pull request to the 3.11 branch. |
hauntsaninja
added a commit
that referenced
this pull request
Feb 19, 2024
…5646) Docs: Add explanation about little/big endian (GH-109841) (cherry picked from commit 177b9cb) Co-authored-by: Simon A. Eugster <simon.eu@gmail.com> Co-authored-by: Adam Turner <9087854+AA-Turner@users.noreply.github.com> Co-authored-by: Shantanu <12621235+hauntsaninja@users.noreply.github.com>
hauntsaninja
added a commit
that referenced
this pull request
Feb 19, 2024
…5647) Docs: Add explanation about little/big endian (GH-109841) (cherry picked from commit 177b9cb) Co-authored-by: Simon A. Eugster <simon.eu@gmail.com> Co-authored-by: Adam Turner <9087854+AA-Turner@users.noreply.github.com> Co-authored-by: Shantanu <12621235+hauntsaninja@users.noreply.github.com>
Contributor
Author
|
Wow, that went fast now, thanks a lot! |
woodruffw
pushed a commit
to woodruffw-forks/cpython
that referenced
this pull request
Mar 4, 2024
Co-authored-by: Adam Turner <9087854+AA-Turner@users.noreply.github.com> Co-authored-by: Shantanu <12621235+hauntsaninja@users.noreply.github.com>
diegorusso
pushed a commit
to diegorusso/cpython
that referenced
this pull request
Apr 17, 2024
Co-authored-by: Adam Turner <9087854+AA-Turner@users.noreply.github.com> Co-authored-by: Shantanu <12621235+hauntsaninja@users.noreply.github.com>
LukasWoodtli
pushed a commit
to LukasWoodtli/cpython
that referenced
this pull request
Jan 22, 2025
Co-authored-by: Adam Turner <9087854+AA-Turner@users.noreply.github.com> Co-authored-by: Shantanu <12621235+hauntsaninja@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Aims to help users like me which always have to look up which way round endianness now works.
📚 Documentation preview 📚: https://cpython-previews--109841.org.readthedocs.build/