[rtems commit] shell: Avoid rtems_error()

Sebastian Huber sebh at rtems.org
Tue May 14 09:12:33 UTC 2019


Module:    rtems
Branch:    master
Commit:    cc060f0b3552a1f95b630fa68de244f0067bc478
Changeset: http://git.rtems.org/rtems/commit/?id=cc060f0b3552a1f95b630fa68de244f0067bc478

Author:    Sebastian Huber <sebastian.huber at embedded-brains.de>
Date:      Fri May 10 14:38:28 2019 +0200

shell: Avoid rtems_error()

Do not use the rtems_error() function since this function pulls in
exit() and abort().  The abort() function pulls in raise() which pulls
in the whole POSIX signals support.  This change saves about 16KiB of
text/rodata on ARM Thumb-2 systems.

---

 cpukit/libmisc/shell/shell.c | 9 ---------
 1 file changed, 9 deletions(-)

diff --git a/cpukit/libmisc/shell/shell.c b/cpukit/libmisc/shell/shell.c
index 812a0b0..c2ea3c4 100644
--- a/cpukit/libmisc/shell/shell.c
+++ b/cpukit/libmisc/shell/shell.c
@@ -20,7 +20,6 @@
 #include <time.h>
 
 #include <rtems.h>
-#include <rtems/error.h>
 #include <rtems/libio.h>
 #include <rtems/libio_.h>
 #include <rtems/shell.h>
@@ -689,7 +688,6 @@ static bool rtems_shell_init_user_env(void)
   /* Make sure we have a private user environment */
   sc = rtems_libio_set_private_env();
   if (sc != RTEMS_SUCCESSFUL) {
-    rtems_error(sc, "rtems_libio_set_private_env():");
     return false;
   }
 
@@ -730,18 +728,15 @@ bool rtems_shell_main_loop(
 
   shell_env = rtems_shell_init_env(shell_env_arg);
   if (shell_env == NULL) {
-    rtems_error(0, "rtems_shell_init_env");
     return false;
   }
 
   eno = pthread_setspecific(rtems_shell_current_env_key, shell_env);
   if (eno != 0) {
-    rtems_error(0, "pthread_setspecific(shell_current_env_key)");
     return false;
   }
 
   if (!rtems_shell_init_user_env()) {
-    rtems_error(0, "rtems_shell_init_user_env");
     return false;
   }
 
@@ -997,14 +992,11 @@ static rtems_status_code rtems_shell_run (
     &task_id
   );
   if (sc != RTEMS_SUCCESSFUL) {
-    rtems_error(sc,"creating task %s in shell_init()",task_name);
     return sc;
   }
 
   shell_env = rtems_shell_init_env( NULL );
   if ( !shell_env )  {
-   rtems_error(RTEMS_NO_MEMORY,
-               "allocating shell_env %s in shell_init()",task_name);
    return RTEMS_NO_MEMORY;
   }
   shell_env->devname       = devname;
@@ -1025,7 +1017,6 @@ static rtems_status_code rtems_shell_run (
   sc = rtems_task_start(task_id, rtems_shell_task,
                           (rtems_task_argument) shell_env);
   if (sc != RTEMS_SUCCESSFUL) {
-    rtems_error(sc,"starting task %s in shell_init()",task_name);
     return sc;
   }
 



More information about the vc mailing list