Code review comment for lp:~epics-core/epics-base/3.15_libcom_from_cvs_trunk

Revision history for this message
Andrew Johnson (anj) wrote :

After committing a few changes to this branch the only parts I don't currently like are modifications to epicsThread.h and epicsTime.h which include a C standard header in a C build or the equivalent C++ header in a C++ build. The epicsThread.h file also adds a using declaration for std::size_t in C++, which does not seem like a good idea to me. We have other include files that pull in C standard headers in C++ builds, and I would like to see what other people think before accepting these kinds of changes.

« Back to merge proposal