labath added inline comments.

================
Comment at: source/Utility/FileSpec.cpp:244-246
+  // Only update style if explicitly requested.
+  if (style)
+    m_style = (*style == Style::native) ? GetNativeStyle() : *style;
----------------
I don't really have a clear opinion on whether the new behaviour here is better 
or not, but it any case this sounds like something worth explicitly calling out.


================
Comment at: unittests/Utility/FileSpecTest.cpp:284
     "//",
-    "//a",
     "//a/",
----------------
Any particular reason for removing this?


Repository:
  rL LLVM

https://reviews.llvm.org/D48084



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

Reply via email to