Am 15. März 2012 12:59 schrieb sebb <seb...@gmail.com>: > On 15 March 2012 11:26, <simonetrip...@apache.org> wrote: >> Author: simonetripodi >> Date: Thu Mar 15 11:26:25 2012 >> New Revision: 1300925 >> >> URL: http://svn.apache.org/viewvc?rev=1300925&view=rev >> Log: >> just incrementally built the hashcode in the constructor >> > > I wondered why you did not use the java.lang.String approach, but I > see the class is used in a HashMap so the hashCode will always be > needed. > Might be an idea to note this in the ctor? >
AccessibleObjectDescriptor is a nested private class of the package private AccessibleObjectRegistry. So users will never create AccessibleObjectDescriptors. The class comment says "Represents the key to looking up an AccessibleObject by reflection." Would be a bit redundant to add yet another comment to the constructor. > --------------------------------------------------------------------- > To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org > For additional commands, e-mail: dev-h...@commons.apache.org > --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org For additional commands, e-mail: dev-h...@commons.apache.org