From cb1c02d80001ae9798899f25e74fe0ea265688e8 Mon Sep 17 00:00:00 2001 From: David Pursehouse Date: Fri, 12 Aug 2016 12:40:46 +0900 Subject: [PATCH] RestApi#isExpected: Use Response.SC_ enum values where possible GWT's Response class provides enum values for all the HTTP status codes used here except 422 and 429. Use the enums where possible. Change-Id: If0076127c477643ab3d1d73ac13de4dc01499238 --- .../java/com/google/gerrit/client/rpc/RestApi.java | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/gerrit-gwtui/src/main/java/com/google/gerrit/client/rpc/RestApi.java b/gerrit-gwtui/src/main/java/com/google/gerrit/client/rpc/RestApi.java index a0e25ad2ac..bcf1ab4bab 100644 --- a/gerrit-gwtui/src/main/java/com/google/gerrit/client/rpc/RestApi.java +++ b/gerrit-gwtui/src/main/java/com/google/gerrit/client/rpc/RestApi.java @@ -89,13 +89,13 @@ public class RestApi { public static boolean isExpected(int statusCode) { switch (statusCode) { case SC_UNAVAILABLE: - case 400: // Bad Request - case 401: // Unauthorized - case 403: // Forbidden - case 404: // Not Found - case 405: // Method Not Allowed - case 409: // Conflict - case 412: // Precondition Failed + case Response.SC_BAD_REQUEST: + case Response.SC_UNAUTHORIZED: + case Response.SC_FORBIDDEN: + case Response.SC_NOT_FOUND: + case Response.SC_METHOD_NOT_ALLOWED: + case Response.SC_CONFLICT: + case Response.SC_PRECONDITION_FAILED: case 422: // Unprocessable Entity case 429: // Too Many Requests (RFC 6585) return true;