Bug 74725 - Implement block-scoped const
Summary: Implement block-scoped const
Status: RESOLVED DUPLICATE of bug 31813
Alias: None
Product: WebKit
Classification: Unclassified
Component: JavaScriptCore (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Andy Wingo
URL:
Keywords:
Depends on: 74509 74718
Blocks:
  Show dependency treegraph
 
Reported: 2011-12-16 09:42 PST by Andy Wingo
Modified: 2015-11-30 11:00 PST (History)
8 users (show)

See Also:


Attachments
Patch (25.64 KB, patch)
2011-12-16 09:46 PST, Andy Wingo
no flags Details | Formatted Diff | Diff
Patch (25.08 KB, patch)
2012-01-18 10:39 PST, Andy Wingo
fpizlo: review-
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Andy Wingo 2011-12-16 09:42:41 PST
This is related to bug 31813, but isn't quite yet baked, as it doesn't do the temporal dead zone dance.  Also it is lacking a test suite, and relies on the flags and options from bug 74510.  But still, it's probably good to get early feedback on this approach.

Also I was getting a bit paranoid, having this patch only on my hard drive and not in the cloud-tubes :)
Comment 1 Andy Wingo 2011-12-16 09:46:13 PST
Created attachment 119625 [details]
Patch
Comment 2 Eric Seidel (no email) 2011-12-21 12:25:50 PST
I would strongly encourage you to engage with the various JSC folks on #webkit in order to get your set of JSC patches reviewed as quickly as possible. :)
Comment 3 Gavin Barraclough 2011-12-21 12:39:06 PST
(In reply to comment #2)
> I would strongly encourage you to engage with the various JSC folks on #webkit in order to get your set of JSC patches reviewed as quickly as possible. :)

OOI discussion of the implementation is taking place on these bugs - recent feedback on the design has been provided on Bug#74718.  But yes - #webkit is the place to be. :o)
Comment 4 Eric Seidel (no email) 2011-12-21 12:45:29 PST
(In reply to comment #3)
> (In reply to comment #2)
> > I would strongly encourage you to engage with the various JSC folks on #webkit in order to get your set of JSC patches reviewed as quickly as possible. :)
> 
> OOI discussion of the implementation is taking place on these bugs - recent feedback on the design has been provided on Bug#74718.  But yes - #webkit is the place to be. :o)

Great!  The first couple bugs of wingo's I ran into had no CC's.  Seeing further bugs I see he's engaged you all in this process.  Thank you!
Comment 5 Andy Wingo 2012-01-18 10:39:01 PST
Created attachment 122962 [details]
Patch
Comment 6 Filip Pizlo 2013-10-31 11:34:48 PDT
Comment on attachment 122962 [details]
Patch

This looks abandoned and in need of a massive rebase.
Comment 7 Emanuele Aina 2015-11-30 07:00:40 PST
Now that bug 31813 has been closed, is this bug (and bug 74509) still useful?
Comment 8 Saam Barati 2015-11-30 10:59:25 PST
(In reply to comment #7)
> Now that bug 31813 has been closed, is this bug (and bug 74509) still useful?

Nope. This is already implemented.
Comment 9 Saam Barati 2015-11-30 11:00:28 PST

*** This bug has been marked as a duplicate of bug 31813 ***