gh-146431: Add the wrapcol and ignorechars parameters for more Base XX codec by serhiy-storchaka · Pull Request #146434 · python/cpython · GitHub
Skip to content

gh-146431: Add the wrapcol and ignorechars parameters for more Base XX codec#146434

Merged
serhiy-storchaka merged 2 commits into
python:mainfrom
serhiy-storchaka:basexx-wrapcol-ignorechars
Apr 1, 2026
Merged

gh-146431: Add the wrapcol and ignorechars parameters for more Base XX codec#146434
serhiy-storchaka merged 2 commits into
python:mainfrom
serhiy-storchaka:basexx-wrapcol-ignorechars

Conversation

@serhiy-storchaka

@serhiy-storchaka serhiy-storchaka commented Mar 25, 2026

Copy link
Copy Markdown
Member

Add the wrapcol parameter to base64 functions b16encode(), b32encode(), b32hexencode(), b85encode() and z85encode(), and binascii functions b2a_base32() and b2a_base85().

Add the ignorechars parameter to base64 functions b16decode(), b32decode(), b32hexdecode(), b85decode() and z85decode(), and binascii functions a2b_hex(), unhexlify(), a2b_base32() and a2b_base85().


📚 Documentation preview 📚: https://cpython-previews--146434.org.readthedocs.build/

…Base XX codec

Add the wrapcol parameter to base64 functions b16encode(), b32encode(),
b32hexencode(), b85encode() and z85encode(), and binascii functions
b2a_base32() and b2a_base85().

Add the ignorechars parameter to base64 functions b16decode(), b32decode(),
b32hexdecode(), b85decode() and z85decode(), and binascii functions
a2b_hex(), unhexlify(), a2b_base32() and a2b_base85().
@serhiy-storchaka serhiy-storchaka force-pushed the basexx-wrapcol-ignorechars branch from ea2c65d to c88fb7a Compare March 25, 2026 21:13

@gpshead gpshead left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

food for thought: could wrapcol be limited to "even" columns of one decodable chunk? (at least for base16/32/64) - It makes little sense to wrap on unusual boundaries.

Comment thread Lib/base64.py Outdated
Comment thread Lib/base64.py Outdated
Comment thread Lib/base64.py Outdated
Comment thread Modules/binascii.c
Comment thread Modules/binascii.c

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
unsigned char this_ch;
unsigned char this_ch = 0;

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is always initialized before use.

Comment thread Doc/library/base64.rst
Co-authored-by: Gregory P. Smith <68491+gpshead@users.noreply.github.com>
Co-authored-by: Serhiy Storchaka <storchaka@gmail.com>
Comment thread Lib/test/test_base64.py
if check_impl_detail():
eq(base64.b64encode(b, wrapcol=sys.maxsize*2),
b'd3d3LnB5dGhvbi5vcmc=')
if func is not base64.b16encode:

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

#147944 will allow to remove some of these special cases.

@serhiy-storchaka

Copy link
Copy Markdown
Member Author

@serhiy-storchaka serhiy-storchaka merged commit 42825e6 into python:main Apr 1, 2026
57 checks passed
@serhiy-storchaka serhiy-storchaka deleted the basexx-wrapcol-ignorechars branch April 1, 2026 07:48
clin1234 pushed a commit to clin1234/cpython that referenced this pull request Apr 16, 2026
…Base XX codec (pythonGH-146434)

Add the wrapcol parameter to base64 functions b16encode(), b32encode(),
b32hexencode(), b85encode() and z85encode(), and binascii functions
b2a_base32() and b2a_base85().

Add the ignorechars parameter to base64 functions b16decode(), b32decode(),
b32hexdecode(), b85decode() and z85decode(), and binascii functions
a2b_hex(), unhexlify(), a2b_base32() and a2b_base85().
ljfp pushed a commit to ljfp/cpython that referenced this pull request Apr 25, 2026
…Base XX codec (pythonGH-146434)

Add the wrapcol parameter to base64 functions b16encode(), b32encode(),
b32hexencode(), b85encode() and z85encode(), and binascii functions
b2a_base32() and b2a_base85().

Add the ignorechars parameter to base64 functions b16decode(), b32decode(),
b32hexdecode(), b85decode() and z85decode(), and binascii functions
a2b_hex(), unhexlify(), a2b_base32() and a2b_base85().
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