curses.resizeterm()

[doerwalter]

BPO	1506645
Nosy	@loewis, @doerwalter
Files	curses.diff curses2.diff

^{Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.}

Show more details

GitHub fields:

assignee = None
closed_at = <Date 2006-06-19.14:54:31.000>
created_at = <Date 2006-06-15.12:53:20.000>
labels = []
title = 'curses.resizeterm()'
updated_at = <Date 2006-06-19.14:54:31.000>
user = 'https://github.com/doerwalter'

bugs.python.org fields:

activity = <Date 2006-06-19.14:54:31.000>
actor = 'scott.dial'
assignee = 'none'
closed = True
closed_date = None
closer = None
components = ['None']
creation = <Date 2006-06-15.12:53:20.000>
creator = 'doerwalter'
dependencies = []
files = ['7327', '7328']
hgrepos = []
issue_num = 1506645
keywords = ['patch']
message_count = 11.0
messages = ['50464', '50465', '50466', '50467', '50468', '50469', '50470', '50471', '50472', '50473', '50474']
nosy_count = 4.0
nosy_names = ['loewis', 'doerwalter', 'nnorwitz', 'scott.dial']
pr_nums = []
priority = 'normal'
resolution = 'accepted'
stage = None
status = 'closed'
superseder = None
type = None
url = 'https://bugs.python.org/issue1506645'
versions = []

[doerwalter]

This patch add curses.resizeterm()

[doerwalter]

Logged In: YES
user_id=89016

This second version adds a wrapper for resize_term() too.

[loewis]

Logged In: YES
user_id=21627

AFAICT, Solaris curses does not have resize_term, so you
need to add a configure test for it.

[doerwalter]

Logged In: YES
user_id=89016

Isn't it sufficient that the code is in the #ifdef
STRICT_SYSV_CURSES block?

[scottdial]

Logged In: YES
user_id=383208

I have taken the liberty (and my inability to sleep) to
update this patch in accordance with the recommendation of a
configure check. This version also includes is_term_resized
which goes along with the resize[_]term functions.

http://scottdial.com/python-dev/curses-resizeterm.diff

• It is worthy of note that my version of autoconf seemed to
  want to quote some things that the trunk version didn't so
  it perhaps you'd like to run "autoconf" to be safe.

[nnorwitz]

Logged In: YES
user_id=33168

I'm ok with the idea of this patch for 2.5 if you can get in
before the freeze. If you do check in, be sure to watch the
buildbots. But I have some specific comments/concerns below.

A couple of notes about Scott's patch. I would prefer the
macro name to be HAVE_CURSES_RESIZETERM. I don't understand
why there are 2 variants with and without the _. I see
there are 2 different curses APIs. Does the single HAVE_
cover the cases of having is_term_resized, resize_term, and
resizeterm? It seems like it might be better to have 2 or 3
configure checks.

[doerwalter]

Logged In: YES
user_id=89016

The man page states: "Most of the work is done by the inner
function resize_term. The outer function resizeterm adds
bookkeeping for the SIGWINCH handler." All three functions
are closely related and are documented on the same man page,
so we should be safe with one check. I'll change the macro
name and check in the patch tomorrow morning.

[doerwalter]

Logged In: YES
user_id=89016

Checked in a version that does three separate configure
checks as r47022. I'll close the patch, once all buildbot
have survived the checkin.

[doerwalter]

Logged In: YES
user_id=89016

Looks good, closing the patch. Thanks to Scoll for the patch!

[doerwalter]

Logged In: YES
user_id=89016

Oops, sorry I meant Scott! ;)

[scottdial]

Logged In: YES
user_id=383208

Haha, nice typo. No problem. Glad to help and hope to
continue to do so in the future.

For what it's worth, I couldn't find a platform that one of
the functions but not the others (though, in fact, I could
only find a Solaris box which had none). But obviously
checking each of them would be wise.