[PATCH] D116638: [clang-format] Fix ignoring JavaScriptWrapImport when ColumnWidth: 0

2022-05-31 Thread Stanisław Małolepszy via Phabricator via cfe-commits
stasm added a comment. In D116638#3547366 , @andmis wrote: > In D116638#3545246 , @stasm wrote: > >> I'm still interested in seeing this fixed. Would it help if I rebased this >> change and addressed the outstand

[PATCH] D116638: [clang-format] Fix ignoring JavaScriptWrapImport when ColumnWidth: 0

2022-05-30 Thread Stanisław Małolepszy via Phabricator via cfe-commits
stasm added a comment. Herald added a project: All. I'm still interested in seeing this fixed. Would it help if I rebased this change and addressed the outstanding review comments? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D116638/new/ https://reviews.llvm.org/D116638 _

[PATCH] D116638: [clang-format] Fix ignoring JavaScriptWrapImport when ColumnWidth: 0

2022-01-08 Thread Stanisław Małolepszy via Phabricator via cfe-commits
stasm added a comment. The reporter of issue 52935 here. Thanks, @andmis, for your work. Thinking about the `ColumnLimit: 0` and `JavaScriptWrapImports: false` case, it seems that there are two issues in the current implementation that could