unit test coverage - Githubissues

lhmouse / intrusive_ptr

Proposal for std::intrusive_ptr

MIT License

37 stars 4 forks source link

unit test coverage #5

Open nlguillemot opened 2 years ago

nlguillemot commented 2 years ago

First of all, thank you for writing this very neat proposal!

I thought I'd make a suggestion to include unit tests with more code coverage. That way it would be easier to check that everything in the implementation works for a given compiler environment.

Cheers!

lhmouse commented 2 years ago

Wow, there haven't been any updates for years...

Perhaps we can take a look at some real use: https://github.com/lhmouse/asteria/blob/master/rocket/refcnt_ptr.hpp