mprobst added a comment.
PTAL.
================
Comment at: lib/Format/SortJavaScriptImports.cpp:162
@@ +161,3 @@
+ std::string ImportsText;
+ for (unsigned i = 0, e = Indices.size(); i != e; ++i) {
+ JsImportExport ImpExp = Imports[Indices[i]];
----------------
djasper wrote:
> Is there any chance you are changing the total length of an import block?
No, this only ever monotonously increases the length. I'm aware things fall
apart if it doesn't ;-)
================
Comment at: lib/Format/SortJavaScriptImports.cpp:226
@@ +225,3 @@
+ ImpExp.URL = Current->TokenText.substr(1, Current->TokenText.size() - 2);
+ if (ImpExp.URL.startswith("..")) {
+ ImpExp.Category = JsImportExport::JsImportCategory::RELATIVE_PARENT;
----------------
djasper wrote:
> No braces.
Just can't get used to that, it feels so wrong...
http://reviews.llvm.org/D20198
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
