Code review comment for lp:~epics-core/epics-base/ioc-shutdown2

Revision history for this message
J. Lewis Muir (jlmuir) wrote :

On 7/11/14, 5:09 PM, Andrew Johnson wrote:
> The dbLockCleanupRecords() routine looks a little strange; it appears

Is "Cleanup" in the function name "dbLockCleanupRecords" a noun? The
camel case suggests that, but looking briefly at the function, it looks
like its a verb? If its a verb, I think the "u" should be upper case
(i.e. "dbLockCleanUpRecords") since "cleanup" is a noun, and "clean up"
is a verb.

Or maybe there's already a precedent spelling in existing code, and
"dbLockCleanupRecords" is just following that?

Lewis

« Back to merge proposal