Revision: 10610
Author: [email protected]
Date: Mon Feb 6 04:11:40 2012
Log: Removed PropertyDetails::IsProperty.
This is just an intermediate step to remove IsRealProperty.
Review URL: https://chromiumcodereview.appspot.com/9325060
http://code.google.com/p/v8/source/detail?r=10610
Modified:
/branches/bleeding_edge/src/objects.cc
/branches/bleeding_edge/src/property-details.h
/branches/bleeding_edge/src/property.h
=======================================
--- /branches/bleeding_edge/src/objects.cc Fri Feb 3 05:37:13 2012
+++ /branches/bleeding_edge/src/objects.cc Mon Feb 6 04:11:40 2012
@@ -10360,7 +10360,7 @@
int result = 0;
for (int i = 0; i < descs->number_of_descriptors(); i++) {
PropertyDetails details(descs->GetDetails(i));
- if (details.IsProperty() && (details.attributes() & filter) == 0) {
+ if (descs->IsProperty(i) && (details.attributes() & filter) == 0) {
result++;
}
}
=======================================
--- /branches/bleeding_edge/src/property-details.h Fri Feb 3 05:37:13 2012
+++ /branches/bleeding_edge/src/property-details.h Mon Feb 6 04:11:40 2012
@@ -118,10 +118,6 @@
inline Smi* AsSmi();
PropertyType type() { return TypeField::decode(value_); }
-
- bool IsProperty() {
- return IsRealProperty(type());
- }
PropertyAttributes attributes() { return
AttributesField::decode(value_); }
=======================================
--- /branches/bleeding_edge/src/property.h Fri Feb 3 05:37:13 2012
+++ /branches/bleeding_edge/src/property.h Mon Feb 6 04:11:40 2012
@@ -264,7 +264,7 @@
// Is the result is a property excluding transitions and the null
// descriptor?
bool IsProperty() {
- return IsFound() && GetPropertyDetails().IsProperty();
+ return IsFound() && IsRealProperty(GetPropertyDetails().type());
}
bool IsCacheable() { return cacheable_; }
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
