[issue39524] Escape sequences in doc string of ast._pad_whitespace

Change by Irit Katriel : -- resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker ___ ___

[issue39524] Escape sequences in doc string of ast._pad_whitespace

miss-islington added the comment: New changeset f3982d666c1df290373a4810edd501b2176b45c7 by Irit Katriel in branch '3.8': [3.8] bpo-39524: Fixed doc-string in ast._pad_whitespace (GH-18340) (GH-22857) https://github.com/python/cpython/commit/f3982d666c1df290373a4810edd501b2176b45c7

[issue39524] Escape sequences in doc string of ast._pad_whitespace

Change by Irit Katriel : -- pull_requests: +21800 pull_request: https://github.com/python/cpython/pull/22857 ___ Python tracker ___

[issue39524] Escape sequences in doc string of ast._pad_whitespace

Irit Katriel added the comment: The backport of the patch to 3.8 has failed. -- nosy: +iritkatriel ___ Python tracker ___ ___ Pytho

[issue39524] Escape sequences in doc string of ast._pad_whitespace

Serhiy Storchaka added the comment: New changeset fbeba8f2481411d608a616366394e07cdc52e0bb by mpheath in branch 'master': bpo-39524: Fixed doc-string in ast._pad_whitespace (GH-18340) https://github.com/python/cpython/commit/fbeba8f2481411d608a616366394e07cdc52e0bb -- _

[issue39524] Escape sequences in doc string of ast._pad_whitespace

Change by mpheath : -- pull_requests: +17714 pull_request: https://github.com/python/cpython/pull/18342 ___ Python tracker ___ ___ P

[issue39524] Escape sequences in doc string of ast._pad_whitespace

mpheath added the comment: I have submitted 2 Pull Requests. One for 3.9 and the other for 3.8. Only way I knew was to create 2 separate branches to checkout, linked to version tags to get the correct patches for ast.py. Hope is OK. -- ___ Python

[issue39524] Escape sequences in doc string of ast._pad_whitespace

Change by mpheath : -- pull_requests: +17713 pull_request: https://github.com/python/cpython/pull/18341 ___ Python tracker ___ ___ P

[issue39524] Escape sequences in doc string of ast._pad_whitespace

Change by mpheath : -- keywords: +patch pull_requests: +17712 stage: needs patch -> patch review pull_request: https://github.com/python/cpython/pull/18340 ___ Python tracker _

[issue39524] Escape sequences in doc string of ast._pad_whitespace

Amir Mohamadi added the comment: But in both cases the 'Output' will contain '\\': "Replace all chars except '\\f\\t' in a line with spaces." -- nosy: +Amir ___ Python tracker ___

[issue39524] Escape sequences in doc string of ast._pad_whitespace

Eric V. Smith added the comment: I think this would be an improvement, and an good issue for a newcomer. I'd vote for the r-string, but it doesn't really matter. -- keywords: +easy, newcomer friendly nosy: +eric.smith stage: -> needs patch ___ Pyt

[issue39524] Escape sequences in doc string of ast._pad_whitespace

Change by Karthikeyan Singaravelan : -- nosy: +serhiy.storchaka ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe

[issue39524] Escape sequences in doc string of ast._pad_whitespace

mpheath added the comment: Correction: Python 3.8.0 and 3.8.1 with 3.8/Lib/ast.py:277 Line 227 is invalid and 277 is correct and forgot to add 3.8 prefix. Sorry. -- ___ Python tracker __

[issue39524] Escape sequences in doc string of ast._pad_whitespace

mpheath added the comment: Correction: Python 3.8.0 and 3.8.1 with Lib/ast.py:277 Line 227 is invalid and 277 is correct. -- ___ Python tracker ___ __

[issue39524] Escape sequences in doc string of ast._pad_whitespace

New submission from mpheath : In the ast module, a function named _pad_whitespace has a doc string with escape sequences of \f and \t. The current doc string from Lib/ast.py:305 is: """Replace all chars except '\f\t' in a line with spaces.""" Example of doc string output in a REPL: