Bug 16 : starting a comment and not finishing it causes NullPointerException or OutOfMemoryError
Last modified: 2010-03-15 13:58




Status:
RESOLVED
Resolution:
FIXED -
Priority:
P5
Severity:
normal

 

Reporter:
fry
Assigned To:
fry

Attachment Type Created Size Actions
Example of not completing a multiline comment application/zip 2005-12-13 14:42 271 bytes

Description:   Opened: 2005-05-12 19:07
should instead die with dignity.
Additional Comment #1 From fry 2005-10-31 20:25
*** Bug 193 has been marked as a duplicate of this bug. ***
Additional Comment #2 From REAS 2005-12-13 14:42
edit]
Example of not completing a multiline comment

It's a common error to type the end of a multiline comment incorrectly.
This creates a lava.lang.OutOfMemoryError
Additional Comment #3 From fry 2005-12-14 09:05
this requires heavy work on the preprocessor, but in the meantime, i can add a workaround
that would check for a non-terminated comment if you're running into this a lot.
Additional Comment #4 From REAS 2005-12-14 10:59
It is a very common problem with beginners. On slower machines, the
OutOfMemoryError often takes such a long time to reveal itself that people
first try force quitting the application and then end up restarting the
machine. This can really kill enthusiasm. On faster machines, the memory
error writes to the console and the sketch stops trying run.
Additional Comment #5 From fry 2005-12-14 14:28
*** Bug 247 has been marked as a duplicate of this bug. ***
Additional Comment #6 From fry 2005-12-14 18:34
in the spirit of bug #5, a workaround (hack) has been added for rev 0099,
so the threat level will be lowered to a five (four below "bears").
Additional Comment #7 From FloHimself 2005-12-20 07:35
you get the same OutOfMemoryError error with a simple:

println("foo); //(a missing ")

on rev 0099. also a very common typo.
Additional Comment #8 From fry 2005-12-20 09:46
please file that as a separate bug, that's nothing to do with an unfinished
comment.
Additional Comment #9 From MrFeinberg 2010-03-15 13:58
fixed; please verify