Issue 2600853004: Minor base/numerics cleanup

Issue 2600853004: Minor base/numerics cleanup (Closed)

Created:
3 years, 12 months ago by jschuh

Modified:
3 years, 11 months ago

Reviewers:
Tom Sepez

CC:
chromium-reviews, vmpstr+watch_chromium.org

Target Ref:
refs/pending/heads/master

Project:
chromium

Visibility:
Public.

Minor base/numerics cleanup Simplifying some arithmetic and removing dead code. TBR=tsepez Committed: https://crrev.com/6dfadc0dc4a5fc68e5ce15db23f89192ff546961 Cr-Commit-Position: refs/heads/master@{#440720}

Created: 3 years, 12 months ago

Download [raw] [tar.bz2]

		Unified diffs	Side-by-side diffs	Delta from patch set	Stats (+8 lines, -22 lines)			Patch
	M	base/numerics/safe_conversions_impl.h	View		2 chunks	+1 line, -14 lines	0 comments	Download
	M	base/numerics/safe_math_impl.h	View		5 chunks	+7 lines, -8 lines	0 comments	Download

Total messages: 13 (10 generated)

commit-bot: I haz the power

CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2600853004/80001

3 years, 12 months ago (2016-12-26 17:26:56 UTC) #8

commit-bot: I haz the power

3 years, 11 months ago (2017-01-02 15:45:11 UTC) #13

Message was sent while issue was closed.

Patchset 1 (id:??) landed as
https://crrev.com/6dfadc0dc4a5fc68e5ce15db23f89192ff546961
Cr-Commit-Position: refs/heads/master@{#440720}