Closed Bug 17050 Opened 25 years ago Closed 25 years ago

';' in unintended location in jsexn.c? function--Exception()

Categories

(Core :: JavaScript Engine, defect, P3)

Product:
Core
Component:
JavaScript Engine
Type:
defect

Tracking

()

Status:
RESOLVED FIXED

People

(Reporter: Brade, Assigned: rogerl)

Details

line 293 of jsexn.c results in a compiler warning on Macintosh.
It flags the ';' at the end of the "if"

is the ';' intentional?
if so, the compiler won't complain if it is enclosed in {} or on a separate line.

based on the current code, maybe the 'if' should be removed?
Assignee: mccabe → rogerl
Roger, can you look at this?
Status: NEW → ASSIGNED
My guess is that the ';' is unintended - we should just return the result of the
JS_DefineProperty call. I'll check that in.
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
done
You need to log in before you can comment on or make changes to this bug.