search

CasperLabs / datasize-rs

Heap memory usage estimation
Other
16 stars 13 forks source link

fix Arc, Rc and add Range impls #7

Closed s3bk closed 1 year ago

s3bk commented 1 year ago

I had to made some changes and would like to ask if they can be merged and new version be released. (It is used in the pdf crate, and I want to release a new version for it...)

s3bk commented 1 year ago

I can't write code this weak, but i am 99% certain that Range is equivalent of a (T, T) tuple. That.. should mean the heap size the sum of the heap sizecof start and end.

marc-casperlabs commented 1 year ago

I get it now, it did not occur to me to use heap-allocating types as range bounds before, but I'm sure there's a legit reason somewhere :)

s3bk commented 1 year ago

I am sure some use could be found...

Thanks!

s3bk commented 1 year ago

Could you upload a new release to crates.io ?