rombert commented on a change in pull request #43:
URL:
https://github.com/apache/sling-org-apache-sling-resourceresolver/pull/43#discussion_r605037206
##########
File path:
src/main/java/org/apache/sling/resourceresolver/impl/ResourceResolverImpl.java
##########
@@ -1117,4 +1142,68 @@ public Resource move(final String srcAbsPath, final
String destAbsPath) throws P
}
return rsrc;
}
+
+
+
+ // A very simple tupel implementation which can be used as key in any map
+ public class String3Tupel {
+
+ @Override
+ public int hashCode() {
+ final int prime = 31;
Review comment:
Well, the maintenance/review cost will be there, so you can ask Eclipse
to do the right thing with recent versions :-)
https://stackoverflow.com/a/51775296/112671
##########
File path:
src/main/java/org/apache/sling/resourceresolver/impl/ResourceResolverImpl.java
##########
@@ -1117,4 +1142,68 @@ public Resource move(final String srcAbsPath, final
String destAbsPath) throws P
}
return rsrc;
}
+
+
+
+ // A very simple tupel implementation which can be used as key in any map
+ public class String3Tupel {
+
+ @Override
+ public int hashCode() {
+ final int prime = 31;
+ int result = 1;
+ result = prime * result + getEnclosingInstance().hashCode();
Review comment:
I am still missing the point of this field, seems it's only used in
hashcode/equals - is it really necessary?
--
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.
For queries about this service, please contact Infrastructure at:
[email protected]
