- Queries
- All Stories
- Search
- Advanced Search
- Transactions
- Transaction Logs
All Stories
Sep 9 2021
Using a set in the proof builder is a better solution but it breaks tests for duplicated inputs, since it's no longer possible to build them with the proof builder and so it requires a bit more test tooling. This is going to happen in a follow-up diff, for now I moved the TestProofBuilder structure to avalanche/test/util.h (will be expanded for duplicate inputs) and added a comment regarding the sorting function that needs to be removed from getProofId().
Going to use a std::set of stakes to keep them sorted all the time. This is more complicated to implement (and test) but more robust in the end since it enforces consistency.
update TX_NOT_STANDARD comment
it was missing an important line: add Check(request); in HandleRequest
Sep 8 2021
add parentheses
remove the arg also from vRPCConvertParams
linter be mad
nit: link to the specific commit in the summary is wrong, points to 3/3 rather than 2/3
Sep 7 2021
Additional note: I had a quick look to see if there is an automated linter solution (autopep8...) for this. I couldn't find any.