summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorRĂ©mi Verschelde <rverschelde@gmail.com>2019-09-02 07:28:18 +0200
committerGitHub <noreply@github.com>2019-09-02 07:28:18 +0200
commit7e908c957b2a88c16c6345a3b92d76195d50cf9c (patch)
tree7d6e88a551f585323596482449705fe4286fe33b
parent8b15ac770c8b4955c1872a253980c9792800374b (diff)
parent7e01986ce1341016f91d3dcaf62ab51c989b3d1f (diff)
Merge pull request #31875 from Calinou/jsonrpc-uppercase-constants
Make JSONRPC constants uppercase for consistency with other classes
-rw-r--r--modules/jsonrpc/jsonrpc.cpp18
-rw-r--r--modules/jsonrpc/jsonrpc.h10
2 files changed, 14 insertions, 14 deletions
diff --git a/modules/jsonrpc/jsonrpc.cpp b/modules/jsonrpc/jsonrpc.cpp
index b18b48d1b0..e1bba60f2f 100644
--- a/modules/jsonrpc/jsonrpc.cpp
+++ b/modules/jsonrpc/jsonrpc.cpp
@@ -47,11 +47,11 @@ void JSONRPC::_bind_methods() {
ClassDB::bind_method(D_METHOD("make_notification", "method", "params"), &JSONRPC::make_notification);
ClassDB::bind_method(D_METHOD("make_response_error", "code", "message", "id"), &JSONRPC::make_response_error, DEFVAL(Variant()));
- BIND_ENUM_CONSTANT(ParseError)
- BIND_ENUM_CONSTANT(InvalidRequest)
- BIND_ENUM_CONSTANT(MethodNotFound)
- BIND_ENUM_CONSTANT(InvalidParams)
- BIND_ENUM_CONSTANT(InternalError)
+ BIND_ENUM_CONSTANT(PARSE_ERROR)
+ BIND_ENUM_CONSTANT(INVALID_REQUEST)
+ BIND_ENUM_CONSTANT(METHOD_NOT_FOUND)
+ BIND_ENUM_CONSTANT(INVALID_PARAMS)
+ BIND_ENUM_CONSTANT(INTERNAL_ERROR)
}
Dictionary JSONRPC::make_response_error(int p_code, const String &p_message, const Variant &p_id) const {
@@ -120,7 +120,7 @@ Variant JSONRPC::process_action(const Variant &p_action, bool p_process_arr_elem
}
if (object == NULL || !object->has_method(method)) {
- ret = make_response_error(JSONRPC::MethodNotFound, "Method not found", id);
+ ret = make_response_error(JSONRPC::METHOD_NOT_FOUND, "Method not found", id);
} else {
Variant call_ret = object->callv(method, args);
if (id.get_type() != Variant::NIL) {
@@ -138,10 +138,10 @@ Variant JSONRPC::process_action(const Variant &p_action, bool p_process_arr_elem
}
ret = arr_ret;
} else {
- ret = make_response_error(JSONRPC::InvalidRequest, "Invalid Request");
+ ret = make_response_error(JSONRPC::INVALID_REQUEST, "Invalid Request");
}
} else {
- ret = make_response_error(JSONRPC::InvalidRequest, "Invalid Request");
+ ret = make_response_error(JSONRPC::INVALID_REQUEST, "Invalid Request");
}
return ret;
}
@@ -155,7 +155,7 @@ String JSONRPC::process_string(const String &p_input) {
String err_message;
int err_line;
if (OK != JSON::parse(p_input, input, err_message, err_line)) {
- ret = make_response_error(JSONRPC::ParseError, "Parse error");
+ ret = make_response_error(JSONRPC::PARSE_ERROR, "Parse error");
} else {
ret = process_action(input, true);
}
diff --git a/modules/jsonrpc/jsonrpc.h b/modules/jsonrpc/jsonrpc.h
index bcb34ecc65..91897d0b55 100644
--- a/modules/jsonrpc/jsonrpc.h
+++ b/modules/jsonrpc/jsonrpc.h
@@ -47,11 +47,11 @@ public:
~JSONRPC();
enum ErrorCode {
- ParseError = -32700,
- InvalidRequest = -32600,
- MethodNotFound = -32601,
- InvalidParams = -32602,
- InternalError = -32603,
+ PARSE_ERROR = -32700,
+ INVALID_REQUEST = -32600,
+ METHOD_NOT_FOUND = -32601,
+ INVALID_PARAMS = -32602,
+ INTERNAL_ERROR = -32603,
};
Dictionary make_response_error(int p_code, const String &p_message, const Variant &p_id = Variant()) const;