Skip to content

Conversation

hduelme
Copy link
Contributor

@hduelme hduelme commented Apr 30, 2022

Currently Java 8 is used and so collapsed catch blocks are supported. So I collapsed catch branches

@pivotal-cla
Copy link

@hduelme Please sign the Contributor License Agreement!

Click here to manually synchronize the status of this Pull Request.

See the FAQ for frequently asked questions.

@pivotal-cla
Copy link

@hduelme Thank you for signing the Contributor License Agreement!

@hduelme hduelme changed the title collapse identical catch branches Collapse identical catch branches Apr 30, 2022
gregturn pushed a commit that referenced this pull request Nov 16, 2022
Also take advantage of Java's try-with-resources feature.

Resolves #1248.
gregturn added a commit that referenced this pull request Nov 16, 2022
Related: #1248.
@gregturn
Copy link
Contributor

Thanks @hduelme. I've taken your PR, collapsed it into a single commit, rebased it against 4.0.x and did some final polishing.

@gregturn gregturn closed this Nov 16, 2022
@gregturn gregturn self-assigned this Nov 16, 2022
@gregturn gregturn added the type: enhancement A general enhancement label Nov 16, 2022
@gregturn gregturn added this to the 4.0.0 milestone Nov 16, 2022
gregturn pushed a commit that referenced this pull request Nov 16, 2022
Also take advantage of Java's try-with-resources feature.

Resolves #1248.
gregturn added a commit that referenced this pull request Nov 16, 2022
Related: #1248.
@gregturn
Copy link
Contributor

Backported to main

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: enhancement A general enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants