On 11/08/2020 14:30, Joel Sherrill wrote:
On Tue, Aug 11, 2020 at 3:41 AM Aschref Ben-Thabet
<[email protected]
<mailto:[email protected]>> wrote:
From: Aschref Ben Thabet <[email protected]
<mailto:[email protected]>>
---
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.
I think the goal should be that RTEMS API header files work well with
C++11, C++17, C++20, and so on.
_______________________________________________
devel mailing list
[email protected]
http://lists.rtems.org/mailman/listinfo/devel
