creadur-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Chris A. Mattmann (JIRA)" <j...@apache.org>
Subject [jira] [Updated] (RAT-148) LicenseAddingReport#report has useless call to metaData.getData
Date Thu, 29 Aug 2013 04:30:54 GMT

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

Chris A. Mattmann updated RAT-148:
----------------------------------

    Attachment: RAT-148.Mattmann.082813.patch.txt
    
> LicenseAddingReport#report has useless call to metaData.getData
> ---------------------------------------------------------------
>
>                 Key: RAT-148
>                 URL: https://issues.apache.org/jira/browse/RAT-148
>             Project: Apache Rat
>          Issue Type: Bug
>          Components: reports
>            Reporter: Chris A. Mattmann
>             Fix For: 0.9
>
>         Attachments: RAT-148.Mattmann.082813.patch.txt
>
>
> While perusing the current trunk code, I noticed that LicenseAddingReport.java#report
has a useless call to metaData.getData:
> {code:java}
> public class LicenseAddingReport extends AbstractReport {
>     private final AbstractLicenceAppender appender;
>     public LicenseAddingReport(String pCopyrightMsg, boolean pForced) {
>         appender = pCopyrightMsg == null ? new ApacheV2LicenceAppender() : new ApacheV2LicenceAppender(pCopyrightMsg);
>         appender.setForce(pForced);
>     }
>     @Override
>     public void report(org.apache.rat.api.Document document) throws RatException {
>         final MetaData metaData = document.getMetaData();
>         final Datum licenseHeader = metaData.get(MetaData.RAT_URL_HEADER_CATEGORY);
>         if (licenseHeader == null
>                 ||  MetaData.RAT_LICENSE_FAMILY_CATEGORY_DATUM_UNKNOWN.getValue().equals(licenseHeader.getValue()))
{
>             final File file = new File(document.getName());
>             if (file.isFile()) {
>                 try {
>                     appender.append(file);
>                 } catch (IOException e) {
>                     throw new RatException(e.getMessage(), e);
>                 }
>             }
>         }
>         metaData.getData();
>     }
> }
> {code}
> Looks to me that metaData is a local final variable, and that the call to getData returns
a MetaData object, which in turn is never used.

--
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

Mime
View raw message