...
review -> review + review-one -> review + review-two -> No tags
significant changes -> review or incomplete
There is some inconsistency with the use of const. We used to have a recommendation that declarations appear like this:
| Code Block |
|---|
int const *ip;
|
But we eventually got rid of it. Most of our declarations are of this form, but some are of this form:
| Code Block |
|---|
const int *ip;
|
The argument we had was that the first form is less prone to error, but the second form is widely used, and not something that people will change.
Our choices are a) apply the first style consistently, b) apply the second style consistently, c) allow a mix of both styles.
Whatever style we adopt should also apply to other qualified type declarations, like volatile.
The bot changes all of these to have rightmost const. There are very few (if any, it would only be what was added recently) the other way. -jp
...
C++ has it's own TODO List
...