make module and file naming consistent
Our src/gromacs modules are named inconsistently when the name has more than two words. The "complexname" pattern can be hard to read, but is somewhat consistent with class naming. The "complex_name" pattern is familiar and is preferred at https://google.github.io/styleguide/cppguide.html#File_Names. The "complex-name" pattern works and saves a shift key press. The "ComplexName" pattern works but might be tricky if we would ever again encounter a non-case-sensitive file system (albeit that we already use case in naming short-range kernels and reference data files).
Names of files within modules should also follow a particular pattern. We should avoid short file names likely to be present in multiple modules (logging.h, util.cpp, etc.). Once a module has a conforming design, then the file names should anyway relate to the class being implemented in them. In some cases that will mean the module name is a prefix for the class name, and thus file name. So we should expect to find the NbnxmGridSearch class implemented in src/gromacs/nbnxm/nbnxm_grid_search.cpp.
Rename all source files from - to _ for consistency.
#1 Updated by Mark Abraham almost 2 years ago
At https://gerrit.gromacs.org/#/c/9021/, Mark preferred consistency, then readability, and doesn't care what we choose. Also there, Roland preferred that we use only one of dash and underscore and observed that there are names including acronyms that are particularly hard to read. At https://gerrit.gromacs.org/#/c/8940/, Szilard preferred underscores. Berk didn't express an opinion. Erik's kernels have had both underscore and case, so he'll cope.
That sounds like the "complex_name" pattern has broad support for module naming.
For naming files within modules, are we OK with things like src/gromacs/listed_forces/gpu_bonded_impl.cu?
I'll put together an updated entry in the style guide if people give me a bit of feedback.
#4 Updated by Eric Irrgang over 1 year ago
Is there still an open request for feedback here?
Case is bad.
Underscores are clear (except when they typeset funny)
src/gromacs/listed_forces/gpu_bonded_impl.cu? look right.
I don't see the need to prefix module name to header file name. The path is part of the file name except in a handful of global headers and where the header is part of the current module. Does it mess up the source checker scripts or something?
#5 Updated by Szilárd Páll over 1 year ago
Eric Irrgang wrote:
The path is part of the file name except in a handful of global headers and where the header is part of the current module. Does it mess up the source checker scripts or something?
I tend to think it's best to preemptively avoid short generic names as much as possible, not because (as a developer) it's hard to look up the full path but because it should ideally be rarely if ever necessary to do so e.g. to know which "utils.h" are we talking about. I can imagine tooling issues too (like
filename:LINE reporting w/o full path).
The Google C++ Style Guide recommends the same.