Author: sebb Date: Fri Mar 11 19:23:04 2011 New Revision: 1080731 URL: http://svn.apache.org/viewvc?rev=1080731&view=rev Log: Avoid possible NPE (Findbugs)
Modified: commons/proper/net/trunk/src/main/java/examples/mail/SMTPMail.java Modified: commons/proper/net/trunk/src/main/java/examples/mail/SMTPMail.java URL: http://svn.apache.org/viewvc/commons/proper/net/trunk/src/main/java/examples/mail/SMTPMail.java?rev=1080731&r1=1080730&r2=1080731&view=diff ============================================================================== --- commons/proper/net/trunk/src/main/java/examples/mail/SMTPMail.java (original) +++ commons/proper/net/trunk/src/main/java/examples/mail/SMTPMail.java Fri Mar 11 19:23:04 2011 @@ -88,14 +88,13 @@ public final class SMTPMail System.out.print("CC <enter one address per line, hit enter to end>: "); System.out.flush(); - // Of course you don't want to do this because readLine() may be null - cc = stdin.readLine().trim(); + cc = stdin.readLine(); - if (cc.length() == 0) + if (cc== null || cc.length() == 0) break; - header.addCC(cc); - ccList.add(cc); + header.addCC(cc.trim()); + ccList.add(cc.trim()); } System.out.print("Filename: ");