remove base constructor

[neheb]

Maybe works now

Signed-off-by: Rosen Penev [email protected]

[ghost]

👇 Click on the image for a new way to code review

• Make big changes easier — review code in small groups of related files

• Know where to start — see the whole change at a glance

• Take a code tour — explore the change with an interactive tour

• Make comments and review — all fully sync’ed with github

  Try it now!

Legend

[codecov]

Codecov Report

Merging #2503 (e7fdebb) into main (976dcd8) will decrease coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #2503      +/-   ##
==========================================
- Coverage   64.67%   64.67%   -0.01%     
==========================================
  Files         104      103       -1     
  Lines       22256    22254       -2     
  Branches    10863    10862       -1     
==========================================
- Hits        14394    14392       -2     
  Misses       5619     5619              
  Partials     2243     2243              

Impacted Files	Coverage Δ
src/datasets.cpp	93.16% <ø> (-0.09%)	⬇️
src/properties.cpp	75.89% <100.00%> (ø)
src/tags.cpp	68.39% <100.00%> (ø)
include/exiv2/metadatum.hpp	78.57% <0.00%> (+1.64%)	⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

[neheb]

even g++-7 compiles. Maybe was an issue with older MSVC. Or rule of 0 saves the day. Not sure.