Good solution. I'm not too knowledgeable about makefiles, but if you could set that part up then I can do the work to make the files compatible?
On Sun, Oct 8, 2023 at 10:44 PM Xiang Xiao <xiaoxiang781...@gmail.com> wrote: > 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 > > > -- *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
