]> asedeno.scripts.mit.edu Git - PuTTY.git/commit - unix/uxplink.c
Move echo/edit state change functionality out of ldisc_send.
authorSimon Tatham <anakin@pobox.com>
Sat, 22 Nov 2014 16:12:47 +0000 (16:12 +0000)
committerSimon Tatham <anakin@pobox.com>
Sat, 22 Nov 2014 16:18:00 +0000 (16:18 +0000)
commitc269dd0135a927d4d22a334cfefb09361f311fcd
tree8da275e33d97e7090c8c465f319f80c05a9f2421
parentd870b5650e48eb04529e8f8d8e9f73ca9923b3a1
Move echo/edit state change functionality out of ldisc_send.

I'm not actually sure why we've always had back ends notify ldisc of
changes to echo/edit settings by giving ldisc_send(ldisc,NULL,0,0) a
special meaning, instead of by having a separate dedicated notify
function with its own prototype and parameter set. Coverity's recent
observation that the two kinds of call don't even have the same
requirements on the ldisc (particularly, whether ldisc->term can be
NULL) makes me realise that it's really high time I separated the two
conceptually different operations into actually different functions.

While I'm here, I've renamed the confusing ldisc_update() function
which that special operation ends up feeding to, because it's not
actually a function applying to an ldisc - it applies to a front end.
So ldisc_send(ldisc,NULL,0,0) is now ldisc_echoedit_update(ldisc), and
that in turn figures out the current echo/edit settings before passing
them on to frontend_echoedit_update(). I think that should be clearer.
12 files changed:
ldisc.c
macosx/osxwin.m
pscp.c
psftp.c
putty.h
ssh.c
telnet.c
terminal.c
unix/gtkwin.c
unix/uxplink.c
windows/window.c
windows/winplink.c