tachyon icon indicating copy to clipboard operation
tachyon copied to clipboard

Discuss and unify the use of `const` keyword for local variables

Open fakedev9999 opened this issue 1 year ago • 0 comments

As written in Google C++ Style Guide: Use of const, indicating the variables that are not changed with const keyword makes easier for people to understand how the variable is used, along with the complier to do better type checking, assuring better code.

So, as in suggestion of this comment, seems reasonable.

However, through out our repository, the use of const keyword for local variables is not organized well. Plus, in the decision section of the Google C++ Style Guide: Use of const, it says

Using const on local variables is neither encouraged nor discouraged.

It's quite trivial, but I thought it would be great to at least have a discussion on this point and have a policy with regards to this throughout our repository.

fakedev9999 avatar Jan 05 '24 02:01 fakedev9999