Closed Bug 888469 Opened 7 years ago Closed 7 years ago

remove dead bits and parameters in the frontend

Categories

(Core :: JavaScript Engine, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla25

People

(Reporter: luke, Assigned: luke)

Details

Attachments

(5 files)

Specifically:
 - PND_BLOCKCHILD / InBlockBool
 - the 'strict' arg in some places
 - the function name in some places
 - startOffset
Attachment #769135 - Flags: review?(bhackett1024)
Attachment #769136 - Flags: review?(bhackett1024)
Attachment #769138 - Flags: review?(bhackett1024)
The one interesting thing in this patch is that there is one place where there is the decision between creating a RootedPropertyName just to pass a HandlePropertyName to a function that just does a few comparisons, so I opted to pass the PropertyName* but I could switch if you felt strongly.
Attachment #769143 - Flags: review?(bhackett1024)
Attachment #769144 - Flags: review?(bhackett1024)
Attachment #769135 - Flags: review?(bhackett1024) → review+
Attachment #769136 - Flags: review?(bhackett1024) → review+
Comment on attachment 769138 [details] [diff] [review]
rm dead assignment to 'body'

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

::: js/src/frontend/Parser.cpp
@@ -1909,5 @@
>  
>      // If the outer scope is strict, immediately parse the function in strict
>      // mode. Otherwise, we parse it normally. If we see a "use strict"
>      // directive, we backup and reparse it as strict.
> -    handler.setFunctionBody(pn, null());

Maybe a brief comment on the call to this function that still exists below?
Attachment #769138 - Flags: review?(bhackett1024) → review+
Attachment #769143 - Flags: review?(bhackett1024) → review+
Attachment #769144 - Flags: review?(bhackett1024) → review+
You need to log in before you can comment on or make changes to this bug.