search

sierrafoxtrot / srecord

SRecord github Mirror
https://srecord.sourceforge.net/
GNU General Public License v3.0
42 stars 22 forks source link

[WIP] Modernization/migrate to smart pointers #49

Open SeanAlling-DojoFive opened 1 year ago

SeanAlling-DojoFive commented 1 year ago

Blocked by PR #48

Start migrating code when applicable to:

  1. Use smart pointers
  2. Container objects

Goal is to reduce call of new/delete within source code.

To help validate a branch utilizing Valgrind was created. Cherry pick commit as needed for testing.

Running CTest configured for Valgrind will take significantly longer than normal to run.

sierrafoxtrot commented 1 year ago

Hi Sean. Apologies for falling behind but did you still want to move forward with this one?

SeanAlling-DojoFive commented 1 year ago

I have been a little busy with life events recently. Will be coming back to this in a couple weeks.

sierrafoxtrot commented 1 year ago

Completely understand. Nothing but empathy from me on that one. I hope all is well.