Closed Bug 707117 Opened 13 years ago Closed 13 years ago

a small 'typo' in lirasm at assemble_load

Categories

(Core Graveyard :: Nanojit, defect)

Product:
Core Graveyard
Component:
Nanojit
Version:
8 Branch
Platform:
x86
Linux
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: gnudles, Unassigned)

Details

User Agent: Mozilla/5.0 (Ubuntu; X11; Linux i686; rv:8.0) Gecko/20100101 Firefox/8.0
Build ID: 20111115184056

Steps to reproduce:

skimmed over the code:
/js/src/lirasm/lirasm.cpp line 664


Actual results:

my eyes spotted two identical lines


Expected results:

I think It should be 
        mTokens[1].find("0X") == 0 ||
instead of 
        mTokens[1].find("0x") == 0 ||
Component: General → Nanojit
Product: Firefox → Core
QA Contact: general → nanojit
Hi, a n00b here. Are you talking about code on lines 558-559?

558         mTokens[1].find("0x") == 0 ||
559         mTokens[1].find("0x") == 0 ||

in this file: http://mxr.mozilla.org/mozilla1.9.2/source/js/src/lirasm/lirasm.cpp

Then I guess you're right.
Good eyes.

NC:
http://hg.mozilla.org/projects/nanojit-central/rev/19a1d081200c

TR:
http://hg.mozilla.org/tamarin-redux/rev/08c43603eb30
Status: UNCONFIRMED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.