Closed Bug 1450574 Opened 2 years ago Closed 2 years ago

More cleanup after expression closure removal

Categories

(Core :: JavaScript Engine, defect, P2)

defect

Tracking

()

RESOLVED FIXED
mozilla61
Tracking Status
firefox61 --- fixed

People

(Reporter: evilpie, Assigned: evilpie)

References

Details

Attachments

(3 files, 1 obsolete file)

There are few more things that can be obviously removed. 

- ExpressionClosureHandling in the Parser
- Probably various other simplifications in the parser that aren't obvious to me
- DeprecatedLanguageExtension telemetry code that only handled expression closures

More .. ?
Actually ExpressionClosureHandling might still be needed for arrow function expression, I am not sure.
Assignee: nobody → evilpies
Attachment #8964876 - Flags: review?(jdemooij)
Attachment #8964876 - Attachment is obsolete: true
Attachment #8964876 - Flags: review?(jdemooij)
Attachment #8964878 - Flags: review?(jdemooij)
Comment on attachment 8964878 [details] [diff] [review]
Remove NoExpressionClosure FunctionSyntaxKind values

Review of attachment 8964878 [details] [diff] [review]:
-----------------------------------------------------------------

*steal*
Attachment #8964878 - Flags: review?(jdemooij) → review+
Priority: -- → P2
We should probably make FunctionSyntaxKind an enum class.
Attachment #8965027 - Flags: review?(jorendorff)
Pushed by evilpies@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/0a136c8bccf1
Remove NoExpressionClosure FunctionSyntaxKind values. r=jorendorff
Keywords: leave-open
Comment on attachment 8965027 [details] [diff] [review]
Reove expressionClosureHandling

Review of attachment 8965027 [details] [diff] [review]:
-----------------------------------------------------------------

Great, thanks.
Attachment #8965027 - Flags: review?(jorendorff) → review+
Pushed by evilpies@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/42b231ec2fbc
Remove expressionClosureHandling from Parser. r=jorendorff
Testing this on try before asking for review.
Attachment #8965512 - Flags: review?(jdemooij)
Comment on attachment 8965512 [details] [diff] [review]
Remove DeprecatedLanguageExtension telemetry code

Review of attachment 8965512 [details] [diff] [review]:
-----------------------------------------------------------------

Excellent.
Attachment #8965512 - Flags: review?(jdemooij) → review+
Pushed by archaeopteryx@coole-files.de:
https://hg.mozilla.org/integration/autoland/rev/14f8ed32f8f0
Remove expressionClosureHandling from Parser: fix leftover merge conflict CLOSED TREE
Keywords: leave-open
Pushed by evilpies@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/84acd9df2fe4
Remove DeprecatedLanguageExtension telemetry code. r=jandem
https://hg.mozilla.org/mozilla-central/rev/14f8ed32f8f0
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla61
Blocks: 1453006
You need to log in before you can comment on or make changes to this bug.