Minor clean-up of Type::Print

Minor clean-up of Type::Print

R=verwaest@chromium.org
BUG=

Committed: http://code.google.com/p/v8/source/detail?r=18010

[Toon Verwaest, 2013-11-18 17:05:45]

lgtm with nit

https://codereview.chromium.org/75423002/diff/1/src/types.cc
File src/types.cc (right):

https://codereview.chromium.org/75423002/diff/1/src/types.cc#newcode548
src/types.cc:548: for (unsigned i = 0; i < sizeof(bitset)*8; ++i) {
8 -> kBitsPerByte

Could also just do >>1 until the value is 0. (You still need i though, to get
the bitset_name).

[rossberg, 2013-11-18 17:37:03]

PTAL

https://codereview.chromium.org/75423002/diff/1/src/types.cc
File src/types.cc (right):

https://codereview.chromium.org/75423002/diff/1/src/types.cc#newcode548
src/types.cc:548: for (unsigned i = 0; i < sizeof(bitset)*8; ++i) {
On 2013/11/18 17:05:46, Toon Verwaest wrote:
> 8 -> kBitsPerByte
> 
> Could also just do >>1 until the value is 0. (You still need i though, to get
> the bitset_name).

Good idea (I don't even see why I still need i then).

[rossberg, 2013-11-22 12:38:56]

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