amccarth added inline comments.

================
Comment at: 
source/Plugins/LanguageRuntime/RenderScript/RenderScriptRuntime/RenderScriptRuntime.cpp:185
@@ -184,3 +184,3 @@
 
-    static const ConstString FallbackStructName;         // Print this as the 
type name of a struct Element
                                                          // If we can't 
resolve the actual struct name
----------------
zturner wrote:
> I wonder about returning this by reference.  It seems like by convention we 
> usually pass `ConstString`s around by value.
I was following the pattern used elsewhere, like 
Breakpoint::BreakpointEventData::GetFlavorString.


http://reviews.llvm.org/D15092



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to