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: noneCamelCase 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: noneI 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
-
sipa commented at 9:11 am on July 24, 2017: memberInteresting; 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: noneThat is interesting ;) I made another revision:
UpperCamelCase (PascalCase)
. -
sipa commented at 3:48 am on July 25, 2017: memberutACK
-
promag commented at 9:55 am on July 25, 2017: memberPlease 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.
-
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: memberPascalCase :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
github-metadata-mirror
This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2024-11-17 15:12 UTC
This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2024-11-17 15:12 UTC
This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me
More mirrored repositories can be found on mirror.b10c.me