[PATCH] D112664: [clang-format][docs] fix indentation of rst code block

2021-11-02 Thread Björn Schäpers via Phabricator via cfe-commits
HazardyKnusperkeks added a comment.

In D112664#3101142 , @MyDeveloperDay 
wrote:

> This should be Format.h, didn't we recently fix this?

Yes we did, this is from before. :) So it is now obsolete.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D112664

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


[PATCH] D112664: [clang-format][docs] fix indentation of rst code block

2021-11-01 Thread MyDeveloperDay via Phabricator via cfe-commits
MyDeveloperDay requested changes to this revision.
MyDeveloperDay added a comment.
This revision now requires changes to proceed.

This should be Format.h, didn't we recently fix this?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D112664

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


[PATCH] D112664: [clang-format][docs] fix indentation of rst code block

2021-10-29 Thread Björn Schäpers via Phabricator via cfe-commits
HazardyKnusperkeks added a comment.

Hi, thanks for you contribution.

But the `rst` is generated by a script, the error is in `Format.h`.

And we already have a review for an even broader fix: D112572 
.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D112664

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


[PATCH] D112664: [clang-format][docs] fix indentation of rst code block

2021-10-27 Thread Mike via Phabricator via cfe-commits
Wirena created this revision.
Wirena added a reviewer: HazardyKnusperkeks.
Wirena requested review of this revision.
Herald added a project: clang.
Herald added a subscriber: cfe-commits.

Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D112664

Files:
  clang/docs/ClangFormatStyleOptions.rst


Index: clang/docs/ClangFormatStyleOptions.rst
===
--- clang/docs/ClangFormatStyleOptions.rst
+++ clang/docs/ClangFormatStyleOptions.rst
@@ -3821,16 +3821,16 @@
 
   .. code-block:: c++
 
-  before:   after:
-  Minimum: 1
-  //if (b) {// if (b) {
-  //  return true;  //   return true;
-  //}   // }
-
-  Maximum: 0
-  /// List: ///List:
-  ///  - Foo/// - Foo
-  ///- Bar  ///   - Bar
+ before:   after:
+ Minimum: 1
+ //if (b) {// if (b) {
+ //  return true;  //   return true;
+ //}   // }
+
+ Maximum: 0
+ /// List: ///List:
+ ///  - Foo/// - Foo
+ ///- Bar  ///   - Bar
 
   Nested configuration flags:
 


Index: clang/docs/ClangFormatStyleOptions.rst
===
--- clang/docs/ClangFormatStyleOptions.rst
+++ clang/docs/ClangFormatStyleOptions.rst
@@ -3821,16 +3821,16 @@
 
   .. code-block:: c++
 
-  before:   after:
-  Minimum: 1
-  //if (b) {// if (b) {
-  //  return true;  //   return true;
-  //}   // }
-
-  Maximum: 0
-  /// List: ///List:
-  ///  - Foo/// - Foo
-  ///- Bar  ///   - Bar
+ before:   after:
+ Minimum: 1
+ //if (b) {// if (b) {
+ //  return true;  //   return true;
+ //}   // }
+
+ Maximum: 0
+ /// List: ///List:
+ ///  - Foo/// - Foo
+ ///- Bar  ///   - Bar
 
   Nested configuration flags:
 
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits