Skip to content

[#11]:svarga:refactor, expand attribute type coverage to match full type engine support#214

Merged
steven-varga merged 1 commit into
stagingfrom
11-attribute-type-coverage
May 16, 2026
Merged

[#11]:svarga:refactor, expand attribute type coverage to match full type engine support#214
steven-varga merged 1 commit into
stagingfrom
11-attribute-type-coverage

Conversation

@steven-varga
Copy link
Copy Markdown
Collaborator

Summary

  • Adds explicit impl::data() and impl::size() overloads for std::array<T,N> in H5Mstl.hpp so awrite creates a correct rank-1 N-element attribute (previously fell through to the scalar POD path)
  • Adds aread<A> overload gated on meta::is_array for clean std::array readback; guards arithmetic branch with !meta::is_array to prevent ambiguity
  • New test H5Aall.cpp: 6 test cases, 39 assertions covering arithmetic scalars, std::string, vector<T>, array<T,N>, initializer_list, and ds[name] = value operator

Test plan

  • ctest -R H5Aall — 6 test cases, 39 assertions pass
  • std::array<int,4>, <float,3>, <double,2> roundtrips all pass
  • CI matrix green

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant