Skip to content

Commit

Permalink
folly: fix initialization for CLOCK_PROCESS_CPUTIME_ID emulation on m…
Browse files Browse the repository at this point in the history
…acOS

Summary:
this fixes TimeTest on macOS.

The problem was that `task_info_count` is an in/out parameter and it wasn't correctly initialized due to a typo.  The lack of initialization meant that the `task_info()` call would fail at runtime.

Reviewed By: yfeldblum

Differential Revision: D3576262

fbshipit-source-id: 8388fa3c5020309d64a0fa7e01fd6cea80200219
  • Loading branch information
wez authored and Facebook Github Bot 2 committed Jul 17, 2016
1 parent 92fd108 commit 4592f5e
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions folly/portability/Time.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -50,8 +50,7 @@ static std::chrono::nanoseconds time_value_to_ns(time_value_t t) {
static int clock_process_cputime(struct timespec* ts) {
// Get CPU usage for live threads.
task_thread_times_info thread_times_info;
mach_msg_type_number_t thread_times_info_count;
TASK_THREAD_TIMES_INFO_COUNT;
mach_msg_type_number_t thread_times_info_count = TASK_THREAD_TIMES_INFO_COUNT;
kern_return_t kern_result = task_info(
mach_task_self(),
TASK_THREAD_TIMES_INFO,
Expand Down

0 comments on commit 4592f5e

Please sign in to comment.