diff options
| author | Berke Güzel <wenekar1@gmail.com> | 2026-10-15 12:29:35 +0300 |
|---|---|---|
| committer | Berke Güzel <wenekar1@gmail.com> | 2026-10-15 12:29:35 +0300 |
| commit | e47f9f20d68eda78a6621917a7e100ae333a1a53 (patch) | |
| tree | 47552c95fc3e47d9379a31568eef1095d17709ab /tests/model.c | |
| parent | 5f03033b0e211f9ed13e7a1351f9a8f9011ab665 (diff) | |
Reject invalid arguments instead of ignoring them
term_feed() returns bool and rejects a NULL terminal, a NULL buffer with
a nonzero length, and any flag bit it does not know. Unknown bits are
rejected now so that later flags can be added without changing what the
old calls mean.
term_new() returns NULL for a negative size component, columns above
TERM_MAX_COLS, a negative scrollback, or a cursor style outside the enum.
Zero still selects the default size and scrollback. It no longer clamps
bad values silently.
term_resize() rejects columns above TERM_MAX_COLS instead of clamping
them, which matches how it already rejects a size below 1.
Co-Authored-By: Claude Haiku 5.5 <noreply@anthropic.com>
Diffstat (limited to 'tests/model.c')
| -rw-r--r-- | tests/model.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tests/model.c b/tests/model.c index 8dead6c..6e368ba 100644 --- a/tests/model.c +++ b/tests/model.c @@ -1290,7 +1290,7 @@ static void resize_plain(struct model *m, int cols, int rows) static bool resize(struct model *m, int cols, int rows) { - if (cols > TERM_MAX_COLS) cols = TERM_MAX_COLS; + if (cols > TERM_MAX_COLS) return false; if (cols == m->cols && rows == m->rows) return true; if (cols > MAX_TABS) return false; m->top = 0; |
