Skip to content

Regenerate all files using wrap-java#627

Merged
ktoso merged 10 commits intoswiftlang:mainfrom
sidepelican:wrap_all
Mar 18, 2026
Merged

Regenerate all files using wrap-java#627
ktoso merged 10 commits intoswiftlang:mainfrom
sidepelican:wrap_all

Conversation

@sidepelican
Copy link
Contributor

Completes #599.

This PR regenerates all files within the generated/ directory using the wrap-java command.
To streamline this process, I have also added a utility script: scripts/wrap-java-generate.sh.

Any manually implemented extension have been moved to separate files.

Notable Changes

  • JarEntry, JarFile, JarInputStream, JarOutputStream, and ZipEntry:
    Static fields originating from ZipConstants are no longer included, as ZipConstants is a private interface.
  • JavaCharacter.UnicodeScript:
    Removed because the generator does not yet support enum generation.
  • AbstractMap.SimpleEntry, AbstractMap.SimpleImmutableEntry:
    This was previously located in TreeMap, but it correctly belongs to AbstractMap. Since AbstractMap is not currently a generation target and has relatively niche utility, I have omitted it for now.

@sidepelican sidepelican requested a review from ktoso as a code owner March 18, 2026 02:45
@ktoso
Copy link
Collaborator

ktoso commented Mar 18, 2026

Very exciting! We should follow up with checking in CI that there’s no change unless it’s also committed… I’ll do that later.

some small nits incoming, this is great though!

@@ -0,0 +1,87 @@
#!/bin/bash
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's a good script but we'll want to make it a bit more automatic...

I think I'm ok with merging this as is and I'll work on CI infra for validation of these things and update the script then as well 👍

@ktoso ktoso merged commit 83540c6 into swiftlang:main Mar 18, 2026
61 checks passed
@sidepelican sidepelican deleted the wrap_all branch March 18, 2026 05:32
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.

2 participants