QUERY · ISSUE

REPL / built-in readline: Add CTRL-L support (clear screen)

openby neuschaeferopened 2023-04-27updated 2023-04-27

enhancement

Right now, the built-in readline implementation (shared/readline/readline.c) does not support CTRL-L, which is a somewhat common key to clear the the screen, and move the prompt and command line to the top of the screen. At least bash supports it.

This feature could be optional, and features like MICROPY_REPL_EMACS_KEYS already are, but the size savings wouldn't very much.

I might implement this myself If I get around to it. If anyone else want to do it, please let me know here.

CANDIDATE · PULL REQUEST

mp-readline/readline.c: Add REPL support for CTRL-L

closedby mctopened 2017-05-21updated 2017-07-22

CTRL-L clears the screen and redraws the current prompt and input buffer. The cursor's position in the input buffer is maintained.

Assessment

S Sonnet · medium

Both items request the same feature: CTRL-L support in the MicroPython REPL readline to clear the screen while preserving the input buffer. PR #3104 implements exactly what issue #11354 requests. However, the file paths differ — PR #3104 targets lib/mp-readline/readline.c (older path) while the issue references shared/readline/readline.c (current path after codebase reorganization). The fact that issue #11354 was filed much later and explicitly states the feature is missing suggests PR #3104 was either never merged or the implementation was lost during the path reorganization. Confidence is medium rather than high because the different paths introduce ambiguity about whether the PR actually addresses the current codebase.

Suggested action

Check whether PR #3104 was ever merged. If it was merged before the readline code moved from `lib/mp-readline` to `shared/readline`, the feature was likely lost in the reorganization and issue #11354 is a legitimate re-request against the current code. If PR #3104 was never merged, link it to issue #11354 as a prior attempt and either revive/rebase the PR against the current path or treat the issue as superseding it.

REPL / built-in readline: Add CTRL-L support (clear screen)

mp-readline/readline.c: Add REPL support for CTRL-L

Keyboard