Refactor classes for atomic types

ZedThree commented 10 months ago

make ctors all inline
use intermediate class NcAtomic to reduce duplication of equality operator
now classes are only a few lines each, move them all into ncType.h header

Includes #144. In total this removes ~1k lines

WardF commented 10 months ago

Reviewing the CXX PR's, I have some work to do with the cxx4 CI, e.g. 'add some'. Once I have that in place I will be able to go through and merge the PR's you've submitted. Thanks!

ZedThree commented 10 months ago

Did you see #139? It adds some basic CI

On Tue, 23 Jan 2024, 19:36 Ward Fisher, @.***> wrote:

Reviewing the CXX PR's, I have some work to do with the cxx4 CI, e.g. 'add some'. Once I have that in place I will be able to go through and merge the PR's you've submitted. Thanks!

— Reply to this email directly, view it on GitHub https://github.com/Unidata/netcdf-cxx4/pull/146#issuecomment-1906794781, or unsubscribe https://github.com/notifications/unsubscribe-auth/AALLAXW3ZH3SAYEC233INYDYQAGKVAVCNFSM6AAAAABBNFNX6KVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMBWG44TINZYGE . You are receiving this because you authored the thread.Message ID: @.***>

Unidata / netcdf-cxx4

Refactor classes for atomic types #146