[cmake-developers] Nitpicking ProcessorCount.cmake
Alexander Neundorf
neundorf at kde.org
Sun Oct 23 09:30:35 EDT 2011
Hi,
I just used ProcessorCount.cmake the first time.
I noticed a small issue:
AFAIK module file names in cmake use CamelCase, while the macros/functions use
underscores: SomeCoolStuff.cmake -> some_cool_stuff()
ProcessorCount.cmake doesn't do this, the function is named processorcount()
IMO we should keep it this way, since it is defacto-standard, and it also
makes sense, since I always recommend to use all-lower-caps style for
commands, or at least all-upper-caps.
And when using all-lower, the function name is not that easy to read anymore.
So, should I rename it to processor_count() and add a processorcount() for
backward compatibility ?
Alex
P.S. it is similar with ExternalPackage.cmake, externalproject_add() doesn't
use the old style, it would have been external_project_add() instead.
More information about the cmake-developers
mailing list