[rtems commit] shell: Make rtems_global_shell_env public

Chris Johns chrisj at rtems.org
Thu May 29 00:17:38 UTC 2014


Wooo this was a little fast. I do sleep. :)

I would like my question answered on the user answered.

I never liked the global in the first place and still do not.

Chris

On 29/05/2014 2:05 am, Sebastian Huber wrote:
> Module:    rtems
> Branch:    master
> Commit:    5f0ab5cfb5aa04cd08b06a43e8258253703e95ea
> Changeset: http://git.rtems.org/rtems/commit/?id=5f0ab5cfb5aa04cd08b06a43e8258253703e95ea
>
> Author:    Sebastian Huber <sebastian.huber at embedded-brains.de>
> Date:      Wed May 28 17:17:29 2014 +0200
>
> shell: Make rtems_global_shell_env public
>
> Provide rtems_global_shell_env as a public read-only variable for
> backward compatibility.
>
> ---
>
>   cpukit/libmisc/shell/shell.c |    2 +-
>   cpukit/libmisc/shell/shell.h |    2 ++
>   2 files changed, 3 insertions(+), 1 deletions(-)
>
> diff --git a/cpukit/libmisc/shell/shell.c b/cpukit/libmisc/shell/shell.c
> index 8666299..0f31f2a 100644
> --- a/cpukit/libmisc/shell/shell.c
> +++ b/cpukit/libmisc/shell/shell.c
> @@ -40,7 +40,7 @@
>   #include <pthread.h>
>   #include <assert.h>
>
> -static rtems_shell_env_t rtems_global_shell_env = {
> +const rtems_shell_env_t rtems_global_shell_env = {
>     .magic         = rtems_build_name('S', 'E', 'N', 'V'),
>     .devname       = CONSOLE_DEVICE_NAME,
>     .taskname      = "SHGL",
> diff --git a/cpukit/libmisc/shell/shell.h b/cpukit/libmisc/shell/shell.h
> index af09ac3..76b34c7 100644
> --- a/cpukit/libmisc/shell/shell.h
> +++ b/cpukit/libmisc/shell/shell.h
> @@ -205,6 +205,8 @@ bool rtems_shell_main_loop(
>     rtems_shell_env_t *rtems_shell_env
>   );
>
> +extern const rtems_shell_env_t rtems_global_shell_env;
> +
>   rtems_shell_env_t *rtems_shell_get_current_env(void);
>   #define rtems_current_shell_env rtems_shell_get_current_env()
>
>
> _______________________________________________
> rtems-vc mailing list
> rtems-vc at rtems.org
> http://www.rtems.org/mailman/listinfo/rtems-vc
>



More information about the devel mailing list