search

zalando / intellij-swagger

A plugin to help you easily edit Swagger and OpenAPI specification files inside IntelliJ IDEA
https://plugins.jetbrains.com/plugin/8347
MIT License
1.18k stars 74 forks source link

Use try with resources for file system #367

Closed asaarilahti closed 1 year ago

asaarilahti commented 1 year ago

Related to #332 - close file system also in case an exception is thrown. Unlikely to be an actual fix, but at least should rule out earlier failures as a cause.

Also switch from a visitor to a stream.

cberg-zalando commented 1 year ago

Any reason for the switch from visitor to stream?

asaarilahti commented 1 year ago

Any reason for the switch from visitor to stream?

Nothing major. I think the visitor is not that common nowadays, so streams might make the code easier to read.