Change reinterpret_cast to std::copy to avoid undefined behavior

Change reinterpret_cast to std::copy to avoid undefined behavior

It seems like the reinterpret_cast is not valid here and should be
changed to std::copy or memcpy. Probably dereferencing the given pointer
is undefined behavior:
http://en.cppreference.com/w/cpp/language/reinterpret_cast

BUG=722170
Review-Url: https://codereview.chromium.org/2876373002
Cr-Commit-Position: refs/heads/master@{#472717}
Committed: https://chromium.googlesource.com/chromium/src/+/88b9bc14b5d592e258623f27363d768ec25ce05b

[sabbakumov, 2017-05-15 04:31:05]

Description was changed from

==========
Change reinterpret_cast to std::copy to avoid undefined behavior

It seems like the reinterpret_cast is not valid here and should be
changed to std::copy or memcpy. Probably dereferencing the given pointer
is undefined behavior:
http://en.cppreference.com/w/cpp/language/reinterpret_cast

BUG=722170
==========

to

==========
Change reinterpret_cast to std::copy to avoid undefined behavior

It seems like the reinterpret_cast is not valid here and should be
changed to std::copy or memcpy. Probably dereferencing the given pointer
is undefined behavior:
http://en.cppreference.com/w/cpp/language/reinterpret_cast

BUG=722170
==========

[sabbakumov, 2017-05-15 04:31:05]

sabbakumov@yandex-team.ru changed reviewers:
+ rkaplow@chromium.org

[rkaplow, 2017-05-17 14:11:28]

lgtm

[sabbakumov, 2017-05-18 02:56:47]

The CQ bit was checked by sabbakumov@yandex-team.ru

[commit-bot: I haz the power, 2017-05-18 02:58:50]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-05-18 07:00:33]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1495076207799100, "parent_rev":
"0a2a69a314c707ddfb83387690242dc86b088eed", "commit_rev":
"88b9bc14b5d592e258623f27363d768ec25ce05b"}

[commit-bot: I haz the power, 2017-05-18 07:00:46]

Description was changed from

==========
Change reinterpret_cast to std::copy to avoid undefined behavior

It seems like the reinterpret_cast is not valid here and should be
changed to std::copy or memcpy. Probably dereferencing the given pointer
is undefined behavior:
http://en.cppreference.com/w/cpp/language/reinterpret_cast

BUG=722170
==========

to

==========
Change reinterpret_cast to std::copy to avoid undefined behavior

It seems like the reinterpret_cast is not valid here and should be
changed to std::copy or memcpy. Probably dereferencing the given pointer
is undefined behavior:
http://en.cppreference.com/w/cpp/language/reinterpret_cast

BUG=722170

Review-Url: https://codereview.chromium.org/2876373002
Cr-Commit-Position: refs/heads/master@{#472717}
Committed:
https://chromium.googlesource.com/chromium/src/+/88b9bc14b5d592e258623f27363d...
==========

[commit-bot: I haz the power, 2017-05-18 07:00:47]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/88b9bc14b5d592e258623f27363d...