Skip to content

Commit

Permalink
Remove error reporting for release builds
Browse files Browse the repository at this point in the history
  • Loading branch information
SenRamakri committed Jul 6, 2018
1 parent a29eee4 commit b6179d2
Showing 1 changed file with 9 additions and 6 deletions.
15 changes: 9 additions & 6 deletions platform/mbed_error.c
Original file line number Diff line number Diff line change
Expand Up @@ -45,6 +45,11 @@
} \
}

#ifndef NDEBUG
#define ERROR_REPORT(ctx, error_msg) print_error_report(ctx, error_msg)
#else
#define ERROR_REPORT(ctx, error_msg) ((void) 0)
#endif

static uint8_t error_in_progress = 0;
static int error_count = 0;
Expand Down Expand Up @@ -78,9 +83,7 @@ WEAK void error(const char *format, ...)

//Call handle_error/print_error_report permanently setting error_in_progress flag
handle_error(MBED_ERROR_UNKNOWN, 0, NULL, 0);
#ifndef NDEBUG
print_error_report(&last_error_ctx, "Fatal Run-time error");
#endif
ERROR_REPORT(&last_error_ctx, "Fatal Run-time error");
error_in_progress = 1;

#ifndef NDEBUG
Expand Down Expand Up @@ -201,10 +204,8 @@ WEAK mbed_error_status_t mbed_error(mbed_error_status_t error_status, const char
return MBED_ERROR_FAILED_OPERATION;
}

#ifndef NDEBUG
//On fatal errors print the error context/report
print_error_report(&last_error_ctx, error_msg);
#endif
ERROR_REPORT(&last_error_ctx, error_msg);
mbed_halt_system();

return MBED_ERROR_FAILED_OPERATION;
Expand Down Expand Up @@ -306,6 +307,7 @@ static void print_threads_info(osRtxThread_t *threads)
}
#endif

#ifndef NDEBUG
static void print_error_report(mbed_error_ctx *ctx, const char *error_msg)
{
uint32_t error_code = MBED_GET_ERROR_CODE(ctx->error_status);
Expand Down Expand Up @@ -390,6 +392,7 @@ static void print_error_report(mbed_error_ctx *ctx, const char *error_msg)

mbed_error_printf("\n-- MbedOS Error Info --\n");
}
#endif //ifndef NDEBUG

#if MBED_CONF_PLATFORM_ERROR_HIST_ENABLED
//Retrieve the error context from error log at the specified index
Expand Down

0 comments on commit b6179d2

Please sign in to comment.