[clang-format] make doc for SLS_Inline more clearly

[Backl1ght]

OMG! I just looked into git history and last edition of this line happened about 4 years ago. Am I doing it wrong?

[MyDeveloperDay]

I'm not sure if that is correct, as that doesn't fit the example (unless the example is wrong)

It feels like its saying merge if its an argument of a function (sort in this case)

sort(a.begin(), a.end(), []() { return x < y; });

The lambda is the argument, not that the lamda has no arguments? either way something isn't correct by making that change (either the description was wrong as well as the example or now the description doesn't match the example)

[Backl1ght]

Seems like I have misunderstood it, after some test I'm sure that it means merge if its an argument of a function.

I think there are something todo to make the doc more clearly.

[HazardyKnusperkeks]

In D139937#3992098, @Backl1ght wrote:

OMG! I just looked into git history and last edition of this line happened about 4 years ago. Am I doing it wrong?

Why? We have lambdas since C++11 so 11 years now.

[Backl1ght]

In D139937#3992987, @HazardyKnusperkeks wrote:

In D139937#3992098, @Backl1ght wrote:

OMG! I just looked into git history and last edition of this line happened about 4 years ago. Am I doing it wrong?

Why? We have lambdas since C++11 so 11 years now.

Based on the git history, last edition of this line happened on a83e2db.

[HazardyKnusperkeks]

In D139937#3993641, @Backl1ght wrote:

In D139937#3992987, @HazardyKnusperkeks wrote:

In D139937#3992098, @Backl1ght wrote:

OMG! I just looked into git history and last edition of this line happened about 4 years ago. Am I doing it wrong?

Why? We have lambdas since C++11 so 11 years now.

Based on the git history, last edition of this line happened on a83e2db.

Let me rephrase it. Why should you think that you do it wrong?

[Backl1ght]

In D139937#3994044, @HazardyKnusperkeks wrote:

In D139937#3993641, @Backl1ght wrote:

In D139937#3992987, @HazardyKnusperkeks wrote:

In D139937#3992098, @Backl1ght wrote:

OMG! I just looked into git history and last edition of this line happened about 4 years ago. Am I doing it wrong?

Why? We have lambdas since C++11 so 11 years now.

Based on the git history, last edition of this line happened on a83e2db.

Let me rephrase it. Why should you think that you do it wrong?

I don't think such typo can exist for 4 years so I think maybe I understand it wrongly. And it turned out that I do understand it in the wrong way.

[Backl1ght]

In D139937#3994044, @HazardyKnusperkeks wrote:

In D139937#3993641, @Backl1ght wrote:

In D139937#3992987, @HazardyKnusperkeks wrote:

In D139937#3992098, @Backl1ght wrote:

OMG! I just looked into git history and last edition of this line happened about 4 years ago. Am I doing it wrong?

Why? We have lambdas since C++11 so 11 years now.

Based on the git history, last edition of this line happened on a83e2db.

Let me rephrase it. Why should you think that you do it wrong?

For "Am I doing it wrong" I mean I am not sure if what I do in diff1 is correct.

[MyDeveloperDay]

I think given that it confused you I'm fine with this change

clang/docs/ClangFormatStyleOptions.rst

* ``SLS_Empty`` (in configuration: ``Empty``)
     .. code-block:: c++
 
       auto lambda = [](int a) {};
       auto lambda2 = [](int a) {
           return a;
       };
 
   * ``SLS_Inline`` (in configuration: ``Inline``)
-    Merge lambda into a single line if argument of a function.
+    Merge lambda into a single line if argument of a function is empty.
 
     .. code-block:: c++
 
       auto lambda = [](int a) {
           return a;
       };
       sort(a.begin(), a.end(), []() { return x < y; });
 

clang/include/clang/Format/Format.h

enum ShortLambdaStyle : int8_t {
     /// Only merge empty lambdas.
     /// \code
     ///   auto lambda = [](int a) {};
     ///   auto lambda2 = [](int a) {
     ///       return a;
     ///   };
     /// \endcode
     SLS_Empty,
-    /// Merge lambda into a single line if argument of a function.
+    /// Merge lambda into a single line if argument of a function is empty.
     /// \code
     ///   auto lambda = [](int a) {
     ///       return a;
     ///   };
     ///   sort(a.begin(), a.end(), []() { return x < y; });
     /// \endcode
     SLS_Inline,
     /// Merge all lambdas fitting on a single line.