curiousdannii-testing / inform7-imported-bugs

0 stars 0 forks source link

[I7-1991] [Mantis 2027] Empty understand causes hard hang at runtime. #93

Closed curiousdannii-testing closed 2 years ago

curiousdannii-testing commented 2 years ago

Reported by : PO8

Description :

The attached source text causes Inform 7 to hang hard at the first attempt to examine the professor. Removing "/--" from the understand restores normal operation.

Obviously, it is wrong to write this: the motivating bug was in a much more complicated use of understand. But really there needs to be a compile-time or runtime error rather than just a hard hang like this.

Steps to reproduce :

"Understand Bug" by "Bart Massey"

The School for Gifted Kiddies is a room.  Professor Zaphier is in School. Understand "Prof/--" as Professor Zaphier.

Additional information :

imported from: [Mantis 2027] Empty understand causes hard hang at runtime.
  • status: Closed
  • resolution: Resolved
  • resolved: 2022-04-10T04:43:23+10:00
  • imported: 2022/01/10
curiousdannii-testing commented 2 years ago

61eedb62875fc10070240916:

Fixed via this commit:
https://github.com/ganelson/inform/commit/3441f2497987d11c2438b1f281a383a5727bbeba

Comment by Graham Nelson
There is now a problem message catching this mistake at compile time.