aprantl added inline comments.

================
Comment at: source/Host/common/MainLoop.cpp:158
   // ppoll(2) is not supported on older all android versions. Also, older
-  // versions android (API <= 19) implemented pselect in a non-atomic way, as a
-  // combination of pthread_sigmask and select. This is not sufficient for us,
----------------
JDevlieghere wrote:
> Is this intentional or maybe an off-by-one error? The `a` is in the 79th 
> colum, which is okay for clang-format. 
My script sets the threshold to 78 columns. When you are on an 80-column 
terminal and your editor doesn't indicate whether long lines continue or not 
then not using the last two characters makes it unambiguous whether the `a` is 
the beginning of a new word or whether it stands for itself.
I'll change the threshold to 80 columns to avoid reflowing paragraphs that 
don't need it.


https://reviews.llvm.org/D46144



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

Reply via email to