https://github.com/PiotrZSL approved this pull request.
LGTM.
Unless you want it merged as "57949090+da-vi...@users.noreply.github.com",
please update your github config.
https://github.com/llvm/llvm-project/pull/69377
___
cfe-commits mailing list
https://github.com/Da-Viper updated
https://github.com/llvm/llvm-project/pull/69377
>From 0e0a3e7ad1a0a7098e05a5164413369eaa58c55b Mon Sep 17 00:00:00 2001
From: Ezike Ebuka
Date: Tue, 17 Oct 2023 20:49:47 +0100
Subject: [PATCH] Fix #41439: Update the documentation with the correct
https://github.com/PiotrZSL edited
https://github.com/llvm/llvm-project/pull/69377
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -10,7 +10,12 @@ Guide:
https://google.github.io/styleguide/cppguide.html#Function_Declarations_and_Definitions
-All parameters should be named, with identical names in the declaration and
-implementation.
+A parameter name may be omitted only if the parameter is not used
https://github.com/PiotrZSL edited
https://github.com/llvm/llvm-project/pull/69377
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -10,7 +10,12 @@ Guide:
https://google.github.io/styleguide/cppguide.html#Function_Declarations_and_Definitions
-All parameters should be named, with identical names in the declaration and
-implementation.
+A parameter name may be omitted only if the parameter is not used
llvmbot wrote:
@llvm/pr-subscribers-clang-tidy
Author: None (Da-Viper)
Changes
Fixes #41439
The documentation is update to say it is allowed to have omitted parameter
names if the variable is not used
---
Full diff: https://github.com/llvm/llvm-project/pull/69377.diff
1 Files
https://github.com/Da-Viper created
https://github.com/llvm/llvm-project/pull/69377
Fixes #41439
The documentation is update to say it is allowed to have omitted parameter
names if the variable is not used
>From 0e0a3e7ad1a0a7098e05a5164413369eaa58c55b Mon Sep 17 00:00:00 2001
From: Ezike