barannikov88 added inline comments.

================
Comment at: clang/lib/Format/UnwrappedLineFormatter.cpp:169-170
 
   /// The indent in characters for each level.
+  /// It remembers the indent of previous lines (that are not PP directives) of
+  /// equal or lower levels. This is used to align formatted lines to the 
indent
----------------
Sedeniono wrote:
> Sedeniono wrote:
> > HazardyKnusperkeks wrote:
> > > And then reformat the comment.
> > Done.
> Mh, it says [that it couldn't apply the 
> patch](https://reviews.llvm.org/harbormaster/build/367648/). I uploaded a 
> patch containing as only change the reformatted comment, via the Web UI 
> ("Update Diff" at the top right corner). Should I have uploaded a "squashed" 
> diff, containing also ALL changes that I made before (the `resize` etc)? In 
> other words, is Phabricator expecting individual patches and it merges and 
> accumulates them itself? Or does each new uploaded patch overwrite all 
> previous patches?
The latter


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D151047/new/

https://reviews.llvm.org/D151047

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

Reply via email to