Skip to content
This repository has been archived by the owner on Jun 8, 2020. It is now read-only.

Remove compiler warnings #541

Open
wants to merge 5 commits into
base: develop
Choose a base branch
from

Conversation

TSavo
Copy link
Contributor

@TSavo TSavo commented Mar 4, 2020

Suppresses the warnings on a few deprecated APIs and an unchecked assignment.

@@ -28,7 +28,7 @@ private static void rateLimit() {
LOG.warn("Bitfinex connection throttle control has been interrupted");
}
}

@SuppressWarnings("deprecation")
Copy link
Collaborator

Choose a reason for hiding this comment

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

Can we not remove the use of the untrusted API?

Not sure suppressing the warning is the right thing to do here - a warning is warranted.

@badgerwithagun badgerwithagun added the awaiting_fixes PR is awaiting submitter to respond to a review label Mar 14, 2020
@badgerwithagun
Copy link
Collaborator

This project is in the process of being merged into the XChange project and no further PRs will be merged here. Once the projects have been merged, there may be a short stabilization period where there will be large-scale renaming of classes and packages, which may cause conflicts. You are advised to wait at least a week from now and then resubmit your PR on the XChange project. Thank you for your support!

@badgerwithagun
Copy link
Collaborator

You can now resubmit your PR on XChange. This project will shortly be marked as archived.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
awaiting_fixes PR is awaiting submitter to respond to a review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants