[GitHub] [ant] bodewig closed pull request #58: Use StringBuilder instead of StringBuffer as it offers high performan…

2021-09-25 Thread GitBox
bodewig closed pull request #58: URL: https://github.com/apache/ant/pull/58 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[GitHub] [ant] bodewig closed pull request #45: Consider returning a zero length array rather than null.

2021-09-25 Thread GitBox
bodewig closed pull request #45: URL: https://github.com/apache/ant/pull/45 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[GitHub] [ant] bodewig merged pull request #159: Use an empty array in favor of pre-sized array.

2021-09-25 Thread GitBox
bodewig merged pull request #159: URL: https://github.com/apache/ant/pull/159 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[GitHub] [ant] bodewig commented on a change in pull request #157: Performance:

2021-09-25 Thread GitBox
bodewig commented on a change in pull request #157: URL: https://github.com/apache/ant/pull/157#discussion_r716030810 ## File path: src/main/org/apache/tools/ant/taskdefs/compilers/JavacExternal.java ## @@ -94,9 +94,8 @@ private int moveArgFileEligibleOptionsToEnd(String[]

[GitHub] [ant] bodewig commented on pull request #157: Performance:

2021-09-25 Thread GitBox
bodewig commented on pull request #157: URL: https://github.com/apache/ant/pull/157#issuecomment-927108994 "performance" is such a big word, I very much doubt there will be any measurable effect on a real build by the changes :-) anyway, thanks for the patch @arturobernalg . Please

[GitHub] [ant] asfgit merged pull request #142: Tar-task: using should preserve symlinks

2021-09-25 Thread GitBox
asfgit merged pull request #142: URL: https://github.com/apache/ant/pull/142 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[GitHub] [ant] bodewig commented on pull request #142: Tar-task: using should preserve symlinks

2021-09-25 Thread GitBox
bodewig commented on pull request #142: URL: https://github.com/apache/ant/pull/142#issuecomment-927106646 many thanks @mstrap I'd like to add you to the contributors list (CONTRIBUTORS and contributors.xml), which name shall I use? -- This is an automated message from the

[GitHub] [ant] bodewig commented on a change in pull request #155: Simplify single assignment and avoid redundant if conditions.

2021-09-25 Thread GitBox
bodewig commented on a change in pull request #155: URL: https://github.com/apache/ant/pull/155#discussion_r716030510 ## File path: src/main/org/apache/tools/ant/taskdefs/XmlProperty.java ## @@ -404,17 +404,14 @@ public Object processNode(Node node, String prefix, Object

[GitHub] [ant] bodewig merged pull request #154: Change StringBuffer in favor of StringBuilder.

2021-09-25 Thread GitBox
bodewig merged pull request #154: URL: https://github.com/apache/ant/pull/154 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[GitHub] [ant] bodewig merged pull request #153: Delete unused import.

2021-09-25 Thread GitBox
bodewig merged pull request #153: URL: https://github.com/apache/ant/pull/153 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[GitHub] [ant] bodewig merged pull request #139: No boxing

2021-09-25 Thread GitBox
bodewig merged pull request #139: URL: https://github.com/apache/ant/pull/139 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

[GitHub] [ant] bodewig commented on a change in pull request #158: Replace assert by simpler but equivalent calls.

2021-09-25 Thread GitBox
bodewig commented on a change in pull request #158: URL: https://github.com/apache/ant/pull/158#discussion_r716031134 ## File path: src/tests/junit/org/apache/tools/ant/util/DeweyDecimalTest.java ## @@ -107,10 +107,10 @@ public void isGreaterThanOrEqual() { @Test