Skip to content

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

Open
Mingun wants to merge 2 commits intokaitai-io:masterfrom
Mingun:translator-tests
Open

Add missing translator tests for C++, Nim and Rust#290
Mingun wants to merge 2 commits intokaitai-io:masterfrom
Mingun:translator-tests

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.

@Mingun Mingun force-pushed the translator-tests branch 7 times, most recently from 607aad2 to e7e708b Compare March 27, 2024 20:22
@Mingun Mingun force-pushed the translator-tests branch 4 times, most recently from 3f3ce4c to 72cfbec Compare March 28, 2024 17:39
@Mingun Mingun force-pushed the translator-tests branch from 72cfbec to ffca8d7 Compare April 16, 2024 14:18
@Mingun Mingun force-pushed the translator-tests branch 2 times, most recently from ffdc854 to eeaf8b8 Compare September 8, 2024 10:02
@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