[GitHub] [incubator-nuttx] v01d commented on a change in pull request #2072: libxx: Update README

[GitBox <gi...@apache.org>]

v01d commented on a change in pull request #2072:
URL: https://github.com/apache/incubator-nuttx/pull/2072#discussion_r510852017



##########
File path: libs/libxx/README.txt
##########
@@ -1,28 +1,44 @@
 libs/libxx/README.txt
 ^^^^^^^^^^^^^^^^^^^^^
 
-This directory contains a fragmentary C++ library that will allow to build
-only the simplest of C++ applications.  In the deeply embedded world, that
-is probably all that is necessary.  If you have a need for more extensive
-C++ support, the following libraries are recommended:
+This directory contains three C++ library:
+
+ - A fragmentary C++ library that will allow to build only the simplest of
+   C++ applications. In the deeply embedded world, that is probably all
+   that is necessary.
+
+   At present, only the following are supported here:
+
+     - void *operator new(std::size_t nbytes);
+     - void operator delete(void* ptr);
+     - void operator delete[](void *ptr);
+     - void __cxa_pure_virtual(void);
+     - int __aeabi_atexit(void* object, void (*destroyer)(void*),
+                          void *dso_handle);
+     - int __cxa_atexit(__cxa_exitfunc_t func, FAR void *arg,
+                        FAR void *dso_handle);
+
+   This implementation is selected when neither of the following
+   two options are enabled.
+
+ - LLVM "libc++" C++ library (http://libcxx.llvm.org/)
+   This implementation is selected with CONFIG_LIBCXX=y.
+
+ - uClibc++ C++ library (http://cxx.uclibc.org/)
+   This implementation is selected with CONFIG_UCLIBCXX=y.
+
+If you have a need for more extensive C++ support, the following
+libraries might be usable too:
 
  - libstdc++  (part of GCC)
  - STLport    http://www.stlport.org/
- - uClibc++   http://cxx.uclibc.org/
  - uSTL       http://ustl.sourceforge.net/

Review comment:
       I would remove all of those, as we're not really officially supporting them. The three options right now I think that are really sufficient.




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org