Skip to content

Replace deprecated (xs, ys).zipped by xs.lazyZip(ys)#327

Open
Mingun wants to merge 1 commit intokaitai-io:masterfrom
Mingun:replace-deprecated-zipped
Open

Replace deprecated (xs, ys).zipped by xs.lazyZip(ys)#327
Mingun wants to merge 1 commit intokaitai-io:masterfrom
Mingun:replace-deprecated-zipped

Conversation

@Mingun
Copy link
Contributor

@Mingun Mingun commented Dec 24, 2025

This fixes 4 warnings (each is duplicated for JVM and JS target):

[warn] /home/runner/work/kaitai_struct_compiler/kaitai_struct_compiler/compiler/shared/src/main/scala/io/kaitai/struct/languages/components/EveryWriteIsExpression.scala:311:40: method zipped in class Ops is deprecated (since 2.13.0): Use xs.lazyZip(ys)
[warn]       (t.classSpec.get.params, t.args).zipped.foreach { (paramDef, argExpr) =>
[warn]                                        ^
[warn] /home/runner/work/kaitai_struct_compiler/kaitai_struct_compiler/compiler/shared/src/main/scala/io/kaitai/struct/languages/components/GenericChecks.scala:401:42: method zipped in class Ops is deprecated (since 2.13.0): Use xs.lazyZip(ys)
[warn]       (ut.classSpec.get.params, ut.args).zipped.foreach { (paramDef, argExpr) =>
[warn]                                          ^

This fixes 4 warnings (each is duplicated for JVM and JS target):

```
[warn] /home/runner/work/kaitai_struct_compiler/kaitai_struct_compiler/compiler/shared/src/main/scala/io/kaitai/struct/languages/components/EveryWriteIsExpression.scala:311:40: method zipped in class Ops is deprecated (since 2.13.0): Use xs.lazyZip(ys)
[warn]       (t.classSpec.get.params, t.args).zipped.foreach { (paramDef, argExpr) =>
[warn]                                        ^
[warn] /home/runner/work/kaitai_struct_compiler/kaitai_struct_compiler/compiler/shared/src/main/scala/io/kaitai/struct/languages/components/GenericChecks.scala:401:42: method zipped in class Ops is deprecated (since 2.13.0): Use xs.lazyZip(ys)
[warn]       (ut.classSpec.get.params, ut.args).zipped.foreach { (paramDef, argExpr) =>
[warn]                                          ^
```
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.

1 participant