jankratochvil added inline comments.

================
Comment at: source/Plugins/SymbolFile/DWARF/DWARFDebugInfo.cpp:248-249
 
-    while (cu->Extract(dwarf2Data->get_debug_info_data(), &offset)) {
+    for (;;) {
+      DWARFCompileUnitSP cu = DWARFCompileUnit::Extract(dwarf2Data, &offset);
+      if (cu.get() == NULL)
----------------
clayborg wrote:
> Might be cleaner to do:
> ```
> DWARFCompileUnitSP cu;
> while ((cu = DWARFCompileUnit::Extract(dwarf2Data, &offset))) {
> ```
> 
I did not know LLVM style can be this brief, that is sure better, implemented.



https://reviews.llvm.org/D40212



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

Reply via email to