[ 
https://issues.apache.org/jira/browse/IO-191?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12666136#action_12666136
 ] 

Jukka Zitting commented on IO-191:
----------------------------------

Replying to Peter.

{quote}
I don't imagine multiple patches are easier to apply.
{quote}

As noted above, we probably don't want to apply all the changes in the patch. 
Having smaller patches would make it easier to selectively apply the changes 
you propose.

{quote}
> Changing single character string literals to character literals in string 
> concatenations.
This is a fair comment. If you feel its worth reviewing on a case by case basis 
I am happy to do this.
{quote}

I don't think it's worth the effort, but perhaps I'm missing some really 
convincing reason why we should do this.

{quote}
> why are some parts of the expression strings and other characters.
Is this a question you would like me to answer or you are just raising this as 
a hypothetical question someone might ask.
{quote}

Just a hypothetical question that a future developer that looks at the code 
might think about.


> Possible improvements using static analysis.
> --------------------------------------------
>
>                 Key: IO-191
>                 URL: https://issues.apache.org/jira/browse/IO-191
>             Project: Commons IO
>          Issue Type: Improvement
>            Reporter: Peter Lawrey
>            Priority: Trivial
>         Attachments: commons-io-static-analysis.patch
>
>   Original Estimate: 3h
>  Remaining Estimate: 3h
>


-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to