Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Foundation Classes - Update Map to store insert order #140

Draft
wants to merge 1 commit into
base: IR
Choose a base branch
from

Conversation

dpasukhi
Copy link
Member

@dpasukhi dpasukhi commented Nov 2, 2024

Changes to keep the insert order to improve iteration procedure.
Making TCollection deprecated as a not clear definition.
Refactor NCollection_Map to be more readable
TODO: update DataMap and DoubleMap

@dpasukhi dpasukhi added 2. Enhancement New feature or request 1. Foundation Classes Containers, system calls wrappers, smart pointers and other low level of OCCT code labels Nov 2, 2024
@dpasukhi dpasukhi self-assigned this Nov 2, 2024
…-SAS#140

- Updated Map to store insert order to improve iteration procedure.
- Fixed linked list management in NCollection_Map.
- Fixed MapNode constructor parameter order and improved null checks.
- Ensured Next pointer is set to nullptr when no placement is provided in NCollection_Map.
- Simplified More() method in NCollection_Map to only check for nullptr.
- Optimized Exchange method in NCollection_Map to swap first and last pointers.
- Added resetSize method and enhanced Destroy method in NCollection_Map for better memory management.
@dpasukhi dpasukhi added this to the Release 8.0 milestone Jan 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1. Foundation Classes Containers, system calls wrappers, smart pointers and other low level of OCCT code 2. Enhancement New feature or request
Projects
Status: Todo
Development

Successfully merging this pull request may close these issues.

1 participant