aboutsummaryrefslogtreecommitdiff
path: root/doc/developer-notes.md
AgeCommit message (Collapse)Author
2017-09-06(finally) remove getinfo in favor of more module-specific infosMatt Corallo
2017-08-25[doc] Add RPC response notesJoão Barbosa
2017-08-22Add recommendation: By default, declare single-argument constructors `explicit`practicalswift
2017-08-22Merge #11050: Avoid treating null RPC arguments different from missing argumentsWladimir J. van der Laan
745d2e3 Clean up getbalance RPC parameter handling (Russell Yanofsky) fd5d71e Update developer notes after params.size() cleanup (Russell Yanofsky) e067673 Avoid treating null RPC arguments different from missing arguments (Russell Yanofsky) e666efc Get rid of redundant RPC params.size() checks (Russell Yanofsky) Pull request description: This is a followup to #10783. - The first commit doesn't change behavior at all, just simplifies code. - The second commit just changes RPC methods to treat null arguments the same as missing arguments instead of throwing type errors. - The third commit updates developer notes after the cleanup. - The forth commit does some additional code cleanup in `getbalance`. Followup changes that should happen in future PRs: - [ ] Replace uses of `.isTrue()` with calls to `.get_bool()` so numbers, objects, and strings cause type errors instead of being interpreted as false. https://github.com/bitcoin/bitcoin/pull/11050#discussion_r133850525 - [ ] Add braces around if statements. https://github.com/bitcoin/bitcoin/pull/11050#discussion_r133851133 - [ ] Maybe improve UniValue type error exceptions and eliminate RPCTypeCheck and RPCTypeCheckArgument functions. https://github.com/bitcoin/bitcoin/pull/11050#discussion_r133829303 Tree-SHA512: e72f696011d20acc0778e996659e41f9426bffce387b29ff63bf59ad1163d5146761e4445b2b9b9e069a80596a57c7f4402b75a15d5d20f69f775ae558cf67e9
2017-08-18Document the preference of nullptr over NULL or (void*)0practicalswift
2017-08-17Prefer compile-time checking over run-time checkingpracticalswift
2017-08-16Merge #10705: Trivial: spelling fixesMarcoFalke
f42fc1d50 doc: spelling fixes (klemens) Pull request description: patch contains some spelling fixes ( just in comments ) as found by a bot ( http://www.misfix.org, https://github.com/ka7/misspell_fixer ). Tree-SHA512: ba6046cfcd81b0783420daae7d776be92dd7b85a593e212f8f1b4403aca9b1b6af12cef7080d4ea5ed4a14952fd25e4300109a59c414e08f5395cdb9947bb750
2017-08-16doc: spelling fixesklemens
2017-08-14Update developer notes after params.size() cleanupRussell Yanofsky
2017-08-07doc: Correct AmountFromValue/ValueFromAmount namesWladimir J. van der Laan
2017-07-25developer-notes: add reference to snake_case and PascalCaseMarko Bencun
CamelCase is ambiguous as it refers to both lowerCamelCase and UpperCamelCase, whereas PascalCase is only UpperCamelCase.
2017-06-26Merge #9544: [trivial] Add end of namespace comments. Improve consistency.Wladimir J. van der Laan
5a9b508 [trivial] Add end of namespace comments (practicalswift) Tree-SHA512: 92b0fcae4d1d3f4da9e97569ae84ef2d6e09625a5815cd0e5f0eb6dd2ecba9852fa85c184c5ae9de5117050330ce995e9867b451fa8cd5512169025990541a2b
2017-06-22Fixed multiple typosDimitris Tsapakidis
A few "a->an" and "an->a". "Shows, if the supplied default SOCKS5 proxy" -> "Shows if the supplied default SOCKS5 proxy". Change made on 3 occurrences. "without fully understanding the ramification of a command" -> "without fully understanding the ramifications of a command". Removed duplicate words such as "the the".
2017-06-11Header include guidelinePieter Wuille
2017-05-31[trivial] Add end of namespace commentspracticalswift
2017-05-26Update style guidePieter Wuille
2017-05-10Shadowing is not enabled by default, update doc accordingly.Pavel Janík
2017-05-02doc: Add RPC interface guidelinesWladimir J. van der Laan
2017-03-20Rename rpc-tests directory to functionalJohn Newbery
2017-03-20Rename qa directory to testJohn Newbery
2017-02-09Fix typo and spelling inconsistency in CONTRIBUTING.mdKoki Takahashi
Fix spellings of GitHub Remove unnecessary changes Fix GitHub spelling on doc/translation_process.md
2017-01-28[doc] Remove unused clang format dev scriptMarcoFalke
Also, update the clang format file to reflect the current coding style mentioned in the developer notes.
2017-01-10Improve style w.r.t. ifPieter Wuille
2016-12-02Developer docs about existing subtrees.Gregory Maxwell
Add some notes about the workflow for changes to code that exists in subtrees.
2016-11-09Add notes about variable names and shadowingPavel Janík
2016-08-26[trivial][doc] Mention ++i as preferred to i++ in dev notesfanquake
2016-06-10Merge #8177: developer notes: updates for C++11Wladimir J. van der Laan
654a211 developer notes: updates for C++11 (Kaz Wesley)
2016-06-09developer notes: updates for C++11Kaz Wesley
- boost::scoped_ptr is obsolete - std::vector::data replaces begin_ptr / end_ptr
2016-06-09Merge #8178: Add git and github tips and tricks to developer notesWladimir J. van der Laan
3144449 Add git and github tips and tricks to developer notes (Pieter Wuille)
2016-06-09Add git and github tips and tricks to developer notesPieter Wuille
2016-06-08util: Add ParseUInt32 and ParseUInt64Wladimir J. van der Laan
Add error and range-checking parsers for unsigned 32 and 64 bit numbers. The 32-bit variant is required for parsing sequence numbers from the command line in `bitcoin-tx` (see #8164 for discussion). I've thrown in the 64-bit variant as a bonus, as I'm sure it will be needed at some point. Also adds tests, and updates `developer-notes.md`.
2016-05-10[doc] Link to clang-format in the developer notesMarcoFalke
2016-04-03[doc] Fix doxygen comments for membersMarcoFalke
2015-11-30[doc] Minor markdown fixesMarcoFalke
2015-11-13doc: Add non-style-related development guidelinesWladimir J. van der Laan
I've collected these over time, mostly adding notes after troubleshooting obscure bugs. As I hope to get the community more involved in the whole process, I think it is useful to add to the developer-notes.
2015-10-23doc: Add developer notes about gitignoreWladimir J. van der Laan
- Add developer notes about `.gitignore.` - Remove qt creator specific files from gitignore, to be consistent.
2015-10-18Correct spelling mistakes in doc folderMitchell Cash
- OSX —> OS X - XCode —> Xcode - github —> GitHub - homebrew —> Homebrew - gitian —> Gitian - Other miscellaneous obvious spelling fixes and whitespace removal
2015-10-09[doc] trivial: developer-notes.md covers more than just coding standardsMarcoFalke
2015-09-26Add CONTRIBUTING.mdBtcDrak
2015-05-19Update documentation to match the #4219 changelpescher
2015-05-01Bugfix: Grammar fixesCorinne Dashjr
2014-12-19Create developer-notes.mdMichael Ford
Moves coding guidelines and development tips/tricks into a single file. Also adds a section explaining pull request terminology.
generated by cgit v1.2.3 (git 2.26.2) at 2024-07-18 21:14:10 +0000