search

terryyin / lizard

A simple code complexity analyser without caring about the C/C++ header files or Java imports, supports most of the popular languages.
Other
1.85k stars 250 forks source link

Add a test for inline C++ operator overload #71

Closed rakhimov closed 9 years ago

rakhimov commented 9 years ago

The name of the class is not attached to the operator overload. This is an example for #70

Review on Reviewable

terryyin commented 9 years ago

Hi @rakhimov , I've passed your test and added a few more. I did it for namespaces too.

What do you think about adding namespaces to the function names? useful or too noise?

rakhimov commented 9 years ago

I think it is a good idea for C++ code. For large codebase with multiple namespaces, it is helpful for reports to get the namespace attached to function names, i.e., namespace::class::function. This can be optional for small codebase. I would suggest Lizard have a flag for this optional feature.