Simon Tatham [Tue, 26 Nov 2002 19:35:40 +0000 (19:35 +0000)]
Increase the size of the `font' buffer in the Config structure, for
the benefit of X font names which are rather more verbose than
Windows. One day I want to replace all these fixed-size buffers with
sensible dynamically allocated stuff, but not today.
Ben Harris [Tue, 26 Nov 2002 01:32:51 +0000 (01:32 +0000)]
Substantial overhaul of colour handling in do_text(). Highlights include:
* No more mucking about with transfer modes. We always set up the foreground
and background sensibly, EraseRect the whole lot and use srcOr. Apple
Tech Note QD505 suggests that this should be faster than srcCopy.
* Always use the Palette Manager if we've got Color QuickDraw, even in 1bpp
modes.
* Apply the cursor colours _after_ all the other stuff, rather than trying to
get them through everything else intact.
Unfortunately, this doesn't make much difference to the speed.
Ben Harris [Sun, 24 Nov 2002 00:38:44 +0000 (00:38 +0000)]
Fix scrolling on systems without Color QuickDraw. It seems that the current
background colour and pen mode have some silly effects on ScrollRect, so
set them back to their defaults before calling it.
Ben Harris [Sat, 23 Nov 2002 22:36:56 +0000 (22:36 +0000)]
Don't try to initialise the palette unless we have Color QuickDraw. This gets
PuTTY running on my Mac SE again (albeit very slowly and with scroll
optimisation broken for some reason).
Ben Harris [Sat, 23 Nov 2002 21:42:46 +0000 (21:42 +0000)]
Fix a bug in scroll_display(): when the scrolled region contains the cursor
we need to update dispcurs as well as disptext.
Add scroll optimisation to scroll(). This makes it rather obvious that scroll
optimisation is breaking with PuTTY's usual policy of lazy updates, since
scrolling is done eagerly. Fixing this so that all the scrolling is saved
up for do_paint would be nice, but non-trivial.
Ben Harris [Sat, 23 Nov 2002 20:02:38 +0000 (20:02 +0000)]
Add a Config * argument to ldisc_create(), and use it in place of the global
cfg throughout ldisc.c. Not tested other than on Mac, but all other ports
just pass &cfg as this argument for now.
Ben Harris [Sat, 23 Nov 2002 19:01:01 +0000 (19:01 +0000)]
Add a "Config *" argument to term_init(), and use that instead of the global
cfg throughout the terminal emulator. Not tested in PuTTY and pterm, but they
just pass in &cfg.
Ben Harris [Sat, 23 Nov 2002 18:22:47 +0000 (18:22 +0000)]
Only try to retrieve a 'vers' resource from the current resource file (the
application), rather from the entire chain. This prevents us displaying the
System version if PuTTY doesn't have one. Also cope with not finding any
'vers' resource.
Ben Harris [Sat, 23 Nov 2002 14:22:11 +0000 (14:22 +0000)]
On a keypress, hide the mouse cursor, send the key to the line discipline
(rather than straight to the backend), and then prod the terminal. I think
this is closer to the correct set of things to do.
Ben Harris [Sat, 23 Nov 2002 13:07:29 +0000 (13:07 +0000)]
Bring in some of my scroll-optimisation stuff from the old Mac port.
This introduces a new front-end function, do_scroll(), which is expected to
scroll a part of the physical display and cause repaint events for any
areas that couldn't be scrolled (e.g. because they were hidden).
scroll_display() is a wrapper around this which also updates disptext to
match.
Currently, scroll_display is only used in response to user scrollback requests
(via term_scroll()), but extending scroll() to use it as well should be
easy.
All of this is conditional on the front end's defining OPTIMISE_SCROLL, since
only the Mac front end currently implements do_scroll().
Ben Harris [Tue, 19 Nov 2002 02:13:46 +0000 (02:13 +0000)]
Tentative merge of ben-mac-port (only dead for three years!) into the trunk.
This doesn't include any mkfiles.pl glue, and is missing one or two other
fixes. The terminal emulator is kind of working, though, as, I believe, is
the store module. Everything else is yet to be done.
Ben Harris [Mon, 18 Nov 2002 23:49:30 +0000 (23:49 +0000)]
When processing input, refer to CR and LF as \015 and \012 respectively,
rather than \r and \n. The latter tend to get swapped around by Mac C
compilers.
Ben Harris [Mon, 18 Nov 2002 22:27:25 +0000 (22:27 +0000)]
ISO-IR entry 157 appears to correspond to ISO/IEC 8859-10, and lists
code point 3/13 as HORIZONTAL BAR, which agrees with unicode.org's mapping
table. Change ours to match (it used to have EM DASH, courtesy of RDB).
This brings all our 8859-to-Unicode tables into line with unicode.org.
Simon Tatham [Sun, 17 Nov 2002 15:06:16 +0000 (15:06 +0000)]
Thanks to Hans-Juergen Petrich for spotting this tiny memory leak:
`otherbuf' should still be freed even if the RegEnumKey function
that was supposed to fill it with data failed. While I'm at it, also
remove the redundant check for its non-NULL-ness (what's the point
of having a malloc wrapper that dies rather than return NULL if you
then waste effort checking its return value for NULL _anyway_, eh?).
Ben Harris [Sun, 17 Nov 2002 02:00:06 +0000 (02:00 +0000)]
Apple's C compilers don't think that putting parentheses around assignments
in "if" conditions is enough. Use an actual comparison against NULL instead.
Ben Harris [Sun, 10 Nov 2002 00:03:55 +0000 (00:03 +0000)]
Use <stddef.h> to get wchar_t, rather than <wchar.h> (or nothing, in putty.h).
Both are required to contain wchar_t in C99, but only <stddef.h> does in the
version of MPW I've got here.
Simon Tatham [Thu, 7 Nov 2002 19:49:03 +0000 (19:49 +0000)]
Robustness fixes for KEXINIT handling and others. In particular, I've
created a self-mallocing variant of sprintf, to obviate any future
need for paranoid %.100s type stuff in format strings.
Simon Tatham [Tue, 5 Nov 2002 13:20:42 +0000 (13:20 +0000)]
Fix command-line error handling in pterm. (Hint: wrapping a multi-
statement macro in `do ... while (0)' and putting a `continue'
within it don't go well together. Oops.)
Simon Tatham [Sun, 3 Nov 2002 08:46:35 +0000 (08:46 +0000)]
Half of Lars Gunnarsson's iXplorer compatibility patch: the PSFTP
login prompt should be fflushed (presumably fgets fails to implicitly
do this when stdin and stdout are redirected weirdly).
Simon Tatham [Sat, 2 Nov 2002 16:27:17 +0000 (16:27 +0000)]
Probably about time we mentioned the nascent Unix port in the FAQ.
Not that I desperately want to shout about it just yet, but I feel a
bit bad about the FAQ saying `we don't have a Unix port, anyone who
told you so was wrong'. :-)
Simon Tatham [Sat, 2 Nov 2002 16:16:35 +0000 (16:16 +0000)]
Improve shadow bold mode: set the default shadow bold offset to +1
not -1 (it turns out _most_ X fonts prefer the former, though
irritatingly my favourite real X font used to prefer the latter
which was why I made the X version of my Font Of Choice do so too),
and also clip to the boundaries of the rectangle we should be
drawing text in. This still doesn't completely prevent display
corruption in the case where text drawn in one sweep is partially
overwritten in a future one, but gnome-terminal has this problem
too, and now we've got the right default SB offset _and_ offer the
opportunity to reconfigure it I think this is pretty good for now.
Simon Tatham [Sat, 2 Nov 2002 16:05:26 +0000 (16:05 +0000)]
Another signal-handling refinement from RJK: the SIGCHLD handler
should be prepared to reap more than one child per invocation if
necessary, since we do after all have two.
Simon Tatham [Fri, 1 Nov 2002 13:36:48 +0000 (13:36 +0000)]
Improve handling of oobinline sockets; Plink in telnet mode now
doesn't hang when you hit ^C, which is nice. I think a better
solution would involve nonblocking sockets; as it stands it's a
little dependent on what may be quirks of the Linux socket layer.
Simon Tatham [Fri, 1 Nov 2002 13:01:14 +0000 (13:01 +0000)]
Another valgrind-caught error. This one has apparently been there
since the Dawn O' Time, and consisted of me putting the two halves
of a short-circuiting bounds check the wrong way round: instead of
`p_in_range && *p', I had `*p && p_in_range'. Oops. valgrind rocks.
Simon Tatham [Thu, 31 Oct 2002 19:49:52 +0000 (19:49 +0000)]
First attempt at a Unix port of Plink. Seems to basically work;
doesn't yet use the SSH agent, no way to specify arbitrary config
options, no manpage yet, couple of other fiddly things need doing,
but it makes SSH connections and doesn't fall over horribly so I say
it's a good start. Now to run it under valgrind...
Simon Tatham [Wed, 30 Oct 2002 18:22:37 +0000 (18:22 +0000)]
RJK's `pterm --help' patch. I _must_ find a better alternative to
this init sequence - it surely can't be right that `pterm --help'
with no DISPLAY complains at the lack of DISPLAY rather than giving
a help message!
Simon Tatham [Wed, 30 Oct 2002 17:57:31 +0000 (17:57 +0000)]
More preparatory work: remove the <windows.h> include from lots of
source files in which it's no longer required (it was previously
required in anything that included <putty.h>, but not any more).
Also moved a couple of stray bits of exposed WinSock back into
winnet.c (getservbyname from ssh.c and AF_INET from proxy.c).
Simon Tatham [Wed, 30 Oct 2002 17:56:05 +0000 (17:56 +0000)]
Preparatory work before attempting a Unix port of plink: mkfiles.pl
needs to be able to handle separate Recipe entries for the same
program with different types (plink [C] and plink [X] for example,
with different object lists).
Simon Tatham [Tue, 29 Oct 2002 16:54:09 +0000 (16:54 +0000)]
Yikes - put back a #include I accidentally removed two revs ago!
That'd have been embarrassing. Mind you I'd like to remove this
particular #include for good anyway, but the time is not now...
Simon Tatham [Mon, 28 Oct 2002 21:58:07 +0000 (21:58 +0000)]
terminal.c should call the frontend beep() routine even with
mode==BELL_VISUAL, otherwise taskbar flashing won't happen on visual
bells. It's up to the frontend routine to spot BELL_VISUAL and avoid
making any noise.
Simon Tatham [Mon, 28 Oct 2002 17:39:21 +0000 (17:39 +0000)]
CloseOnExit now defaults to COE_ALWAYS for pterm, bringing it back
into line with most other xtermalikes. On Unix, the exit code of a
shell is the last exit code of one of its child processes, even if
it's an interactive shell - so some pterms will close and some will
not for no particularly good reason. Power-detaching a screen
session is especially bad for this.
COE_NORMAL is still useful for specialist purposes (running a single
command in its own pterm), but I don't think it's a sane default,
unfortunately.
Simon Tatham [Mon, 28 Oct 2002 17:34:45 +0000 (17:34 +0000)]
Fix Alt+numberpad in pterm. For a start, there was a bug whereby if
a pterm came up while Alt was down, then releasing it would cause a
^@ to be generated. Also, though, I've decided that Alt plus a
single numberpad key should not generate a low-numbered control
code, because that's too easy to do by mistake and the codes are too
powerful. Anyone who really _wants_ to create a ^C or ^D from the
numberpad can do Alt-03 or Alt-04 easily enough; two-digit codes and
more such as Alt-65 are unaffected.
Simon Tatham [Mon, 28 Oct 2002 17:30:47 +0000 (17:30 +0000)]
Another deglobalisation bug: failed to initialise term->curstype.
Thanks to valgrind for finding this one (aha! I knew there would be
benefits from doing a Unix port!).
Simon Tatham [Mon, 28 Oct 2002 09:38:28 +0000 (09:38 +0000)]
First bug discovered as a result of global-removal: pasting into
pterm caused a crash because I had the wrong prototype for the
selection_received event handler. Should be fixed.
Simon Tatham [Sat, 26 Oct 2002 12:58:13 +0000 (12:58 +0000)]
Yet more global-removal. The static variables in logging.c are now
absent, and also (I think) all the frontend request functions (such
as request_resize) take a context pointer, so that multiple windows
can be handled sensibly. I wouldn't swear to this, but I _think_
that only leaves the Unicode stuff as the last stubborn holdout.
Simon Tatham [Sat, 26 Oct 2002 11:08:59 +0000 (11:08 +0000)]
Reorganised the Unicode layer somewhat: moved luni_send and
lpage_send out into the line discipline, making them _clients_ of
the Unicode layer rather than part of it. This means they can access
ldisc->term, which in turn means I've been able to remove the
temporary global variable `term'. We're slowly getting there.
Simon Tatham [Sat, 26 Oct 2002 10:33:59 +0000 (10:33 +0000)]
Port forwarding module now passes backend handles around properly.
As a result I've now been able to turn the global variables `back'
and `backhandle' into module-level statics in the individual front
ends. Now _that's_ progress!