tree ddca14b1346e2779950eb3c58a58998a54e96106
parent d51532dc9beef435bb59342c8ae235923e7a8d4e
author Ereth McKnight-MacNeil <ereth@google.com> 1611214264 -0800
committer Commit Bot <commit-bot@chromium.org> 1612466390 +0000

power: shellcheck fixes in send_metrics_on_resume

Declare and assign shell variables separately. This prevents unsafe word
splitting under Dash. (SC2155)

False positives for single quotes disabled via derictives. (SC2016)

Quoting for $@. (SC2068)

Extra quotes around literal {}. (SC1083)

Drop $ where unneeded in arithmetic expressions. (SC2004)

Quoting for command expansion. (SC2046)

Stop using -a. (SC2166)

BUG=chromium:1168555
TEST=test_that [...] power_Resume, confirm in chrome://histograms

Change-Id: Idece53074b00d437dee3ad49ef35b165df1c699e
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform2/+/2645956
Tested-by: Ereth McKnight-MacNeil <ereth@chromium.org>
Reviewed-by: Puthikorn Voravootivat <puthik@chromium.org>
Reviewed-by: Eric Caruso <ejcaruso@chromium.org>
Commit-Queue: Ereth McKnight-MacNeil <ereth@chromium.org>
(cherry picked from commit d6d747a4c112fbda5e4f61f3166f328c8d82c5cc)
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform2/+/2666512
Auto-Submit: Ereth McKnight-MacNeil <ereth@chromium.org>