code_review.md

Self Review Checklist

After every exercise:

1. Did I use any raw new or delete? (if yes, fix it)
2. Is every resource owned by exactly one RAII wrapper?
3. Does this compile cleanly with -Wall -Wextra -Werror?
4. Can I explain why I chose shared_ptr vs unique_ptr vs raw reference here?
5. If I used mutex, is there any scenario where this deadlocks?
6. If I used an atomic, is the memory orering correct and minimal?
7. Would moving this type into a std::vector trigger a copy instead of a move? Why?