Issue 29557: binhex documentation claims unknown bug (original) (raw)

Created on 2017-02-14 16:20 by David Wilemski, last changed 2022-04-11 14:58 by admin. This issue is now closed.

Pull Requests
URL Status Linked Edit
PR 90 merged David Wilemski,2017-02-14 16:20
PR 474 merged Mariatta,2017-03-05 06:51
PR 475 merged Mariatta,2017-03-05 06:51
Messages (10)
msg287781 - (view) Author: David Wilemski (David Wilemski) * Date: 2017-02-14 16:20
The binhex documentation has a line in it that says "As of this writing, hexbin() appears to not work in all cases." This is a bit ambiguous and there is no context for what the bug may be. It appears this line is from the first commit to the hexbin function's documentation in 1995 (https://hg.python.org/cpython/rev/3911d4a89ab0#l4.40) but the commit also has no information about what this bug may be. I also see no current open bugs for the binhex module that seem related. In 2009, there was a patch for the hexbin function but I have no idea if this is the bug from the original documentation or if it was something else entirely: https://bugs.python.org/issue6369 Given this, I believe the line should be removed unless more details about the referenced buggy behavior can be found.
msg287801 - (view) Author: Berker Peksag (berker.peksag) * (Python committer) Date: 2017-02-14 22:37
New changeset 6de2b7817fa9403e81dc38f13f3690f0bbf3d064 by Berker Peksag in branch 'master': bpo-29557: Remove ambiguous line in binhex docs (#90) https://github.com/python/cpython/commit/6de2b7817fa9403e81dc38f13f3690f0bbf3d064
msg287819 - (view) Author: Serhiy Storchaka (serhiy.storchaka) * (Python committer) Date: 2017-02-15 06:02
I think this should be read in the context of previous lines: If you code or decode textfiles on non-Macintosh platforms they will still use the old Macintosh newline convention (carriage-return as end of line). And may be related to the in-code comment: # XXXX Do translation on non-mac systems
msg287828 - (view) Author: Raymond Hettinger (rhettinger) * (Python committer) Date: 2017-02-15 09:37
Georg, do remember why this admonition was present?
msg287860 - (view) Author: Ronald Oussoren (ronaldoussoren) * (Python committer) Date: 2017-02-15 15:38
I think the comment about needing translation on non-mac systems is incorrect. Binhex is (or rather was) a textual encoding for binary files used on classic MacOS. The encoded file is more likely to be binary data than a text file and replacing b'\r' by the line ending of the current platform could well corrupt that data.
msg287861 - (view) Author: Serhiy Storchaka (serhiy.storchaka) * (Python committer) Date: 2017-02-15 15:49
File flags are saved in binhex format, therefore it is possible to distinguish text files from binary files.
msg287868 - (view) Author: Georg Brandl (georg.brandl) * (Python committer) Date: 2017-02-15 17:29
I don't remember anything specific about this, sorry.
msg289003 - (view) Author: Serhiy Storchaka (serhiy.storchaka) * (Python committer) Date: 2017-03-05 07:46
See also . Perhaps hexbin() bug is related to that bug.
msg290287 - (view) Author: Mariatta (Mariatta) * (Python committer) Date: 2017-03-24 22:49
New changeset 6b7bc45e33ec953c542788420adc305ec026fa40 by Mariatta in branch '3.5': bpo-29557: Remove ambiguous line in binhex docs (GH-90) (GH-474) https://github.com/python/cpython/commit/6b7bc45e33ec953c542788420adc305ec026fa40
msg290288 - (view) Author: Mariatta (Mariatta) * (Python committer) Date: 2017-03-24 22:49
New changeset 130c4ec5fbeb0878b88ec6aa06e47c9672566c17 by Mariatta in branch '3.6': bpo-29557: Remove ambiguous line in binhex docs (GH-90) (GH-475) https://github.com/python/cpython/commit/130c4ec5fbeb0878b88ec6aa06e47c9672566c17
History
Date User Action Args
2022-04-11 14:58:43 admin set github: 73743
2017-03-25 05:53:17 serhiy.storchaka set status: open -> closedresolution: fixedstage: patch review -> resolved
2017-03-24 22:49:25 Mariatta set messages: +
2017-03-24 22:49:19 Mariatta set nosy: + Mariattamessages: +
2017-03-05 07:46:31 serhiy.storchaka set messages: +
2017-03-05 06:51:46 Mariatta set pull_requests: + <pull%5Frequest390>
2017-03-05 06:51:15 Mariatta set pull_requests: + <pull%5Frequest389>
2017-02-15 17:29:12 georg.brandl set messages: +
2017-02-15 15:49:01 serhiy.storchaka set messages: +
2017-02-15 15:38:54 ronaldoussoren set messages: +
2017-02-15 09:37:41 rhettinger set nosy: + rhettinger, georg.brandlmessages: +
2017-02-15 06:03:03 serhiy.storchaka set nosy: + ronaldoussoren, ned.deilycomponents: + macOS
2017-02-15 06:02:35 serhiy.storchaka set nosy: + serhiy.storchakamessages: +
2017-02-14 22:40:10 berker.peksag set stage: patch reviewtype: behaviorversions: + Python 3.5, Python 3.6
2017-02-14 22:37:52 berker.peksag set nosy: + berker.peksagmessages: +
2017-02-14 16:20:44 David Wilemski create