matthiasblaesing commented on code in PR #6118:
URL: https://github.com/apache/netbeans/pull/6118#discussion_r1242642203
##########
enterprise/web.core/src/org/netbeans/modules/web/jspcompiler/SmapResolver.java:
##########
@@ -318,10 +319,10 @@ public boolean isResolved() {
/**
* get all the filenames in the SMAP
*/
- public Map getFileNames() {
- Hashtable h = new Hashtable(fsection.size());
- Collection c = fsection.values();
- Iterator i = c.iterator();
+ public Map<Integer, String> getFileNames() {
+ Map<Integer, String> h = new HashMap<>(fsection.size());
Review Comment:
Verified. The class is not exported API (and this modules declares one). So
users of this class would need to have an implementation anyway. With that in
mind, I see no problem keeping the methdo as you redesigned it.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
For further information about the NetBeans mailing lists, visit:
https://cwiki.apache.org/confluence/display/NETBEANS/Mailing+lists
