Develop #98
Develop #98
Conversation
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Feautre/new mem
Feature/map
There was a problem hiding this comment.
Pull Request Overview
This PR represents a major development phase ("Develop" branch) implementing comprehensive test coverage and significant API refactoring across the LibJuno library. The changes introduce new test suites, update existing implementations to use pointer-based APIs, and add template code for common patterns.
Key Changes:
- Added comprehensive test suites achieving 100% code coverage for queue, heap, map, and memory modules
- Refactored core data structures (queue, heap, map) to use pointer-based APIs instead of direct memory references
- Updated time conversion functions to use integer-only arithmetic with proper overflow checks
- Added template files for common patterns (message types, arrays, applications, libraries)
Reviewed Changes
Copilot reviewed 116 out of 119 changed files in this pull request and generated 6 comments.
Show a summary per file
| File | Description |
|---|---|
| tests/test_queue.c | New comprehensive test suite for queue API with 960 lines covering all operations and edge cases |
| tests/test_heap.c | Complete rewrite expanding from 425 to 1117 lines with full coverage of heap operations |
| tests/test_map.c | Comprehensive map API tests expanded from 356 to 959 lines covering collision handling |
| tests/test_memory.c | Updated memory tests for new pointer-based API, removing reference counting |
| tests/test_math.c | Fixed function calls to use L2Norm2 variants and added quaternion tests |
| tests/test_crc.c | Added license header and corrected expected CRC values for empty data |
| tests/test_cpp.cpp | Updated includes to reflect API reorganization |
| tests/test_buff_cpp.cpp | Updated type names to use _ROOT_T suffix |
| tests/test_buff.c | Removed obsolete buffer implementation |
| tests/test_hash.c | Removed obsolete hash implementation |
| templates/* | Added template files for message types, arrays, applications, and libraries |
| src/juno_time.c | Refactored conversion functions to use integer arithmetic and proper overflow checking |
| src/juno_memory_block.c | Updated to pointer-based API with alignment checks and improved error handling |
| src/juno_map.c | Complete rewrite to use array-backed storage with linear probing |
| src/juno_heap.c | Major refactor to pointer-based API with new insert/pop/heapify functions |
| src/juno_crc.c | Added null/size checks and fixed include paths |
| src/juno_buff_*.c | New implementations for queue and stack using array API |
| src/juno_broker.c | New service broker implementation for pub-sub messaging |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Copilot reviewed 117 out of 120 changed files in this pull request and generated 3 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull Request Overview
Copilot reviewed 117 out of 120 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
No description provided.