179366 - --> after whitespace after line start should mean comments to line end

Status: VERIFIED FIXED

(Rhino Graveyard :: Core, defect)

Description

[Igor Bukanov]

User-Agent:       Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.0.1) Gecko/20021003
Build Identifier: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.0.1) Gecko/20021003

This is a Rhino version of bug 31255.

Rhino should follow SpiderMonkey and allow to treat --> after a possible
whitespace after line start to mean comments until line end.


Reproducible: Always

Steps to Reproduce:

Comment 1

[Igor Bukanov]

Attachment: Pure JS test case to check that ^\s*--> means comments

Tests in bug 31255 use scripts embedded in HTML, this is a pure JS version that
can be run against stand-along script engine.

Comment 2

[Igor Bukanov]

Attachment: patch to omj/TokenStream.java

This is a replica of the SM solution

Comment 3

[Phil Schwartau]

Igor's testcase added to JS testsuite. As is customary, I have 
named it after the original bug:

      mozilla/js/tests/js1_5/Regress/regress-31255.js

Comment 4

[Igor Bukanov]

I commited the fix

Comment 5

[Phil Schwartau]

Marking Verified FIXED.

The above testcase used to fail in Rhino; now it passes in both
compiled and interpreted mode on WinNT -

Comment 6

[Igor Bukanov]

Targeting as resolved against 1.5R4