From d8982a08591a36d781eddc49ce95c7ce9b04cf79 Mon Sep 17 00:00:00 2001 From: Mark Mentovai Date: Wed, 10 Aug 2016 23:51:20 -0400 Subject: [PATCH] minidump: Fix unit multiplier in variable names MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit There’s no behavior change, but two variables that store quantities in units of Hz were misnamed *_mhz. Change-Id: I8b7d9576a7638a4ee9789dc9d1050009ac009696 Reviewed-on: https://chromium-review.googlesource.com/368080 Reviewed-by: Scott Graham --- minidump/minidump_misc_info_writer.cc | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/minidump/minidump_misc_info_writer.cc b/minidump/minidump_misc_info_writer.cc index 4d3b1435..1d25b115 100644 --- a/minidump/minidump_misc_info_writer.cc +++ b/minidump/minidump_misc_info_writer.cc @@ -149,14 +149,14 @@ void MinidumpMiscInfoWriter::InitializeFromSnapshot( const SystemSnapshot* system_snapshot = process_snapshot->System(); - uint64_t current_mhz; - uint64_t max_mhz; - system_snapshot->CPUFrequency(¤t_mhz, &max_mhz); + uint64_t current_hz; + uint64_t max_hz; + system_snapshot->CPUFrequency(¤t_hz, &max_hz); const uint32_t kHzPerMHz = static_cast(1E6); SetProcessorPowerInfo( - InRangeCast(current_mhz / kHzPerMHz, + InRangeCast(current_hz / kHzPerMHz, std::numeric_limits::max()), - InRangeCast(max_mhz / kHzPerMHz, + InRangeCast(max_hz / kHzPerMHz, std::numeric_limits::max()), 0, 0,