Github user jaikiran commented on a diff in the pull request:
    --- Diff: src/main/org/apache/tools/ant/listener/ ---
    @@ -102,7 +102,7 @@
         private static final String DEFAULT_MIME_TYPE = "text/plain";
         /** Buffer in which the message is constructed prior to sending */
    -    private StringBuffer buffer = new StringBuffer();
    +    private StringBuilder buffer = new StringBuilder();
    --- End diff --
    I'm not too sure this change here, in this class is correct. The 
`StringBuffer` is a thread-safe class whereas the `StringBuilder` isn't. Having 
said that I haven't checked yet whether MailLogger class is expected to be 
thread safe nor have I checked its usage thoroughly.


To unsubscribe, e-mail:
For additional commands, e-mail:

Reply via email to