We can add a simple test like this: 1. Create a context target in Makefile to generate a .cxx file which list and include all header files in nuttx/include 2. Add the generated file to CXXSRCS
If we just want to ensure the public nuttx header files are compatible with c++ syntax. On Mon, Oct 9, 2023 at 9:47 AM Daniel Appiagyei <daniel.appiag...@braincorp.com.invalid> wrote: > Late follow up, sorry > > Thanks guys, I do understand that only headers need to be compatible, > however, the GCC check I mentioned above does not discriminate between > headers and source files. If we don't have some sort of automated check for > C++ compatibility, then we will run into issues in the future like I > mentioned above. The nuttx style guide mentions that all C code must be C89 > compliant. As long as C++ compatibility doesn't interfere with that > standard then we won't be losing much. > Are there any other ideas on how to automatically enforce headers being c++ > comaptible? > Best, > Daniel > > On Sun, Sep 17, 2023 at 12:56 PM Petro Karashchenko < > petro.karashche...@gmail.com> wrote: > > > Since C++ most likely will never be used in the kernel I see that > > compatibility border should stay in the "include" folder in nuttx repo. > The > > nuttx-apps headers should be C++ compatible as we can mix things at > > application level. > > > > > -- > > > *Daniel Appiagyei | Embedded Software Engineer *Email: > daniel.appiag...@braincorp.com > <bog...@braincorporation.com>*Brain* > * Corp™ *10182 Telesis Ct, Suite 100 > San Diego, CA 92121 > > (858)-689-7600 > www.braincorp.com >
