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:43:16 UTC

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

     [ https://issues.apache.org/jira/browse/RAT-139?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Sebb resolved RAT-139.
----------------------

    Resolution: Fixed

URL: http://svn.apache.org/r1484129
Log:
RAT-139 FullTextMatchingLicense.prune uses inefficient deleteAtChar
Moved prune to BaseLicense class

Modified:
    creadur/rat/trunk/apache-rat-core/src/main/java/org/apache/rat/analysis/license/AppliedApacheSoftwareLicense20.java
    creadur/rat/trunk/apache-rat-core/src/main/java/org/apache/rat/analysis/license/BaseLicense.java
    creadur/rat/trunk/apache-rat-core/src/main/java/org/apache/rat/analysis/license/FullTextMatchingLicense.java

                
> FullTextMatchingLicense.prune uses inefficient deleteAtChar
> -----------------------------------------------------------
>
>                 Key: RAT-139
>                 URL: https://issues.apache.org/jira/browse/RAT-139
>             Project: Apache Rat
>          Issue Type: Improvement
>    Affects Versions: 0.9
>            Reporter: Sebb
>             Fix For: 0.10
>
>
> 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