Move kGroupCount out of DependencyGroup enum.

Move kGroupCount out of DependencyGroup enum.

BUG=
R=svenpanne@chromium.org, yangguo@chromium.org

Committed: https://code.google.com/p/v8/source/detail?r=23366

[Yang, 2014-08-25 12:12:43]

https://codereview.chromium.org/488993004/diff/1/src/objects.h
File src/objects.h (right):

https://codereview.chromium.org/488993004/diff/1/src/objects.h#newcode5962
src/objects.h:5962: enum { kGroupCount = kAllocationSiteTransitionChangedGroup +
1 };
does this have to be an enum though? Can it be simply a constant?

[Vyacheslav Egorov (Google), 2014-08-25 12:16:51]

https://codereview.chromium.org/488993004/diff/1/src/objects.h
File src/objects.h (right):

https://codereview.chromium.org/488993004/diff/1/src/objects.h#newcode5962
src/objects.h:5962: enum { kGroupCount = kAllocationSiteTransitionChangedGroup +
1 };
On 2014/08/25 12:12:43, Yang wrote:
> does this have to be an enum though? Can it be simply a constant?

Fixed.

(I became accustomed to declaring constants like this via enums recently)

[Vyacheslav Egorov (Google), 2014-08-25 12:17:24]

+svenpanne, +verwaest as reviewers

[Yang, 2014-08-25 12:23:05]

On 2014/08/25 12:17:24, Vyacheslav Egorov (Google wrote:
> +svenpanne, +verwaest as reviewers

lgtm

[Sven Panne, 2014-08-25 13:18:30]

LGTM from my side, too.

https://codereview.chromium.org/488993004/diff/20001/src/objects.h
File src/objects.h (right):

https://codereview.chromium.org/488993004/diff/20001/src/objects.h#newcode5962
src/objects.h:5962: static const int kGroupCount =
kAllocationSiteTransitionChangedGroup + 1;
One bright day we'll have constexpr... ;-)

[Vyacheslav Egorov (Google), 2014-08-25 15:41:22]

Committed patchset #2 manually as 23366 (presubmit successful).