mirrors/cdt
1
0
Fork 0
mirror of https://github.com/eclipse-cdt/cdt synced 2025-07-24 09:25:31 +02:00
Code Issues Wiki Activity

[182454] improve getAbsoluteName() documentation

Browse source
This commit is contained in:
Martin Oberhuber 2007-04-25 11:49:36 +00:00
parent 7fc7c243b4
commit acd39c3849
1 changed files with 15 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

21
rse/plugins/org.eclipse.rse.ui/UI/org/eclipse/rse/internal/ui/view/SystemView.java
View file

@ -4314,8 +4314,10 @@ public class SystemView extends SafeTreeViewer
{
if(a==b) return true;
if(a==null || b==null) return false;
//TODO not sure if this equals() check is a good idea
//It may be expensive and unnecessary
//TODO not sure if this equals() check is a good idea.
//It may be expensive and unnecessary. It might be better
//to do this as a fallback instead, in case the adapter
//is not found.
if(a.equals(b)) return true;
if( (a instanceof IAdaptable) && (b instanceof IAdaptable) ) {
@ -4351,13 +4353,20 @@ public class SystemView extends SafeTreeViewer
if(ident!=null) {
String absName = ident.getAbsoluteName(element);
if(absName!=null) return absName.hashCode();
return ident.hashCode();
//Since one adapter is typically used for many elements in RSE,
//performance is better if the original Element's hashCode
//is used rather than the adapter's hashCode. The problem with
//this is, that if the remote object changes, it cannot be
//identified any more. But by returning null for the absolute
//name, the extender has agreed that the remote object's
//equals() and hashCode() methods will always work properly.
//return ident.hashCode();
}
}
if (element != null) // adding check because I hit a null exception here once at startup
if (element != null) { // adding check because I hit a null exception here once at startup
return element.hashCode();
else
{
} else {
//System.out.println("null element");
return 0;
}