...
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.
...
Run spider with new changes.
...