Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add missing translator tests for C++, Nim and Rust #290

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Mingun
Copy link
Contributor

@Mingun Mingun commented Mar 27, 2024

Do not merge yet, it is not finished. Not a draft because I want to run CI checks

I noticed that TranslatorSpec.scala does not test some languages, which seems to be oversight for Nim, and unfinished work for Rust.

Nim compiler seems to be production ready, so tests needs to be added.

Rust compiler is not finished yet, and tests should make finishing it easy.

@pombredanne
Copy link

Thanks for the work there. Rust would be awesome!

@generalmimon
Copy link
Member

generalmimon commented Oct 27, 2024

@pombredanne:

Rust would be awesome!

There is already quite decent support for Rust on unreleased master branches, as explained at https://app.gitter.im/#/room/!gxFYJoZuKcRouBfhlb:gitter.im/$U2qXwDnOd6HKOMzdHpcXehPmfg95pxgP4VA2D0CTtMk. So if you don't mind building the Kaitai Struct compiler from source (which is actually easy, as you can see at https://doc.kaitai.io/serialization.html#_building_the_compiler_from_source - just make sure to check out the master branch, not serialization as it says), you can already try it out now before a new version of Kaitai Struct is released. If you do, please give us feedback in kaitai-io/kaitai_struct#22 or open issues in https://github.com/kaitai-io/kaitai_struct/issues for bugs/suggestions/questions.

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.

3 participants