klimek added inline comments.

================
Comment at: cfe/trunk/docs/ClangFormatStyleOptions.rst:384
+
+  * ``SIS_WithoutElse`` (in configuration: ``WithoutElse``)
+    Allow short if functions on the same line, as long as else
----------------
MyDeveloperDay wrote:
> klimek wrote:
> > Is that actually the status quo or is the current behavior missing?
> Yes that is "as was"
Ok, that seems... wrong :)
Specifically, I'd at least expect the first one to also do
  if (a) return;
  else return;

That would be SIS_Always now? If so, could we make "true" just be "Always"?


Repository:
  rL LLVM

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

https://reviews.llvm.org/D59087



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

Reply via email to