Just do it. Although, I suspect that piece of code may be generated from the API WSDL
On 1/30/14 9:38 AM, "Daan Hoogland" <daan.hoogl...@gmail.com> wrote: >H Alex and others, > >I hesitate to ask you as it seems obvious, but in the awsapi project >in com.cloud.bridge.model.SObjectItem.equals(Object other) a >comparison is made > if >(!theObject.getId().equals(((SObjectItemVO)other).getTheObject())) >which i think should be > if >(!theObject.getId().equals(((SObjectItemVO)other).getTheObject().getId())) >The obvious question would be, 'am i correct?', or 'can i make the >change?'. > >Now my real question is should I just go ahead and fix the scariest >bugs that findbugs comes up with or should I consult with package >maintainers for each part of the system on errors like these? > >(to comfort all of you, i restricted myself to doing this in master) > >regards, >Daan