Skip to content

My changes#44

Closed
Achref280 wants to merge 4 commits intoudacity:masterfrom
Achref280:my-changes
Closed

My changes#44
Achref280 wants to merge 4 commits intoudacity:masterfrom
Achref280:my-changes

Conversation

@Achref280
Copy link

No description provided.

- Add copy constructor, copy assignment, move constructor, and move assignment operators to ChatBot
- Replace raw pointers with std::unique_ptr for _nodes and _childEdges
- Implement move semantics for ChatBot ownership in GraphNode
- Use std::make_unique for safer memory allocation
- Update .gitignore to exclude build/ and .DS_Store files
- Add debug output for constructor/destructor calls and GUI interactions
- Improve null pointer checking in GetImageHandle()
@Achref280 Achref280 requested a review from a team as a code owner February 17, 2026 21:14
@Achref280 Achref280 requested review from sarah-udacity and removed request for a team February 17, 2026 21:14
@Achref280 Achref280 marked this pull request as draft February 17, 2026 21:16
@Achref280 Achref280 closed this Feb 17, 2026
@Achref280 Achref280 deleted the my-changes branch February 17, 2026 21:20
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