search

atilaneves / cerealed

Powerful binary serialisation library for D
BSD 3-Clause "New" or "Revised" License
92 stars 3 forks source link

Fix nested dynamic array serialization #20

Closed janpapaj closed 5 years ago

janpapaj commented 5 years ago

Hi, This PR fixes compile error when dynamic arrays of nested structures are cerealised. Tested with both, dmd and ldc compilers.

codecov-io commented 5 years ago

Codecov Report

Merging #20 into master will increase coverage by <.01%. The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #20      +/-   ##
==========================================
+ Coverage   98.91%   98.92%   +<.01%     
==========================================
  Files          25       25              
  Lines        1018     1025       +7     
==========================================
+ Hits         1007     1014       +7     
  Misses         11       11
Impacted Files Coverage Δ
tests/nested.d 100% <100%> (ø) :arrow_up:
src/cerealed/cerealiser.d 100% <100%> (ø) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update f024772...cc728a9. Read the comment docs.

atilaneves commented 5 years ago

Thanks!

janpapaj commented 5 years ago

Thank you too! When do you think you could create a new release with this patch to make it available through DUB package? Thanks.

atilaneves commented 5 years ago

I just tagged and pushed v0.6.11