[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