Skip to content
This repository was archived by the owner on Jun 18, 2024. It is now read-only.

Use IOUtils.closeQuietly instead of reimplementing that method in each location#439

Merged
serious6 merged 1 commit intoOfficeDev:masterfrom
candrews:closeQuietly
Nov 8, 2015
Merged

Use IOUtils.closeQuietly instead of reimplementing that method in each location#439
serious6 merged 1 commit intoOfficeDev:masterfrom
candrews:closeQuietly

Conversation

@candrews
Copy link
Copy Markdown
Contributor

@candrews candrews commented Sep 8, 2015

Add a dependency on commons-io and use it appropriately.

Unfortunately, javax.xml.stream.XMLStreamWriter doesn't implement java.io.Closeable, so IOUtils.closeQuietly couldn't be used in a few places.

…h location

Add a dependency on commons-io and use it appropriately.
@azurecla
Copy link
Copy Markdown

azurecla commented Sep 8, 2015

Hi @candrews, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
We will now validate the agreement and then real humans will evaluate your PR.

TTYL, AZPRBOT;

serious6 pushed a commit that referenced this pull request Nov 8, 2015
Use IOUtils.closeQuietly instead of reimplementing that method in each location
@serious6 serious6 merged commit 0109670 into OfficeDev:master Nov 8, 2015
@serious6
Copy link
Copy Markdown
Member

serious6 commented Nov 8, 2015

@candrews merged. Thanks for your contribution.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants