Skip to content

C++ Core Guidelines #1218

Description

@pcarruscag

I would like to bring your attention to the existence of this gem:
https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#main

Of which I use maybe a subset in my often verbose and perhaps repetitive code reviews.
The things I comment on are as much a matter of style as multiplication taking precedence over addition is.
It is how "They" do it.

So do please browse the guidelines.
Ingraining some good guidelines, goes a long way towards freeing one's mind to focus on important implementation aspects.
Does this thing need to be const? Just make it, the compiler will tell you if it can't or if you were about to have a bug, the opposite does not happen.
I need this piece of code from somewhere. -> Put it in a function.
My function does not fit the screen. -> Too long.
new su2double[nSomething] -> Some container type, which one? The simplest that will do the job.
etc.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions