poco icon indicating copy to clipboard operation
poco copied to clipboard

Configuration: remove raw pointers from interfaces

Open obiltschnig opened this issue 7 years ago • 3 comments

Similar to Logging.

obiltschnig avatar Jan 02 '18 16:01 obiltschnig

same as here, although more severe - there were duplicate add calls (non-shared and shared version), so adding AutoPtr overloads on top of that seemed like too much; I'm not quite sure why those overloads existed and I'm thinking everything should be shared, simpler and cleaner. If we decide that we really, really want to keep the pointer versions, then those can be reinstated, but I think internally (for Logging and Configuration) things should only be kept as AutoPtr

Note: valgrind reports tons of leaks from the new PropertyListConfiguration, I guess those should be resolved when this issue is taken care of /cc @DraconPern

aleks-f avatar Mar 01 '18 03:03 aleks-f

Tracking. Let me see if I can fix the memory leaks.

DraconPern avatar Mar 01 '18 06:03 DraconPern

This issue is stale because it has been open for 365 days with no activity.

github-actions[bot] avatar Aug 26 '22 03:08 github-actions[bot]

This issue was closed because it has been inactive for 60 days since being marked as stale.

github-actions[bot] avatar Jan 02 '23 02:01 github-actions[bot]