[PATCH v2] rtems/printer.h Fix build warnings -Wclass-memaccess

[Joel Sherrill]

On Tue, Aug 11, 2020 at 3:41 AM Aschref Ben-Thabet <
aschref.ben-thabet at embedded-brains.de> wrote:

> From: Aschref Ben Thabet <aschref.ben-thabet at embedded-brains.de>
>
> ---
>  cpukit/include/rtems/printer.h | 6 +++++-
>  1 file changed, 5 insertions(+), 1 deletion(-)
>
> diff --git a/cpukit/include/rtems/printer.h
> b/cpukit/include/rtems/printer.h
> index dbd887221e..310937cc2f 100644
> --- a/cpukit/include/rtems/printer.h
> +++ b/cpukit/include/rtems/printer.h
> @@ -130,7 +130,11 @@ static inline void rtems_printer_task_initialize(
>    rtems_printer_task_context *context
>  )
>  {
> -  memset( context, 0, sizeof( *context ) );
> +  /*
> +   * Some C++ compiler think that the structure is complex enough to need
> a
> +   * proper constructor. Cast to void * to silence a warning.
> +   */
> +  memset( (void *) context, 0, sizeof( *context ) );
>

I understand the patch but does this mean all header files have to be C++XX
clean? Where XX is what version of C++.

I'm more concerned by the implications of the patch than the patch itself.

--joel


>  }
>
>  static inline void rtems_printer_task_set_stack_size(
> --
> 2.26.2
>
> _______________________________________________
> devel mailing list
> devel at rtems.org
> http://lists.rtems.org/mailman/listinfo/devel
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.rtems.org/pipermail/devel/attachments/20200811/af5f65c4/attachment.html>