asbachb commented on code in PR #6118:
URL: https://github.com/apache/netbeans/pull/6118#discussion_r1241495154
##########
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:
I tried to double check, but I could not find a caller of that method.
The only usage of `SmapResolver` I could find is in `JspJavacAntLogger`.
For testing purposes I removed the method `getFileNames` and did a `ant
-Dcluster.config=enterprise build-nozip`.
Do you think this is enough evidence to remove the method entirely?
--
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
