Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(65)

Unified Diff: src/compiler.h

Issue 3533007: Remove some unused fields from class CompilationInfo. (Closed)
Patch Set: Created 10 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | src/handles.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/compiler.h
diff --git a/src/compiler.h b/src/compiler.h
index ed26603f42c538f084392b3045448a490504d1a1..10d93dc017428dc1190c66d98d95c9f0a281174f 100644
--- a/src/compiler.h
+++ b/src/compiler.h
@@ -42,15 +42,11 @@ namespace internal {
class CompilationInfo BASE_EMBEDDED {
public:
// Lazy compilation of a JSFunction.
- CompilationInfo(Handle<JSFunction> closure,
- int loop_nesting,
- Handle<Object> receiver)
+ CompilationInfo(Handle<JSFunction> closure, int loop_nesting)
: closure_(closure),
function_(NULL),
is_eval_(false),
- loop_nesting_(loop_nesting),
- receiver_(receiver) {
- Initialize();
+ loop_nesting_(loop_nesting) {
ASSERT(!closure_.is_null() &&
shared_info_.is_null() &&
script_.is_null());
@@ -62,7 +58,6 @@ class CompilationInfo BASE_EMBEDDED {
function_(NULL),
is_eval_(false),
loop_nesting_(0) {
- Initialize();
ASSERT(closure_.is_null() &&
!shared_info_.is_null() &&
script_.is_null());
@@ -74,7 +69,6 @@ class CompilationInfo BASE_EMBEDDED {
function_(literal),
is_eval_(is_eval),
loop_nesting_(0) {
- Initialize();
ASSERT(closure_.is_null() &&
shared_info_.is_null() &&
!script_.is_null());
@@ -112,11 +106,6 @@ class CompilationInfo BASE_EMBEDDED {
// Simple accessors.
bool is_eval() { return is_eval_; }
int loop_nesting() { return loop_nesting_; }
- bool has_receiver() { return !receiver_.is_null(); }
- Handle<Object> receiver() { return receiver_; }
-
- bool has_this_properties() { return has_this_properties_; }
- void set_has_this_properties(bool flag) { has_this_properties_ = flag; }
bool has_global_object() {
return !closure().is_null() && (closure()->context()->global() != NULL);
@@ -126,18 +115,10 @@ class CompilationInfo BASE_EMBEDDED {
return has_global_object() ? closure()->context()->global() : NULL;
}
- bool has_globals() { return has_globals_; }
- void set_has_globals(bool flag) { has_globals_ = flag; }
-
// Derived accessors.
Scope* scope() { return function()->scope(); }
private:
- void Initialize() {
- has_this_properties_ = false;
- has_globals_ = false;
- }
-
Handle<JSFunction> closure_;
Handle<SharedFunctionInfo> shared_info_;
Handle<Script> script_;
@@ -147,11 +128,6 @@ class CompilationInfo BASE_EMBEDDED {
bool is_eval_;
int loop_nesting_;
- Handle<Object> receiver_;
-
- bool has_this_properties_;
- bool has_globals_;
-
DISALLOW_COPY_AND_ASSIGN(CompilationInfo);
};
« no previous file with comments | « no previous file | src/handles.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698