Yeah, generally I think so, and maybe make a note of it somewhere prominent for new contributors.

On Nov 21, 2006, at 10:19 PM, Scott Gray wrote:

Si Chen (JIRA) wrote:
2. With contactListCommStatusRecord.getString("statusId").equals ("COM_COMPLETE"), it is better to write as "COM_COMPLETE".equals (...getString('statusId")) in case there are any null values. Otherwise sooner or later you'd get an NPE

Hi Si

I did a quick search for .equals(" and there are 812 occurrences in the code, is this something we need to tidy up at some point?

Regards
Scott

Best Regards,

Si
[EMAIL PROTECTED]



Reply via email to