Fix test failures when trap errors are enabled and other fixes

Author: SenRamakri

TESTS/mbed_platform/error_handling/main.cpp

@@ -316,7 +316,7 @@ void test_error_logging()
 
 }
 
-#define NUM_TEST_THREADS 10
+#define NUM_TEST_THREADS 5
 
 //Error logger threads
 void err_thread_func(mbed_error_status_t *error_status)
@@ -471,7 +471,7 @@ void test_save_error_log()
 
 utest::v1::status_t test_setup(const size_t number_of_cases)
 {
-    GREENTEA_SETUP(100, "default_auto");
+    GREENTEA_SETUP(300, "default_auto");
     return utest::v1::verbose_test_setup_handler(number_of_cases);
 }
 

TESTS/mbedmicro-rtos-mbed/event_flags/main.cpp

@@ -51,6 +51,12 @@ Semaphore sync_sem(0, 1);
 void error(const char* format, ...) {
     (void) format;
 }
+
+//Override the set_error function to trap the errors 
+mbed_error_status_t set_error(mbed_error_status_t error_status, const char *error_msg, unsigned int error_value, const char *filename, int line_number) 
+{
+    return ERROR_SUCCESS;
+}
 #endif
 
 template<uint32_t flags, uint32_t wait_ms>

TESTS/mbedmicro-rtos-mbed/rtostimer/main.cpp

@@ -85,6 +85,11 @@ void error(const char* format, ...)
 {
     (void) format;
 }
+
+mbed_error_status_t set_error(mbed_error_status_t error_status, const char *error_msg, unsigned int error_value, const char *filename, int line_number) 
+{
+    return ERROR_SUCCESS;
+}
 #endif
 
 /** Test one-shot not restarted when elapsed

TESTS/mbedmicro-rtos-mbed/signals/main.cpp

@@ -55,6 +55,11 @@ struct Sync {
 void error(const char* format, ...) {
     (void) format;
 }
+
+mbed_error_status_t set_error(mbed_error_status_t error_status, const char *error_msg, unsigned int error_value, const char *filename, int line_number) 
+{
+    return ERROR_SUCCESS;
+}
 #endif
 
 

platform/mbed_error.c

@@ -163,7 +163,7 @@ mbed_error_status_t set_warning(mbed_error_status_t error_status, const char *er
 }
 
 //Sets a fatal error 
-mbed_error_status_t set_error(mbed_error_status_t error_status, const char *error_msg, unsigned int error_value, const char *filename, int line_number) 
+WEAK mbed_error_status_t set_error(mbed_error_status_t error_status, const char *error_msg, unsigned int error_value, const char *filename, int line_number) 
 {
     //set the error reported and then halt the system
     if( ERROR_SUCCESS != handle_error(error_status, error_msg, error_value, filename, line_number) )

platform/mbed_error_report.c

@@ -119,7 +119,7 @@ void mbed_error_print(char *fmtstr, uint32_t *values)
     while(fmtstr[i] != '\0') {
         if(fmtstr[i]=='%') {
             i++;
-            if(fmtstr[i]=='x' || fmtstr[i]=='d') {
+            if(fmtstr[i]=='x') {
                 //print the number in hex format
                 value_to_hex_str(values[vidx++],num_str);
                 for(idx=7; idx>=0; idx--) {