Make clear function argument case in dev notes #14625

pull dongcarl wants to merge 1 commits into bitcoin:master from dongcarl:patch-3 changing 1 files +1 −1

dongcarl commented at 12:25 AM on November 1, 2018: member

Rationale:

For new developers, they might be confused if they see that function arguments are sometimes camelCase'd in the codebase. This makes it clear that they should be snake_case'd (maybe because no one's gotten to fixing them yet).
Make clear function argument case in dev notes 9605bbd315
fanquake added the label Docs on Nov 1, 2018
fanquake commented at 3:34 AM on November 1, 2018: member

utACK 9605bbd

Related IRC discussion https://botbot.me/freenode/bitcoin-core-dev/2018-11-01/?msg=105900824&page=1
ken2812221 commented at 5:12 AM on November 1, 2018: contributor

ACK 9605bbd315eb14690427560fd9a274fe837f59f5
practicalswift commented at 10:41 AM on November 1, 2018: contributor

ACK 9605bbd315eb14690427560fd9a274fe837f59f5
MarcoFalke commented at 1:56 PM on November 1, 2018: member

ACK 9605bbd
laanwj commented at 3:31 PM on November 1, 2018: member

we discussed this on IRC thanks ! utACK 9605bbd315eb14690427560fd9a274fe837f59f5
laanwj merged this on Nov 1, 2018
laanwj closed this on Nov 1, 2018
laanwj referenced this in commit 5049f7f7a9 on Nov 1, 2018
PastaPastaPasta referenced this in commit 7cde6b1495 on Jun 27, 2021
PastaPastaPasta referenced this in commit 5f0e39b662 on Jun 28, 2021
PastaPastaPasta referenced this in commit c8a6e3b5d5 on Jun 29, 2021
PastaPastaPasta referenced this in commit 96fd9d1600 on Jul 1, 2021
PastaPastaPasta referenced this in commit e1e5219306 on Jul 1, 2021
PastaPastaPasta referenced this in commit fb2c3f06e6 on Jul 1, 2021
PastaPastaPasta referenced this in commit a54a48b70d on Jul 3, 2021
MarcoFalke locked this on Sep 8, 2021

Contributors

Labels