diff options
author | Thiago de Arruda <tpadilha84@gmail.com> | 2014-05-16 10:21:30 -0300 |
---|---|---|
committer | Thiago de Arruda <tpadilha84@gmail.com> | 2014-05-17 08:05:44 -0300 |
commit | 76a2fb5667461a8b7fa1abfb5a2c7381ced3f519 (patch) | |
tree | c5f06f37b7019107b9f70f24b5eb9951a5619e17 /src/nvim/os | |
parent | a8b0c9e576e9c0155546b03944314449d3f1a5c3 (diff) | |
download | rneovim-76a2fb5667461a8b7fa1abfb5a2c7381ced3f519.tar.gz rneovim-76a2fb5667461a8b7fa1abfb5a2c7381ced3f519.tar.bz2 rneovim-76a2fb5667461a8b7fa1abfb5a2c7381ced3f519.zip |
Use more descriptive names for API primitive types
Instead of exposing native C types to a public API that can be consumed by other
platforms, we are now using the following translation:
int64_t -> Integer
double -> Float
bool -> Boolean
Diffstat (limited to 'src/nvim/os')
-rw-r--r-- | src/nvim/os/msgpack_rpc.c | 48 | ||||
-rw-r--r-- | src/nvim/os/msgpack_rpc.h | 24 |
2 files changed, 36 insertions, 36 deletions
diff --git a/src/nvim/os/msgpack_rpc.c b/src/nvim/os/msgpack_rpc.c index 4e14373b88..ec9969f8f1 100644 --- a/src/nvim/os/msgpack_rpc.c +++ b/src/nvim/os/msgpack_rpc.c @@ -73,13 +73,13 @@ void msgpack_rpc_error(char *msg, msgpack_packer *res) msgpack_pack_nil(res); } -bool msgpack_rpc_to_bool(msgpack_object *obj, bool *arg) +bool msgpack_rpc_to_boolean(msgpack_object *obj, Boolean *arg) { *arg = obj->via.boolean; return obj->type == MSGPACK_OBJECT_BOOLEAN; } -bool msgpack_rpc_to_int64_t(msgpack_object *obj, int64_t *arg) +bool msgpack_rpc_to_integer(msgpack_object *obj, Integer *arg) { if (obj->type == MSGPACK_OBJECT_POSITIVE_INTEGER @@ -92,7 +92,7 @@ bool msgpack_rpc_to_int64_t(msgpack_object *obj, int64_t *arg) return obj->type == MSGPACK_OBJECT_NEGATIVE_INTEGER; } -bool msgpack_rpc_to_double(msgpack_object *obj, double *arg) +bool msgpack_rpc_to_float(msgpack_object *obj, Float *arg) { *arg = obj->via.dec; return obj->type == MSGPACK_OBJECT_DOUBLE; @@ -107,17 +107,17 @@ bool msgpack_rpc_to_string(msgpack_object *obj, String *arg) bool msgpack_rpc_to_buffer(msgpack_object *obj, Buffer *arg) { - return msgpack_rpc_to_int64_t(obj, arg); + return msgpack_rpc_to_integer(obj, arg); } bool msgpack_rpc_to_window(msgpack_object *obj, Window *arg) { - return msgpack_rpc_to_int64_t(obj, arg); + return msgpack_rpc_to_integer(obj, arg); } bool msgpack_rpc_to_tabpage(msgpack_object *obj, Tabpage *arg) { - return msgpack_rpc_to_int64_t(obj, arg); + return msgpack_rpc_to_integer(obj, arg); } bool msgpack_rpc_to_object(msgpack_object *obj, Object *arg) @@ -128,17 +128,17 @@ bool msgpack_rpc_to_object(msgpack_object *obj, Object *arg) return true; case MSGPACK_OBJECT_BOOLEAN: - arg->type = kObjectTypeBool; - return msgpack_rpc_to_bool(obj, &arg->data.boolean); + arg->type = kObjectTypeBoolean; + return msgpack_rpc_to_boolean(obj, &arg->data.boolean); case MSGPACK_OBJECT_POSITIVE_INTEGER: case MSGPACK_OBJECT_NEGATIVE_INTEGER: - arg->type = kObjectTypeInt; - return msgpack_rpc_to_int64_t(obj, &arg->data.integer); + arg->type = kObjectTypeInteger; + return msgpack_rpc_to_integer(obj, &arg->data.integer); case MSGPACK_OBJECT_DOUBLE: arg->type = kObjectTypeFloat; - return msgpack_rpc_to_double(obj, &arg->data.floating_point); + return msgpack_rpc_to_float(obj, &arg->data.floating); case MSGPACK_OBJECT_RAW: arg->type = kObjectTypeString; @@ -235,7 +235,7 @@ bool msgpack_rpc_to_dictionary(msgpack_object *obj, Dictionary *arg) return true; } -void msgpack_rpc_from_bool(bool result, msgpack_packer *res) +void msgpack_rpc_from_boolean(Boolean result, msgpack_packer *res) { if (result) { msgpack_pack_true(res); @@ -244,12 +244,12 @@ void msgpack_rpc_from_bool(bool result, msgpack_packer *res) } } -void msgpack_rpc_from_int64_t(int64_t result, msgpack_packer *res) +void msgpack_rpc_from_integer(Integer result, msgpack_packer *res) { msgpack_pack_int64(res, result); } -void msgpack_rpc_from_double(double result, msgpack_packer *res) +void msgpack_rpc_from_float(Float result, msgpack_packer *res) { msgpack_pack_double(res, result); } @@ -262,17 +262,17 @@ void msgpack_rpc_from_string(String result, msgpack_packer *res) void msgpack_rpc_from_buffer(Buffer result, msgpack_packer *res) { - msgpack_rpc_from_int64_t(result, res); + msgpack_rpc_from_integer(result, res); } void msgpack_rpc_from_window(Window result, msgpack_packer *res) { - msgpack_rpc_from_int64_t(result, res); + msgpack_rpc_from_integer(result, res); } void msgpack_rpc_from_tabpage(Tabpage result, msgpack_packer *res) { - msgpack_rpc_from_int64_t(result, res); + msgpack_rpc_from_integer(result, res); } void msgpack_rpc_from_object(Object result, msgpack_packer *res) @@ -282,16 +282,16 @@ void msgpack_rpc_from_object(Object result, msgpack_packer *res) msgpack_pack_nil(res); break; - case kObjectTypeBool: - msgpack_rpc_from_bool(result.data.boolean, res); + case kObjectTypeBoolean: + msgpack_rpc_from_boolean(result.data.boolean, res); break; - case kObjectTypeInt: - msgpack_rpc_from_int64_t(result.data.integer, res); + case kObjectTypeInteger: + msgpack_rpc_from_integer(result.data.integer, res); break; case kObjectTypeFloat: - msgpack_rpc_from_double(result.data.floating_point, res); + msgpack_rpc_from_float(result.data.floating, res); break; case kObjectTypeString: @@ -350,8 +350,8 @@ void msgpack_rpc_free_object(Object value) { switch (value.type) { case kObjectTypeNil: - case kObjectTypeBool: - case kObjectTypeInt: + case kObjectTypeBoolean: + case kObjectTypeInteger: case kObjectTypeFloat: break; diff --git a/src/nvim/os/msgpack_rpc.h b/src/nvim/os/msgpack_rpc.h index af2334f6dc..0e9d474d8d 100644 --- a/src/nvim/os/msgpack_rpc.h +++ b/src/nvim/os/msgpack_rpc.h @@ -38,9 +38,9 @@ void msgpack_rpc_error(char *msg, msgpack_packer *res); /// @param obj The object to convert /// @param[out] arg A pointer to the avalue /// @return true if the convertion succeeded, false otherwise -bool msgpack_rpc_to_bool(msgpack_object *obj, bool *arg); -bool msgpack_rpc_to_int64_t(msgpack_object *obj, int64_t *arg); -bool msgpack_rpc_to_double(msgpack_object *obj, double *arg); +bool msgpack_rpc_to_boolean(msgpack_object *obj, Boolean *arg); +bool msgpack_rpc_to_integer(msgpack_object *obj, Integer *arg); +bool msgpack_rpc_to_float(msgpack_object *obj, Float *arg); bool msgpack_rpc_to_position(msgpack_object *obj, Position *arg); bool msgpack_rpc_to_string(msgpack_object *obj, String *arg); bool msgpack_rpc_to_buffer(msgpack_object *obj, Buffer *arg); @@ -57,9 +57,9 @@ bool msgpack_rpc_to_dictionary(msgpack_object *obj, Dictionary *arg); /// /// @param result A pointer to the result /// @param res A packer that contains the response -void msgpack_rpc_from_bool(bool result, msgpack_packer *res); -void msgpack_rpc_from_int64_t(int64_t result, msgpack_packer *res); -void msgpack_rpc_from_double(double result, msgpack_packer *res); +void msgpack_rpc_from_boolean(Boolean result, msgpack_packer *res); +void msgpack_rpc_from_integer(Integer result, msgpack_packer *res); +void msgpack_rpc_from_float(Float result, msgpack_packer *res); void msgpack_rpc_from_position(Position result, msgpack_packer *res); void msgpack_rpc_from_string(String result, msgpack_packer *res); void msgpack_rpc_from_buffer(Buffer result, msgpack_packer *res); @@ -71,9 +71,9 @@ void msgpack_rpc_from_array(Array result, msgpack_packer *res); void msgpack_rpc_from_dictionary(Dictionary result, msgpack_packer *res); /// Helpers for initializing types that may be freed later -#define msgpack_rpc_init_bool -#define msgpack_rpc_init_int64_t -#define msgpack_rpc_init_double +#define msgpack_rpc_init_boolean +#define msgpack_rpc_init_integer +#define msgpack_rpc_init_float #define msgpack_rpc_init_position #define msgpack_rpc_init_string #define msgpack_rpc_init_buffer @@ -87,9 +87,9 @@ void msgpack_rpc_from_dictionary(Dictionary result, msgpack_packer *res); /// Helpers for freeing arguments/return value /// /// @param value The value to be freed -#define msgpack_rpc_free_bool(value) -#define msgpack_rpc_free_int64_t(value) -#define msgpack_rpc_free_double(value) +#define msgpack_rpc_free_boolean(value) +#define msgpack_rpc_free_integer(value) +#define msgpack_rpc_free_float(value) #define msgpack_rpc_free_position(value) // Strings are not copied from msgpack and so don't need to be freed(they // probably "live" in the msgpack streaming buffer) |