Fix GCC warning in heap-profile-table.cc.

Fix GCC warning in heap-profile-table.cc.

The current code throws the following warning:

../../third_party/tcmalloc/chromium/src/heap-profile-table.cc:522:42: warning:
  format '%lx' expects argument of type 'long unsigned int', but argument 4
  has type 'const void*' [-Wformat=]

BUG=None

Committed: https://crrev.com/9460c33b35c889740a3dc659eb9c9d1161fd155d
Cr-Commit-Position: refs/heads/master@{#334813}

[Thiemo Nagel, 2015-06-17 09:43:20]

tnagel@chromium.org changed reviewers:
+ jochen@chromium.org

[Thiemo Nagel, 2015-06-17 09:43:20]

Hi Jochen,

could you please take a look?

Cheers,
Thiemo

[jochen (gone - plz use gerrit), 2015-06-17 09:48:09]

https://codereview.chromium.org/1180903010/diff/1/third_party/tcmalloc/chromi...
File third_party/tcmalloc/chromium/src/heap-profile-table.cc (right):

https://codereview.chromium.org/1180903010/diff/1/third_party/tcmalloc/chromi...
third_party/tcmalloc/chromium/src/heap-profile-table.cc:522: snprintf(addr, 16,
"0x%08" PRIxPTR, (uintptr_t)ptr);
please use reinterpret_cast<intptr_t>

[Thiemo Nagel, 2015-06-17 11:22:46]

Thank you!

https://codereview.chromium.org/1180903010/diff/1/third_party/tcmalloc/chromi...
File third_party/tcmalloc/chromium/src/heap-profile-table.cc (right):

https://codereview.chromium.org/1180903010/diff/1/third_party/tcmalloc/chromi...
third_party/tcmalloc/chromium/src/heap-profile-table.cc:522: snprintf(addr, 16,
"0x%08" PRIxPTR, (uintptr_t)ptr);
On 2015/06/17 09:48:09, jochen wrote:
> please use reinterpret_cast<intptr_t>

Just out of curiosity: Why do you prefer intptr_t over uintptr_t?

[jochen (gone - plz use gerrit), 2015-06-17 11:27:34]

On 2015/06/17 at 11:22:46, tnagel wrote:
> Thank you!
> 
>
https://codereview.chromium.org/1180903010/diff/1/third_party/tcmalloc/chromi...
> File third_party/tcmalloc/chromium/src/heap-profile-table.cc (right):
> 
>
https://codereview.chromium.org/1180903010/diff/1/third_party/tcmalloc/chromi...
> third_party/tcmalloc/chromium/src/heap-profile-table.cc:522: snprintf(addr,
16, "0x%08" PRIxPTR, (uintptr_t)ptr);
> On 2015/06/17 09:48:09, jochen wrote:
> > please use reinterpret_cast<intptr_t>
> 
> Just out of curiosity: Why do you prefer intptr_t over uintptr_t?

Sorry, typo. You're right that it should be uintptr_t

[Thiemo Nagel, 2015-06-17 11:32:52]

> > Just out of curiosity: Why do you prefer intptr_t over uintptr_t?
> 
> Sorry, typo. You're right that it should be uintptr_t

Alright.  (Not that it matters much, I guess.)

I've updated the CL again.  Could you please take another look?

[jochen (gone - plz use gerrit), 2015-06-17 11:37:00]

lgtm

[Thiemo Nagel, 2015-06-17 11:49:07]

The CQ bit was checked by tnagel@chromium.org

[commit-bot: I haz the power, 2015-06-17 11:49:27]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1180903010/40001

[commit-bot: I haz the power, 2015-06-17 12:09:34]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-06-17 12:10:25]

Patchset 3 (id:??) landed as
https://crrev.com/9460c33b35c889740a3dc659eb9c9d1161fd155d
Cr-Commit-Position: refs/heads/master@{#334813}