Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refine FilterOutputStreamSlowMultibyteWrite findings #2031

Merged
merged 2 commits into from
Jan 11, 2022

Conversation

schlosna
Copy link
Contributor

Before this PR

FilterOutputStreamSlowMultibyteWrite would mistakenly warn on intermediate types that override both single and multibyte write methods.

After this PR

==COMMIT_MSG==
Refine FilterOutputStreamSlowMultibyteWrite findings

Avoid warning FilterOutputStreamSlowMultibyteWrite when intermediate
type overrides both single and multibyte write methods.
==COMMIT_MSG==

Possible downsides?

Avoid warning FilterOutputStreamSlowMultibyteWrite when intermediate
type overrides both single and multibyte write methods.
@changelog-app
Copy link

changelog-app bot commented Jan 11, 2022

Generate changelog in changelog/@unreleased

Type

  • Feature
  • Improvement
  • Fix
  • Break
  • Deprecation
  • Manual task
  • Migration

Description

Refine FilterOutputStreamSlowMultibyteWrite findings

Avoid warning FilterOutputStreamSlowMultibyteWrite when intermediate
type overrides both single and multibyte write methods.

Check the box to generate changelog(s)

  • Generate changelog entry

@policy-bot policy-bot bot requested a review from robert3005 January 11, 2022 00:07
@bulldozer-bot bulldozer-bot bot merged commit fe299f3 into develop Jan 11, 2022
@bulldozer-bot bulldozer-bot bot deleted the ds/FilterOutputStreamSlowMultibyteWrite branch January 11, 2022 00:20
bulldozer-bot bot pushed a commit to palantir/witchcraft-api that referenced this pull request Jan 12, 2022
###### _excavator_ is a bot for automating changes across repositories.

Changes produced by the roomba/latest-baseline-oss check.

# Release Notes
## 4.57.0
| Type | Description | Link |
| ---- | ----------- | ---- |
| Improvement | Upon missing jupiter on classpath, suggest putting it on runtime rather than implementation to avoid making checkUnusedDependenciesTest fail in return. | palantir/gradle-baseline#2021 |
| Fix | Refine FilterOutputStreamSlowMultibyteWrite findings<br><br>Avoid warning FilterOutputStreamSlowMultibyteWrite when intermediate<br>type overrides both single and multibyte write methods. | palantir/gradle-baseline#2031 |
| Improvement | Ban AfterburnerModule to reduce JDK-17 risk | palantir/gradle-baseline#2032 |



To enable or disable this check, please contact the maintainers of Excavator.
This was referenced Jan 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants