Crash on import of reqif files with 64bit integer datatype
Closed this issue · 2 comments
cbalster commented
Hi Helmut,
first of all thank you for open sourcing all this work!
When experimenting with hoReverse I noticed a crash with reqif exports that define an integer datatype with a min/max of 64bit. I pinned this down to the reqifsharp library, that assumed integer to always be 32bit. I already opened an issue with them and it was fixed in record time, see STARIONGROUP/reqifsharp#13
Just wanted to make you aware and kindly ask if you could update your dependency and release a new version when possible.
Thanks again!
Helmut-Ortmann commented
Hi Christian,
thanks for your engagement!
Yes, my experiences with the reqifsharp library are the same - very quick.
I've updated the reqifsharp library to 2.0.0 and released it in hoReverse
V3.7.0. Hope it works!
Best regards,
Helmut
…On Wed, Oct 30, 2019 at 9:02 AM Christian Balster ***@***.***> wrote:
Hi Helmut,
first of all thank you for open sourcing all this work!
When experimenting with hoReverse I noticed a crash with reqif exports
that define an integer datatype with a min/max of 64bit. I pinned this down
to the reqifsharp library, that assumed integer to always be 32bit. I
already opened an issue with them and it was fixed in record time, see
STARIONGROUP/reqifsharp#13
<STARIONGROUP/reqifsharp#13>
Just wanted to make you aware and kindly ask if you could update your
dependency and release a new version when possible.
Thanks again!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#2?email_source=notifications&email_token=ABJWGLANSPZFIBHUSCAEJIDQRE5QJA5CNFSM4JGU7VXKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HVKE3HQ>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABJWGLBUCXOIMYQGNQHD6FDQRE5QJANCNFSM4JGU7VXA>
.
cbalster commented
Thank you!