Comment 1 Dirk Baeumer 2002-11-25 13:51:36 EST

GetResource maps to getUnderlyingResource which has a different semantic then 
getCorrespondingResource.

Adam, why do you suggest to change this.

Comment 2 Adam Kiezun 2002-11-26 04:51:02 EST

let me investigate further a bit to give good reasons or close as 'invalid'

Comment 3 Martin Aeschlimann 2002-11-28 03:45:07 EST

The reason why to change to 'getResource' is that getUnderlyingResource does 
a 'exists()' test which is not needed in this context. (See in the code: the 
not exist exception is caught and getResource is returned)

Comment 4 Erich Gamma 2002-12-04 08:12:59 EST

but the semantic difference still existing.
In the case of a Java element from a JAR you would get the JAR file, currently 
you get null.

This change is subtle and unless we hare real performance data we should not 
change this.

Comment 5 Adam Kiezun 2002-12-09 08:56:42 EST

no proof that this is a performance problem
if we find one we will ask jcore for a fster implementation of 
getCorrespondingResource