You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@creadur.apache.org by "Sebb (JIRA)" <ji...@apache.org> on 2013/05/18 17:39:16 UTC

[jira] [Commented] (RAT-139) FullTextMatchingLicense.prune uses inefficient deleteAtChar

    [ https://issues.apache.org/jira/browse/RAT-139?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13661375#comment-13661375 ] 

Sebb commented on RAT-139:
--------------------------

Likewise for AppliedApacheSoftwareLicense20#prune
                
> FullTextMatchingLicense.prune uses inefficient deleteAtChar
> -----------------------------------------------------------
>
>                 Key: RAT-139
>                 URL: https://issues.apache.org/jira/browse/RAT-139
>             Project: Apache Rat
>          Issue Type: Improvement
>            Reporter: Sebb
>
> FullTextMatchingLicense.prune is quite inefficient.
> It first copies the entire string to a StringBuilder, then scans the buffer deleting characters that are not letters or digits.
> It should be a lot quicker to just copy the letters and digits once, skipping the rest.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira