aboutsummaryrefslogtreecommitdiffhomepage
path: root/src
diff options
context:
space:
mode:
authorGravatar ncteisen <ncteisen@gmail.com>2017-03-09 17:04:24 -0800
committerGravatar ncteisen <ncteisen@gmail.com>2017-03-09 19:02:31 -0800
commitceddd293919a9580cef928fa56a35480de278682 (patch)
treec823b5e23f35d1004633335a0d969e7bfd655591 /src
parent234d1748d52314555a1bd4cc500cbefa73d3c789 (diff)
Address github comments
Diffstat (limited to 'src')
-rw-r--r--src/core/lib/iomgr/error.c17
-rw-r--r--src/core/lib/iomgr/error_internal.h4
-rw-r--r--src/core/lib/transport/error_utils.c4
3 files changed, 13 insertions, 12 deletions
diff --git a/src/core/lib/iomgr/error.c b/src/core/lib/iomgr/error.c
index f0b7e01d49..c6851b0b6c 100644
--- a/src/core/lib/iomgr/error.c
+++ b/src/core/lib/iomgr/error.c
@@ -155,7 +155,7 @@ grpc_error *grpc_error_ref(grpc_error *err) {
static void unref_errs(grpc_error *err) {
uint8_t slot = err->first_err;
while (slot != UINT8_MAX) {
- linked_error *lerr = (linked_error *)(err->arena + slot);
+ grpc_linked_error *lerr = (grpc_linked_error *)(err->arena + slot);
GRPC_ERROR_UNREF(lerr->err);
GPR_ASSERT(err->last_err == slot ? lerr->next == UINT8_MAX
: lerr->next != UINT8_MAX);
@@ -254,25 +254,26 @@ static void internal_set_time(grpc_error **err, grpc_error_times which,
}
static void internal_add_error(grpc_error **err, grpc_error *new) {
- linked_error new_last = {new, UINT8_MAX};
- uint8_t slot = get_placement(err, sizeof(linked_error));
+ grpc_linked_error new_last = {new, UINT8_MAX};
+ uint8_t slot = get_placement(err, sizeof(grpc_linked_error));
if ((*err)->first_err == UINT8_MAX) {
GPR_ASSERT((*err)->last_err == UINT8_MAX);
(*err)->last_err = slot;
(*err)->first_err = slot;
} else {
GPR_ASSERT((*err)->last_err != UINT8_MAX);
- linked_error *old_last = (linked_error *)((*err)->arena + (*err)->last_err);
+ grpc_linked_error *old_last =
+ (grpc_linked_error *)((*err)->arena + (*err)->last_err);
old_last->next = slot;
(*err)->last_err = slot;
}
- memcpy((*err)->arena + slot, &new_last, sizeof(linked_error));
+ memcpy((*err)->arena + slot, &new_last, sizeof(grpc_linked_error));
}
#define SLOTS_PER_INT (sizeof(intptr_t) / sizeof(intptr_t))
#define SLOTS_PER_STR (sizeof(grpc_slice) / sizeof(intptr_t))
#define SLOTS_PER_TIME (sizeof(gpr_timespec) / sizeof(intptr_t))
-#define SLOTS_PER_LINKED_ERROR (sizeof(linked_error) / sizeof(intptr_t))
+#define SLOTS_PER_LINKED_ERROR (sizeof(grpc_linked_error) / sizeof(intptr_t))
// size of storing one int and two slices and a timespec. For line, desc, file,
// and time created
@@ -345,7 +346,7 @@ static void ref_strs(grpc_error *err) {
static void ref_errs(grpc_error *err) {
uint8_t slot = err->first_err;
while (slot != UINT8_MAX) {
- linked_error *lerr = (linked_error *)(err->arena + slot);
+ grpc_linked_error *lerr = (grpc_linked_error *)(err->arena + slot);
GRPC_ERROR_REF(lerr->err);
slot = lerr->next;
}
@@ -636,7 +637,7 @@ static void add_errs(grpc_error *err, char **s, size_t *sz, size_t *cap) {
uint8_t slot = err->first_err;
bool first = true;
while (slot != UINT8_MAX) {
- linked_error *lerr = (linked_error *)(err->arena + slot);
+ grpc_linked_error *lerr = (grpc_linked_error *)(err->arena + slot);
if (!first) append_chr(',', s, sz, cap);
first = false;
const char *e = grpc_error_string(lerr->err);
diff --git a/src/core/lib/iomgr/error_internal.h b/src/core/lib/iomgr/error_internal.h
index bdebbe1792..fb4814e41f 100644
--- a/src/core/lib/iomgr/error_internal.h
+++ b/src/core/lib/iomgr/error_internal.h
@@ -39,9 +39,9 @@
#include <grpc/support/sync.h>
-typedef struct linked_error linked_error;
+typedef struct grpc_linked_error grpc_linked_error;
-struct linked_error {
+struct grpc_linked_error {
grpc_error *err;
uint8_t next;
};
diff --git a/src/core/lib/transport/error_utils.c b/src/core/lib/transport/error_utils.c
index 707aac024b..ef55e561fb 100644
--- a/src/core/lib/transport/error_utils.c
+++ b/src/core/lib/transport/error_utils.c
@@ -46,7 +46,7 @@ static grpc_error *recursively_find_error_with_field(grpc_error *error,
// Otherwise, search through its children.
uint8_t slot = error->first_err;
while (slot != UINT8_MAX) {
- linked_error *lerr = (linked_error *)(error->arena + slot);
+ grpc_linked_error *lerr = (grpc_linked_error *)(error->arena + slot);
grpc_error *result = recursively_find_error_with_field(lerr->err, which);
if (result) return result;
slot = lerr->next;
@@ -114,7 +114,7 @@ bool grpc_error_has_clear_grpc_status(grpc_error *error) {
}
uint8_t slot = error->first_err;
while (slot != UINT8_MAX) {
- linked_error *lerr = (linked_error *)(error->arena + slot);
+ grpc_linked_error *lerr = (grpc_linked_error *)(error->arena + slot);
if (grpc_error_has_clear_grpc_status(lerr->err)) {
return true;
}