null+****@clear*****
null+****@clear*****
2012年 3月 29日 (木) 12:53:42 JST
Susumu Yata 2012-03-29 12:53:42 +0900 (Thu, 29 Mar 2012)
New Revision: a34f0c0b6dc0c03b0806209a0f4cda2728a3b3e2
Log:
Add const specifiers and type casts from void *.
Modified files:
lib/hash.c
Modified: lib/hash.c (+13 -11)
===================================================================
--- lib/hash.c 2012-03-29 11:45:26 +0900 (f0522b4)
+++ lib/hash.c 2012-03-29 12:53:42 +0900 (2952e55)
@@ -320,13 +320,15 @@ grn_array_init(grn_ctx *ctx, grn_array *array,
grn_array *
grn_array_create(grn_ctx *ctx, const char *path, uint32_t value_size, uint32_t flags)
{
- grn_array *array;
- if (ctx && (array = GRN_MALLOC(sizeof(grn_array)))) {
- GRN_DB_OBJ_SET_TYPE(array, GRN_TABLE_NO_KEY);
- if (!grn_array_init(ctx, array, path, value_size, flags)) {
- return array;
+ if (ctx) {
+ grn_array * const array = (grn_array *)GRN_MALLOC(sizeof(grn_array));
+ if (array) {
+ GRN_DB_OBJ_SET_TYPE(array, GRN_TABLE_NO_KEY);
+ if (!grn_array_init(ctx, array, path, value_size, flags)) {
+ return array;
+ }
+ GRN_FREE(array);
}
- GRN_FREE(array);
}
return NULL;
}
@@ -335,11 +337,11 @@ grn_array *
grn_array_open(grn_ctx *ctx, const char *path)
{
if (ctx) {
- grn_io *io = grn_io_open(ctx, path, grn_io_auto);
+ grn_io * const io = grn_io_open(ctx, path, grn_io_auto);
if (io) {
- struct grn_array_header *header = grn_io_header(io);
+ struct grn_array_header * const header = grn_io_header(io);
if (grn_io_get_type(io) == GRN_TABLE_NO_KEY) {
- grn_array *array = GRN_MALLOC(sizeof(grn_array));
+ grn_array * const array = (grn_array *)GRN_MALLOC(sizeof(grn_array));
if (array) {
if (!(header->flags & GRN_ARRAY_TINY)) {
GRN_DB_OBJ_SET_TYPE(array, GRN_TABLE_NO_KEY);
@@ -353,9 +355,9 @@ grn_array_open(grn_ctx *ctx, const char *path)
array->io = io;
array->header = header;
array->lock = &header->lock;
- return (grn_array *)array;
+ return array;
} else {
- GRN_LOG(ctx, GRN_LOG_NOTICE, "invalid array flag. (%x)", header->flags);
+ GRN_LOG(ctx, GRN_LOG_NOTICE, "invalid array flags. (%x)", header->flags);
}
GRN_FREE(array);
}