Skip to content

019 perf dedup cleanup#8

Merged
timschwartz merged 4 commits intomainfrom
019-perf-dedup-cleanup
Apr 13, 2026
Merged

019 perf dedup cleanup#8
timschwartz merged 4 commits intomainfrom
019-perf-dedup-cleanup

Conversation

@timschwartz
Copy link
Copy Markdown
Member

No description provided.

…ding RPC dispatch refactor and data model updates
…g refactoring of peer operations and RPC dispatch
- Introduced a dispatch map to handle various RPC requests in RpcServer.
- Added handler functions for multiple RPC methods including stream management and peer operations.
- Updated the RpcServer header to include necessary includes and new member functions.

Enhance logging utilities in utils.hpp

- Added lazy log macros for different log levels to improve logging efficiency.
- Updated the generate_uuid_v4 function declaration to ensure proper formatting.

Improve test helpers for block creation and chain building

- Added make_block function to TestHelpers for fast block creation with difficulty 0.
- Updated chunk persistence tests to utilize new TestHelpers functions for directory management and block creation.

Refactor consensus tests to use TestHelpers for block mining

- Replaced inline block mining logic with TestHelpers::mineTestBlock for consistency and readability.
- Updated various test cases to utilize the new helper functions for block creation.

Refactor sync tests to use TestHelpers for block creation

- Replaced inline block mining logic with TestHelpers::mineTestBlock and chain building with TestHelpers::buildValidChain.
- Improved readability and maintainability of sync tests.

Add serialization tests for packet headers and payloads

- Implemented a test case to verify the correctness of packet serialization and deserialization for Block objects.
@timschwartz timschwartz merged commit a67cd6b into main Apr 13, 2026
4 checks passed
@timschwartz timschwartz deleted the 019-perf-dedup-cleanup branch April 13, 2026 21:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant