clang-tidy uses FunctionCase for class methods

Question

I have a .clang-tidy file which includes

CheckOptions:
  - { key: readability-identifier-naming.ClassMethodCase, value: CamelCase }
  - { key: readability-identifier-naming.FunctionCase, value: aNy_CasE }

and a test file with this content

test.hpp

class TestClass {
  int myMethod();
};

I would expect the method myMethod to trigger the ClassMethodCase warning, but its not

clang-tidy test.hpp

reports nothing

If I set the FunctionCase option to CamelCase, then it triggers warning.

Why is clang-tidy considering myMethod is a function? Is this a bug or am I misunderstanding something?

That seems to do the trick. What is the purpose of ClassMethodCase then? — jjcasmar
– jjcasmar, Commented Apr 10 at 9:37
I don't know, nor do I know what logic is used for fallback if you don't specify. — Jeff Garrett
– Jeff Garrett, Commented Apr 10 at 13:45
Thanks. If you convert the comment to an answer, Ill mark it as resolved — jjcasmar
– jjcasmar, Commented Apr 10 at 20:50

Jeff Garrett · Accepted Answer · 2025-04-11 01:13:24Z

0

Use readability-identifier-naming.PrivateMethodCase

answered Apr 11 at 1:13

Jeff Garrett

8,4791 gold badge22 silver badges19 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Scott McPeak · Accepted Answer · 2025-04-11 02:02:49Z

0

ClassMethodCase pertains to static methods, i.e., methods that operate on the class as opposed to an object. I find this naming choice confusing, and the documentation does not clarify it; I discovered the meaning by studying the unit test.

MethodCase pertains to all methods, both static and non-static, so that's what you want in this case.

answered Apr 11 at 2:02

Scott McPeak

13.8k3 gold badges54 silver badges93 bronze badges

Collectives™ on Stack Overflow

clang-tidy uses FunctionCase for class methods

2 Answers 2

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related