As a security precaution, we have turned on the setting "Require API key authentication for API requests" for everyone. If this has broken something, please contact
Last Comment Bug 442358 - jsinterp control flow doesn't flow as specified
: jsinterp control flow doesn't flow as specified
: fixed1.9.0.2
Product: Core
Classification: Components
Component: JavaScript Engine (show other bugs)
: Trunk
: x86 Mac OS X
: -- normal (vote)
: ---
Assigned To: Blake Kaplan (:mrbkap)
: Jason Orendorff [:jorendorff]
Depends on:
Blocks: 432917
  Show dependency treegraph
Reported: 2008-06-27 13:44 PDT by (dormant account)
Modified: 2008-08-11 11:27 PDT (History)
7 users (show)
sayrer: wanted1.9.1+
bob: in‑testsuite-
bob: in‑litmus-
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

Obvious patch (969 bytes, patch)
2008-07-08 03:22 PDT, Blake Kaplan (:mrbkap)
igor: review+
samuel.sidler+old: approval1.9.0.2+
Details | Diff | Splinter Review

Description User image (dormant account) 2008-06-27 13:44:12 PDT is a return statement and there is another one a few lines down. 

This breaks the rule that code most flow through exit2 at
Comment 1 User image Blake Kaplan (:mrbkap) 2008-07-08 03:22:51 PDT
Created attachment 328463 [details] [diff] [review]
Obvious patch
Comment 2 User image Blake Kaplan (:mrbkap) 2008-07-08 05:09:01 PDT
Pushed as changeset 32207c9fccb6.
Comment 3 User image Blake Kaplan (:mrbkap) 2008-08-04 18:18:06 PDT
Comment on attachment 328463 [details] [diff] [review]
Obvious patch

I'm guessing we want this in 1.9.
Comment 4 User image Samuel Sidler (old account; do not CC) 2008-08-06 16:15:51 PDT
Comment on attachment 328463 [details] [diff] [review]
Obvious patch

Approved for Please land in CVS. a=ss
Comment 5 User image Blake Kaplan (:mrbkap) 2008-08-11 11:27:01 PDT
Fix checked into the 1.9 branch.

Note You need to log in before you can comment on or make changes to this bug.