RE: Patch Review: unused local variables
"Conrad T. Pino" <[email protected]> Thu, 29 Nov 2007 10:48:10 -0800
| Newsgroups | gmane.comp.java.classpath.extensions.javamail |
|---|---|
| Message-ID | <[email protected]> |
Conrad T. Pino wrote: > > Patch goals: > > > > 1. Remove local variable is never read warnings, 32 in total. > > > > Note: IMAPFolder retains "s.getConnection()" method call > > to preserve state change side effects, if any. > > Yes, there are side effects. The patch seems fine. Committed.