-
Notifications
You must be signed in to change notification settings - Fork 823
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
Cleanup HashMap raw type warnings.. #7062
base: master
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- this seems to go across the project and therefore needs the
ci:all-tests
label - adding the diamond to the right side without adding generics to the type (left side) doesn't fix anything
@Override
should not be removed
platform/openide.filesystems/src/org/openide/filesystems/DefaultAttributes.java
Outdated
Show resolved
Hide resolved
.../groovy.support/src/org/netbeans/modules/groovy/support/actions/singlefilerun/JPDAStart.java
Outdated
Show resolved
Hide resolved
java/ant.debugger/src/org/netbeans/modules/ant/debugger/AntDebugger.java
Outdated
Show resolved
Hide resolved
java/java.file.launcher/src/org/netbeans/modules/java/file/launcher/actions/JPDAStart.java
Outdated
Show resolved
Hide resolved
java/java.openjdk.project/src/org/netbeans/modules/java/openjdk/jtreg/JPDAStart.java
Outdated
Show resolved
Hide resolved
...mon/extbrowser.chrome/src/org/netbeans/modules/extbrowser/plugins/ExternalBrowserPlugin.java
Outdated
Show resolved
Hide resolved
...mon/extbrowser.chrome/src/org/netbeans/modules/extbrowser/plugins/ExternalBrowserPlugin.java
Outdated
Show resolved
Hide resolved
...mon/extbrowser.chrome/src/org/netbeans/modules/extbrowser/plugins/ExternalBrowserPlugin.java
Outdated
Show resolved
Hide resolved
This change cleans up the majority of warnings related to HashMap. [repeat] /home/bwalker/src/netbeans/ide/xml.core/src/org/netbeans/modules/xml/dtd/grammar/DTDParser.java:130: warning: [rawtypes] found raw type: HashMap [repeat] attrs = new HashMap(); [repeat] ^ [repeat] missing type arguments for generic class HashMap<K,V> [repeat] where K,V are type-variables: [repeat] K extends Object declared in class HashMap [repeat] V extends Object declared in class HashMap There are a few still remaining. Those are in generated code, so not currently possible to easily cleanup.
48f0164
to
03f6bce
Compare
Hey @mbien , thanks for the help. I've made all the corrections you requested. One other thing, I'm still confused about the test failure. Doesn't seem like it related to my changes. I really don't like it when tests fails, for any reason. |
@BradWalker sorry for the delay, for some reason I forgot to come back to this PR. Restarted tests. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this interface is in public API I think.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this one too
@Override | ||
public void internalStartElement(String elemName, HashMap mapMandatory, HashMap mapAllowed) | ||
public void internalStartElement(String elemName, Map mapMandatory, Map mapAllowed) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
override needs to come back
@Override | ||
public void internalStartElement(String elemName, HashMap mapMandatory, HashMap mapAllowed) | ||
public void internalStartElement(String elemName, Map mapMandatory, Map mapAllowed) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
override removed
This change cleans up the majority of warnings related to HashMap.
There are a few still remaining. Those are in generated code, so not currently possible to easily cleanup.