developer-notes: add reference to snake_case and PascalCase #10917

pull benma wants to merge 1 commits into bitcoin:master from benma:pascalcase changing 1 files +3 −3

benma commented at 6:48 AM on July 24, 2017: none

CamelCase is ambiguous as it refers to both lowerCamelCase and UpperCamelCase, whereas PascalCase is only UpperCamelCase.
benma commented at 6:50 AM on July 24, 2017: none

I realize that the way CamelCase was written, it implied UpperCamelCase, but I still did a double take as I have only ever seen camelCase refer to lowerCamelCase.
fanquake added the label Docs and Output on Jul 24, 2017
bytting commented at 8:00 AM on July 24, 2017: contributor

Variable and namespace names are all lowercase, and may use _ to separate words.

This describes what is called "snake_case". Maybe the documentation should mention that too.
benma force-pushed on Jul 24, 2017
benma commented at 9:10 AM on July 24, 2017: none

@corebob good idea; added.
sipa commented at 9:11 AM on July 24, 2017: member

Interesting; I had never heard of PascalCase, and only seen CamelCase to refer to it.
benma force-pushed on Jul 24, 2017
benma commented at 9:41 AM on July 24, 2017: none

That is interesting ;) I made another revision: UpperCamelCase (PascalCase).
sipa commented at 3:48 AM on July 25, 2017: member

utACK
promag commented at 9:55 AM on July 25, 2017: member

Please rename PR title and commit to something like Add reference to snake_case and PascalCase in developer notes. Missing snake_case in PR description. Otherwise ACK.

developer-notes: add reference to snake_case and PascalCase

CamelCase is ambiguous as it refers to both lowerCamelCase and
UpperCamelCase, whereas PascalCase is only UpperCamelCase.

4a7a4ff1f1

benma commented at 10:39 AM on July 25, 2017: none

@promag done
benma force-pushed on Jul 25, 2017
benma renamed this:
~~developer-notes: CamelCase -> PascalCase~~
developer-notes: add reference to snake_case and PascalCase
on Jul 25, 2017
laanwj commented at 9:04 AM on July 26, 2017: member

PascalCase :heart: utACK 4a7a4ff
laanwj merged this on Jul 26, 2017
laanwj closed this on Jul 26, 2017
laanwj referenced this in commit 5c003cb1af on Jul 26, 2017
benma deleted the branch on Sep 19, 2019
PastaPastaPasta referenced this in commit ddd4b8b4fc on Sep 19, 2019
PastaPastaPasta referenced this in commit 8e9d322901 on Sep 23, 2019
PastaPastaPasta referenced this in commit db912de175 on Sep 24, 2019
PastaPastaPasta referenced this in commit e0f11963f4 on Nov 19, 2019
PastaPastaPasta referenced this in commit ce785099aa on Dec 9, 2019
PastaPastaPasta referenced this in commit 6a6c267c0f on Jan 1, 2020
PastaPastaPasta referenced this in commit cca4376fbd on Jan 2, 2020
ckti referenced this in commit 5aa470499e on Mar 28, 2021
DrahtBot locked this on Dec 16, 2021

Contributors

Labels