Title: Consider emitting buffered DEDENT tokens on the last line · Issue #104976 · python/cpython · GitHub
Open Graph Title: Consider emitting buffered DEDENT tokens on the last line · Issue #104976 · python/cpython
X Title: Consider emitting buffered DEDENT tokens on the last line · Issue #104976 · python/cpython
Description: In Python 3.12, porting the tokenizer to use the C tokenizer underneath to support PEP 701 has now a documented change in docs.python.org/3.12/whatsnew/3.12.html#changes-in-the-python-api: Some final DEDENT tokens are now emitted within ...
Open Graph Description: In Python 3.12, porting the tokenizer to use the C tokenizer underneath to support PEP 701 has now a documented change in docs.python.org/3.12/whatsnew/3.12.html#changes-in-the-python-api: Some fin...
X Description: In Python 3.12, porting the tokenizer to use the C tokenizer underneath to support PEP 701 has now a documented change in docs.python.org/3.12/whatsnew/3.12.html#changes-in-the-python-api: Some fin...
Opengraph URL: https://github.com/python/cpython/issues/104976
X: @github
Domain: github.com
{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"Consider emitting buffered DEDENT tokens on the last line","articleBody":"In Python 3.12, porting the tokenizer to use the C tokenizer underneath to support PEP 701 has now a documented change in [docs.python.org/3.12/whatsnew/3.12.html#changes-in-the-python-api](https://docs.python.org/3.12/whatsnew/3.12.html#changes-in-the-python-api):\r\n\r\n\u003e Some final DEDENT tokens are now emitted within the bounds of the input. This means that for a file containing 3 lines, the old version of the tokenizer returned a DEDENT token in line 4 whilst the new version returns the token in line 3.\r\n\r\nApparently, this affects negatively some formatting tools (see https://github.com/PyCQA/pycodestyle/issues/1142). Let's consider what options do we have and see if we can fix this without adding a lot of maintenance burden to the C tokenizer or slowing down everything.\n\n\u003c!-- gh-linked-prs --\u003e\n### Linked PRs\n* gh-104980\n* gh-105000\n\u003c!-- /gh-linked-prs --\u003e\n","author":{"url":"https://github.com/pablogsal","@type":"Person","name":"pablogsal"},"datePublished":"2023-05-26T13:29:30.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":6},"url":"https://github.com/104976/cpython/issues/104976"}
| route-pattern | /_view_fragments/issues/show/:user_id/:repository/:id/issue_layout(.:format) |
| route-controller | voltron_issues_fragments |
| route-action | issue_layout |
| fetch-nonce | v2:5081a48c-2153-ae9e-487f-8f41688903c4 |
| current-catalog-service-hash | 81bb79d38c15960b92d99bca9288a9108c7a47b18f2423d0f6438c5b7bcd2114 |
| request-id | 882E:17EBDB:10FE05A:16BC90C:6969A9AA |
| html-safe-nonce | b87d39e100c6f7e08e0b604f62c8b0c73f3808c89cdb2bfaca4e4719d019a498 |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiI4ODJFOjE3RUJEQjoxMEZFMDVBOjE2QkM5MEM6Njk2OUE5QUEiLCJ2aXNpdG9yX2lkIjoiNTE3MzgwNjcyNzEwMTUyNDM5NCIsInJlZ2lvbl9lZGdlIjoiaWFkIiwicmVnaW9uX3JlbmRlciI6ImlhZCJ9 |
| visitor-hmac | 84a45e8a69b294c1c3490d126800c301798027f8102f3bd2a1e9e36c83d18979 |
| hovercard-subject-tag | issue:1727632490 |
| github-keyboard-shortcuts | repository,issues,copilot |
| google-site-verification | Apib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I |
| octolytics-url | https://collector.github.com/github/collect |
| analytics-location | / |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/_view_fragments/issues/show/python/cpython/104976/issue_layout |
| twitter:image | https://opengraph.githubassets.com/ccfb97c1f85fb4d264539e2e57e2d6e5901d051927a2b918e185016ac195a1ed/python/cpython/issues/104976 |
| twitter:card | summary_large_image |
| og:image | https://opengraph.githubassets.com/ccfb97c1f85fb4d264539e2e57e2d6e5901d051927a2b918e185016ac195a1ed/python/cpython/issues/104976 |
| og:image:alt | In Python 3.12, porting the tokenizer to use the C tokenizer underneath to support PEP 701 has now a documented change in docs.python.org/3.12/whatsnew/3.12.html#changes-in-the-python-api: Some fin... |
| og:image:width | 1200 |
| og:image:height | 600 |
| og:site_name | GitHub |
| og:type | object |
| og:author:username | pablogsal |
| hostname | github.com |
| expected-hostname | github.com |
| None | 24c4c97a2d520cb286b35e1a4c22d7a4df3c26a2fa28dd7cdf0e65db327b4de7 |
| turbo-cache-control | no-preview |
| go-import | github.com/python/cpython git https://github.com/python/cpython.git |
| octolytics-dimension-user_id | 1525981 |
| octolytics-dimension-user_login | python |
| octolytics-dimension-repository_id | 81598961 |
| octolytics-dimension-repository_nwo | python/cpython |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 81598961 |
| octolytics-dimension-repository_network_root_nwo | python/cpython |
| turbo-body-classes | logged-out env-production page-responsive |
| disable-turbo | false |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | 124667f43168afb6c9c03b7c02eb5b1d2e1be3d9 |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width