curses.update_lines.cols() are normally usable within c programs. Withthis change, it can now be used too. It only calls the preexistingfunction that where only used internally. The cast in the returnstatement are possibly false? But it works.Because this aren't a big change, it would apply to some morepython-versions.

Your function PyCurses_update_lines_cols() has no documentation. Can you add it?Can you also give an use case of update_lines_cols()? The function is already called by curses.resizeterm() and curses.resize_term().

Hi Victor,i use this to get updated versions of curses.COLS and curses.LINES in the fact of an curses.KEY_RESIZE event.So i can use curses within python even without to have panels.It seems that curses.panel are the prefered way to deal with terminal resizes.The curses.resize[_]term() are only to change the terminal manualy.What i also found in PyTone is more or less this functionality from update_lines_cols():def getmaxyx(self):    # taken fromhttp://dag.wieers.com/home-made/dstat/    try:        h, w = int(os.environ["LINES"]), int(os.environ["COLUMNS"])    except KeyError:        try:            h, w = curses.tigetnum('lines'), curses.tigetnum('cols')        except:            try:                s = struct.pack('HHHH', 0, 0, 0, 0)                x = fcntl.ioctl(sys.stdout.fileno(), termios.TIOCGWINSZ,s)                h, w = struct.unpack('HHHH', x)[:2]            except:                h, w = 25, 80And please excuse me for the missing documentation. English aren't my native language so i would document it like:Updates curses.LINES and curses.COLS :)It's my first day here on the issue tracker.

> i use this to get updated versions of curses.COLS and curses.LINES in> the fact of an curses.KEY_RESIZE event.I didn't know this event. Is a key in a special keyboard? Or an event raised by some curses internals?> Updates curses.LINES and curses.COLSYour documentation is incomplete. You may reused this comment:/* Internal helper used for updating curses.LINES, curses.COLS,  * _curses.LINES and _curses.COLS */Oh I just realized that _curses functions have no documentation, great :-/> It's my first day here on the issue tracker.Welcome on the tracker ;-)

>> i use this to get updated versions of curses.COLS and curses.LINES in>> the fact of an curses.KEY_RESIZE event.>I didn't know this event. Is a key in a special keyboard? Or an event >raised >by some curses internals?Internal curses event.>Oh I just realized that _curses functions have no documentation, >great :-/Thats why python_curses should be more compatible to the existing c_curses documentations.>> It's my first day here on the issue tracker.Welcome on the tracker ;-)Thanks.Any questions? :)

No documentation?Doc/library/curses.rst exists; there's alsoDoc/howto/curses.rst.  What functions in _curses need to be documented?

BTW, the code portion of the patch looks OK to me.

msg82783 states code portion of patch looks ok, what do we need to take this forward?

Creating an automatic test for this issue proves difficult, as using curses to resize the terminal already updates the values, and using ioctls to try to bypass curses, doesn't work.I'm adding a manual test program (mostly taken fromhttp://www.gossamer-threads.com/lists/python/python/226137), and a newer patch with documentation.

Thanks for the patch update.The doc addition needs a .. versionadded directive, and there should be an entry in the 3.5 whatsnew document.

Fixed David's comments

Thanks for updating.In the doc change you say it updates environment variables, but in the whatsnew change you say it changes constants of the curses module. I believe the whats new version is correct?  Also, doc lines should be wrapped to 80 columns.Please omit the NEWS item from the patch...with our current system putting NEWS items in patches just makes the patches get stale really quickly.  The committer will add the NEWS item.

I'm adding patch without the update toMisc/NEWS.I'm not sure why, but the curses doc refers to LINES and COLS as "environment variables". In the doc change I referred to them as just "variables" and used the notation that works for linking to them, using :envvar:

Adding a test that only calls the function.

Arnon - we'll need you to fill outhttps://www.python.org/psf/contrib/contrib-form/ before we can merge this patch in. (The dev guide isn't clear on whether trivial patches are okay, but the advice I was given is that we should have a CLA for everything.)

New changeset132b5376bf34 by Steve Dower in branch 'default':Issue#4254: Adds _curses.update_lines_cols() Patch by Arnon Yaarihttps://hg.python.org/cpython/rev/132b5376bf34

Arnon has signed the CLA at PyCon, it just hasn't been processed yet.

Looks to like there is nothing left to do here, so I'm closing it.