Helmut-Ortmann / EnterpriseArchitect_hoReverse

ReqIF for EA, reverse Engineering workbench for Enterprise Architect and C/C++
MIT License
18 stars 7 forks source link

Crash on import of reqif files with 64bit integer datatype #2

Closed cbalster closed 4 years ago

cbalster commented 4 years ago

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 https://github.com/RHEAGROUP/reqifsharp/issues/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 4 years ago

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 notifications@github.com 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 RHEAGROUP/reqifsharp#13 https://github.com/RHEAGROUP/reqifsharp/issues/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 https://github.com/Helmut-Ortmann/EnterpriseArchitect_hoReverse/issues/2?email_source=notifications&email_token=ABJWGLANSPZFIBHUSCAEJIDQRE5QJA5CNFSM4JGU7VXKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HVKE3HQ, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABJWGLBUCXOIMYQGNQHD6FDQRE5QJANCNFSM4JGU7VXA .

cbalster commented 4 years ago

Thank you!