New issue
Advanced search Search tips

Issue 761267 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Sep 2017
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Rename 'powerd_prefs' property in master configuration

Project Member Reported by sjg@chromium.org, Sep 1 2017

Issue description

The convention is to use hyphen rather than underscore, so we should rename this property.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Sep 6 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/platform2/+/3126664e62c29c8fe382c83a0dab27df519d8b0d

commit 3126664e62c29c8fe382c83a0dab27df519d8b0d
Author: Simon Glass <sjg@chromium.org>
Date: Wed Sep 06 19:42:47 2017

chromeos-config: powermanager: Rename powerd_prefs propery

The convention is to use hyphen rather than underscore for property names.
Rename this property to comply with that convention.

BUG= chromium:761267 
TEST=none

Change-Id: I1355f429b4b89a4c043d548ac4c8a7e7cda81cbc
Signed-off-by: Simon Glass <sjg@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/646996
Reviewed-by: Benjamin Gordon <bmgordon@chromium.org>
Reviewed-by: C Shapiro <shapiroc@google.com>

[modify] https://crrev.com/3126664e62c29c8fe382c83a0dab27df519d8b0d/chromeos-config/README.md
[modify] https://crrev.com/3126664e62c29c8fe382c83a0dab27df519d8b0d/power_manager/common/prefs.cc

Comment 2 by sjg@chromium.org, Sep 7 2017

Status: Fixed (was: Untriaged)

Sign in to add a comment