TM/nanojit: merge TR whitespace changes in the i386 backend

RESOLVED FIXED

Status

()

Core
JavaScript Engine
RESOLVED FIXED
8 years ago
8 years ago

People

(Reporter: njn, Assigned: njn)

Tracking

Trunk
x86
Linux
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

8 years ago
Created attachment 390358 [details] [diff] [review]
patch

This patch just merges whitespace changes from TR to TM (well, I also added braces in one place and removed parentheses in one place) in order to bring them closer together.

Really the only thing to check here is that I based it off the right TR code, ie. it actually is bringing TM and TR closer together.
Attachment #390358 - Flags: review?(rreitmai)

Comment 1

8 years ago
Comment on attachment 390358 [details] [diff] [review]
patch

+'d ohhh how i love da whitespace.
Attachment #390358 - Flags: review?(rreitmai) → review+
(Assignee)

Comment 2

8 years ago
http://hg.mozilla.org/tracemonkey/rev/64b2ea3926de

BTW, I didn't completely sync TM and TR, because there were a handful of places
where TR's indentation was wrong and TM's was right.  So a small amount of
syncing on the TR side will still be necessary.  But this got rid of 97% of the
differences.

Comment 3

8 years ago
http://hg.mozilla.org/mozilla-central/rev/64b2ea3926de
Status: ASSIGNED → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.