From ad5e5d019f9cd3e988751646c2ae2633d1dc27e2 Mon Sep 17 00:00:00 2001 From: Luis Vidal Date: Wed, 13 Apr 2016 17:37:33 -0700 Subject: Remove public default constructor from CMWeatherManager.RequestStatus [1/2] RequestStatus holds only final constants and should not provide public constructors. This class also should be final. This patch also fix minor typos on RequestStatus javadoc Change-Id: I0deb1d6a9b23eed83451dec352bb4c410e2378d6 TICKET: CYNGNOS-2441 --- sdk/src/java/cyanogenmod/weather/CMWeatherManager.java | 11 +++++++---- 1 file changed, 7 insertions(+), 4 deletions(-) (limited to 'sdk') diff --git a/sdk/src/java/cyanogenmod/weather/CMWeatherManager.java b/sdk/src/java/cyanogenmod/weather/CMWeatherManager.java index 8292b58..89b1740 100644 --- a/sdk/src/java/cyanogenmod/weather/CMWeatherManager.java +++ b/sdk/src/java/cyanogenmod/weather/CMWeatherManager.java @@ -57,13 +57,16 @@ public class CMWeatherManager { /** * The different request statuses */ - public static class RequestStatus { + public static final class RequestStatus { + + private RequestStatus() {} + /** - * Request Successfully completed + * Request successfully completed */ public static final int COMPLETED = 1; /** - * An error occurred while trying to honor the request. + * An error occurred while trying to honor the request */ public static final int FAILED = -1; /** @@ -71,7 +74,7 @@ public class CMWeatherManager { */ public static final int SUBMITTED_TOO_SOON = -2; /** - * Another request in already in progress + * Another request is already in progress */ public static final int ALREADY_IN_PROGRESS = -3; /** -- cgit v1.1