[comp.sources.bugs] calctoolv2.4: 's' '+' '0' locks up calctool

dave@ttrnds.UUCP (David M. Karr) (03/07/90)

In calctool, version 2.4, I wanted to try adding the display to a
register by pressing 's' (store), then '+' (add to register), and then
'0', for register 0.  At that point, calctool refused to take any more
keys, even though it still appeared to be running.

I tried running Saber-C on it, and I found some information, but I
decided I didn't know the structure of calctool well enough to make a
change.

What I noticed was that in "do_sto_rcl", before the assignment to
pending_op, it found current == '0', pending == 's', and pending_op ==
'a'.  The value of pending_op should have been '+'.  It is not clear
to me why it was ignoring keys from that point on.  The "STO" in the
display never went away.

Unrelated to this problem, I noticed a possible problem in
"convert_display".  It appeared that the line

  exp *= exp_sign ;

was being executed before "exp_sign" was set.  It is not clear whether
the resulting value was actually used for anything.

In the header of the calctool files, it mentioned both Rich Burridge
and Ed Falk from Sun having responsibility for calctool, but no mail
addresses were included, which is why this note is here, instead of in
their mailboxes.
-- 
----------------------------------------
David Karr    dave@ttrnds.UUCP  or  ...amc-gw!ttrnds!dave
Teltrend Corp., 12034 115th Ave. NE, Kirkland, WA 98034 (206)820-6500
"The above statements do not necessarily reflect the opinions of my employer."