search

rive-app / rive-cpp-legacy

C++ runtime for Rive
MIT License
287 stars 43 forks source link

Use unique_ptr when we know we have to manage its lifetime #210

Closed mikerreed closed 2 years ago

mikerreed commented 2 years ago
  1. Don't need custom destructors
  2. Won't ever forget to delete them
  3. Someone reading the code get's the contract -- they know who will delete the object
mikerreed commented 2 years ago

as I was looking at hittesting, I realized I was confused about commandpath lifetimes... hence this PR