The Wayback Machine - https://web.archive.org/web/20211027232421/https://github.com/python/cpython/pull/27935

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Format the Python-tokenize module and fix exit path #27935

Merged

ambv merged 1 commit into python:main from pablogsal:tok_cleanup

Aug 25, 2021

Member

pablogsal commented Aug 24, 2021

No description provided.

the-knights-who-say-ni added the CLA signed label

bedevere-bot added the awaiting core review label

pablogsal requested a review from ambv

pablogsal added skip issue skip news labels

pablogsal force-pushed the tok_cleanup branch from 8ec3f6f to 2254bac

Member Author

pablogsal commented Aug 24, 2021 •

edited

I used clang-format to get a PEP-7 consistent formatting.

pablogsal force-pushed the tok_cleanup branch from 2254bac to 2c831c9


          Format the Python-tokenize module and fix exit path

1b16d56

pablogsal force-pushed the tok_cleanup branch from 2c831c9 to 1b16d56

pablogsal mentioned this pull request

Add tests for the C tokenizer and expose it as a private module #27924

Merged

Member Author

pablogsal commented Aug 25, 2021

ambv reviewed

Python/Python-tokenize.c

  
                if (filename == NULL) {

                    return NULL;

                }

                self->tok = PyTokenizer_FromUTF8(source, 1);

                if (self->tok == NULL) {

                    Py_DECREF(filename);

Contributor

ambv Aug 25, 2021

👍🏻

ambv merged commit 214c2e5 into python:main

11 checks passed

bedevere-bot removed the awaiting core review label

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment