[Lldb-commits] [PATCH] D24952: Remove Args::m_argv

[Pavel Labath via lldb-commits]

labath added a subscriber: labath.

================
Comment at: include/lldb/Interpreter/Args.h:150
@@ -147,19 +149,3 @@
   ///
   /// @return
   ///     An array of NULL terminate C string argument pointers that
----------------
I think doxygen will complain about `@return` in a function returning void.

BTW, with the NRVO, move semantics and everything, is there any reason to keep using the return-through-reference-argument pattern? It would be a lot more natural if this indeed _was_ a return value.


https://reviews.llvm.org/D24952