Repository navigation
gh-160141: Curses core fixes - #160142
gh-160141: Curses core fixes#160142cdorsman wants to merge 15 commits into
Conversation
|
The following commit authors need to sign the Contributor License Agreement: |
|
Most changes to Python require a NEWS entry. Add one using the blurb_it web app or the blurb command-line tool. If this change has little impact on Python users, wait for a maintainer to apply the |
|
Most changes to Python require a NEWS entry. Add one using the blurb_it web app or the blurb command-line tool. If this change has little impact on Python users, wait for a maintainer to apply the |
Documentation build overview
|
|
Most changes to Python require a NEWS entry. Add one using the blurb_it web app or the blurb command-line tool. If this change has little impact on Python users, wait for a maintainer to apply the |
|
Most changes to Python require a NEWS entry. Add one using the blurb_it web app or the blurb command-line tool. If this change has little impact on Python users, wait for a maintainer to apply the |
|
Most changes to Python require a NEWS entry. Add one using the blurb_it web app or the blurb command-line tool. If this change has little impact on Python users, wait for a maintainer to apply the |
picnixz
left a comment
There was a problem hiding this comment.
ISTM that this makes unrelated code changes and also doesn't use AC where it could be used. I also wonder whether this API is really necessayr.
| /* | ||
| * Interface to the ncurses panel library | ||
| * | ||
| * Original version by Thomas Gellekum |
There was a problem hiding this comment.
I don't understand this specific change. Panels are already written in _curses_panel.c. Why duplicating this?
| return curses_check_err(module, putp(string), "putp", NULL); | ||
| } | ||
|
|
||
| static PyObject * |
|
A Python core developer has requested some changes be made to your pull request before we can consider merging it. If you could please address their requests along with any other requests in other reviews from core developers that would be appreciated. Once you have made the requested changes, please leave a comment on this pull request containing the phrase |
|
I'm clsoing the PR because the feature has not been discussed by the maintainer. What are the use cases for those specific functions? panel is implemented by the _curses_panel.c file |
Summary
Complete the core curses bindings in CPython and adapt the port to the
current stateful curses module architecture.
The change covers the broader curses core API, including:
ripofflineExisting functionality already present on
mainis preserved andintegrated with the port. The curses form and menu APIs are intentionally
excluded from this change.
Tests
make -j2TERM=dumb ./python -m test -u curses test_curses -vResult:
provides ncurses 6.4.
Notes
The implementation uses the current CPython module-state and Argument
Clinic conventions rather than copying the older implementation
verbatim.
Closes gh-160141