Mingun

Move test data from `kaitai_struct_tests` to a separate repository

2

I developing [an alternative kaitai struct compiler](https://github.com/Mingun/ksc-rs) and I want to test it against existing test files. For that I plan to add a repository with tests as a git...

Bit-sized integers allow strange syntax

4

This definitions is valid and it seems they should be forbidden: ```yaml seq: - id: zeroes type: b00000008 # == b8 - id: zero type: b0 # should be forbidden...

Analysing code when working [on my own implementation](https://github.com/Mingun/ksc-rs) of the compiler, I found that the [`parent`](https://doc.kaitai.io/user_guide.html#_enforcing_parent_type) key is [extracted](https://github.com/kaitai-io/kaitai_struct_compiler/blob/e8542816277c0f78a17afcd4eed756337df9cd70/shared/src/main/scala/io/kaitai/struct/format/AttrSpec.scala#L189) as an expression, but actually the only possible expressions are: -...

This should fix 4 tests: https://github.com/kaitai-io/kaitai_struct_compiler/runs/4203900545?check_suite_focus=true#step:4:589 https://github.com/kaitai-io/kaitai_struct_compiler/runs/4203900545?check_suite_focus=true#step:4:692 https://github.com/kaitai-io/kaitai_struct_compiler/runs/4203900545?check_suite_focus=true#step:4:746 ``` [info] - cpp_stl:[0, 1, 100500] *** FAILED *** [info] java.lang.RuntimeException: C++ literal arrays are not implemented yet [info] at io.kaitai.struct.translators.CppTranslator.doArrayLiteral(CppTranslator.scala:119) [info]...

This should fix 3 tests: https://github.com/kaitai-io/kaitai_struct_compiler/runs/4203900545?check_suite_focus=true#step:4:651 ``` [info] - go:other.as.bar *** FAILED *** [info] scala.NotImplementedError: an implementation is missing [info] at scala.Predef$.$qmark$qmark$qmark(Predef.scala:288) [info] at io.kaitai.struct.translators.GoTranslator.doCast(GoTranslator.scala:257) [info] at io.kaitai.struct.translators.GoTranslator.doCast(GoTranslator.scala:15) [info] at...

`_sizeof` special property does not properly handle conditional fields

1

The following KSY successfully compiles although `_sizeof` expression depends on the presence of the `value` field which is dependent on the `_sizeof` result. ```yaml meta: id: sizeof_bug seq: - id:...

This KSY when used without providing argument to the parameter (as WebIDE does) produces a `value` with value of `[undefined]`, because [the first `Array` constructor](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/Array) will be used (that accept...

https://ide.kaitai.io/devel/ (and thus the [current master](https://github.com/kaitai-io/kaitai_struct_webide/commit/a8f6f9c022761a06a3860f728ff68af63f04b483) for WebIDE) does not report an error for unknown fields: ```yaml meta: id: unknown_attribute_bug seq: - id: value size: unknown ``` compiled succesfully

Optimize internally tagged enums -- do not use internal buffer if tag is the first field

7

Fixes #1495 Also that change has one positive side-effect: if tag is the first field, negative effects from #1183 is eliminated, because buffering is not used @RReverser, feel free to...

Add pass to find unreachable rules

1

Steadily in the project there are questions of why some grammar can't parse certain input data: * #487 * #484 * #482 * #435 * #394 * #326 * #302...