[issue24082] Obsolete note in argument parsing (c-api/arg.rst)
New submission from Petr Viktorin: A note in the docs for the "u" format unit saus NULs are not allowed, but the previous sentence says they aren't accepted. ---------- assignee: docs@python components: Documentation files: 0002-Remove-obsolete-note-in-argument-parsing-docs.patch keywords: patch messages: 242271 nosy: docs@python, encukou priority: normal severity: normal status: open title: Obsolete note in argument parsing (c-api/arg.rst) versions: Python 3.4 Added file: http://bugs.python.org/file39236/0002-Remove-obsolete-note-in-argument-parsi... _______________________________________ Python tracker <report@bugs.python.org> <http://bugs.python.org/issue24082> _______________________________________
R. David Murray added the comment: The note is still valid concerning the length. (Perhaps Victor should have deprected u instead of fixing it.) ---------- nosy: +haypo, r.david.murray _______________________________________ Python tracker <report@bugs.python.org> <http://bugs.python.org/issue24082> _______________________________________
Serhiy Storchaka added the comment: About wording see also issue23088. About possible deprecating see issue24009. ---------- nosy: +serhiy.storchaka _______________________________________ Python tracker <report@bugs.python.org> <http://bugs.python.org/issue24082> _______________________________________
Petr Viktorin added the comment: The note is *correct* concerning the length, but I don't think it's relevant: in a NUL-terminated string without embedded NULs, the length is unambiguous. The other issues are about "u" itself, not the note. (I have nothing against the "it is recommended to use u# or U instead" part, but the reasons the note gives are wrong and irrelevant, respectively.) ---------- _______________________________________ Python tracker <report@bugs.python.org> <http://bugs.python.org/issue24082> _______________________________________
Berker Peksag added the comment: The patch looks good to me. Note that the patch in issue 24278 also removes the same note. ---------- nosy: +berker.peksag stage: -> patch review type: -> behavior versions: +Python 3.5, Python 3.6 -Python 3.4 _______________________________________ Python tracker <report@bugs.python.org> <http://bugs.python.org/issue24082> _______________________________________
Martin Panter added the comment: I committed my patch for Issue 24278 (revision ad7da726bea6), so this notice is now removed ---------- nosy: +martin.panter resolution: -> fixed stage: patch review -> resolved status: open -> closed superseder: -> Docs on Parsing arguments should say something about mem mgmt for formatters returning C strings _______________________________________ Python tracker <report@bugs.python.org> <http://bugs.python.org/issue24082> _______________________________________
participants (5)
-
Berker Peksag -
Martin Panter
-
Petr Viktorin
-
R. David Murray
-
Serhiy Storchaka