CfgLister case LIR_jtbl asserts.

RESOLVED FIXED

Status

Core Graveyard
Nanojit
RESOLVED FIXED
6 years ago
3 years ago

People

(Reporter: Edwin Smith, Assigned: Edwin Smith)

Tracking

(Blocks: 1 bug)

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: fixed-in-nanojit, fixed-in-tamarin)

Attachments

(1 attachment)

(Assignee)

Description

6 years ago
Created attachment 566618 [details] [diff] [review]
Obvious fix, plus a bit of whitespace cleanup.

It is calling ins->getTarget() instead of ins->getTarget(i).
(Assignee)

Updated

6 years ago
Attachment #566618 - Attachment is patch: true
Attachment #566618 - Flags: review?(rreitmai)
(Assignee)

Updated

6 years ago
Blocks: 639475
(Assignee)

Updated

6 years ago
Assignee: nobody → edwsmith

Comment 1

6 years ago
Comment on attachment 566618 [details] [diff] [review]
Obvious fix, plus a bit of whitespace cleanup.

r+.

BTW, '[diff] git=true' in your .hgrc file will get rid of the unhelpful file names in bugzilla diff display, assuming this was an hg derived patch.  See 'hg help diffs'
Attachment #566618 - Flags: review?(rreitmai) → review+
(Assignee)

Comment 2

6 years ago
(In reply to Rick Reitmaier from comment #1)
> Comment on attachment 566618 [details] [diff] [review] [diff] [details] [review]
> Obvious fix, plus a bit of whitespace cleanup.
> 
> r+.
> 
> BTW, '[diff] git=true' in your .hgrc file will get rid of the unhelpful file
> names in bugzilla diff display, assuming this was an hg derived patch.  See
> 'hg help diffs'

yeah, willfix.  need to sync my .hgrc across a couple machines.

Comment 3

6 years ago
changeset: 6639:0a6a685cecbc
user:      Edwin Smith <edwsmith>
summary:   Bug 694119 - "CfgLister case LIR_jtbl asserts." [r=rreitmai]

http://hg.mozilla.org/tamarin-redux/rev/0a6a685cecbc
(Assignee)

Updated

6 years ago
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
(Assignee)

Updated

6 years ago
Whiteboard: fixed-in-nanojit, fixed-in-tamarin
Component: Nanojit → Nanojit
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.