Reviewers: mvstanton,
Message:
Hey Michael,
Here's an easy starter for 2014... ;-)
The NULL check is redundant, as the IsAllocationSite() already tests whether
it's a HeapObject (which can never be NULL).
PTAL
-- Benedikt
Description:
Remove redundant NULL check.
Please review this at https://codereview.chromium.org/108363008/
SVN Base: https://v8.googlecode.com/svn/branches/bleeding_edge
Affected files (+1, -1 lines):
M src/objects.cc
Index: src/objects.cc
diff --git a/src/objects.cc b/src/objects.cc
index
cf21e802ebc14b32afd64beb0a891939bc213ea5..4eae7a864d5e45e76eeb62a22c2572b89b510267
100644
--- a/src/objects.cc
+++ b/src/objects.cc
@@ -12770,7 +12770,7 @@ const double AllocationSite::kPretenureRatio = 0.60;
bool AllocationSite::IsNestedSite() {
ASSERT(FLAG_trace_track_allocation_sites);
Object* current = GetHeap()->allocation_sites_list();
- while (current != NULL && current->IsAllocationSite()) {
+ while (current->IsAllocationSite()) {
AllocationSite* current_site = AllocationSite::cast(current);
if (current_site->nested_site() == this) {
return true;
--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
---
You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/groups/opt_out.
