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

Don't use Hashtable #128

Open
willardgibbs opened this issue Apr 18, 2019 · 0 comments
Open

Don't use Hashtable #128

willardgibbs opened this issue Apr 18, 2019 · 0 comments
Labels
code Concerns code discussion point The issue contains a request or an idea, that has to be discussed first enhancement New feature or request

Comments

@willardgibbs
Copy link
Contributor

willardgibbs commented Apr 18, 2019

In code we use Hashtable, because MQQueueManager constructor have it in signature.
But in [a64f23d] I made the following feature:

public void establish(String qmNqme, Map<String, Object> connectionProperties) throws MQException {
        queueManager = new MQQueueManager(qmNqme, new Hashtable<>(connectionProperties));
}

Now we can replaсe all Hashtable with HashMap except this method (SonarLint treats it well).

@willardgibbs willardgibbs added enhancement New feature or request code Concerns code discussion point The issue contains a request or an idea, that has to be discussed first labels Apr 18, 2019
@willardgibbs willardgibbs added this to the Release 1.1.0. milestone Apr 18, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
code Concerns code discussion point The issue contains a request or an idea, that has to be discussed first enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant