buildframework/helium/sf/java/antlint/src/com/nokia/helium/antlint/ant/taskdefs/AntLintTask.java
author wbernard
Fri, 13 Aug 2010 14:59:05 +0300
changeset 628 7c4a911dc066
parent 588 c7c26511138f
child 645 b8d81fa19e7d
permissions -rw-r--r--
helium_11.0.0-e00f171ca185
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
587
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
     1
/*
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
     2
 * Copyright (c) 2007-2008 Nokia Corporation and/or its subsidiary(-ies).
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
     3
 * All rights reserved.
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
     4
 * This component and the accompanying materials are made available
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
     5
 * under the terms of the License "Eclipse Public License v1.0"
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
     6
 * which accompanies this distribution, and is available
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
     7
 * at the URL "http://www.eclipse.org/legal/epl-v10.html".
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
     8
 *
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
     9
 * Initial Contributors:
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
    10
 * Nokia Corporation - initial contribution.
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
    11
 *
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
    12
 * Contributors:
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
    13
 *
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
    14
 * Description: 
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
    15
 *
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
    16
 */
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
    17
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
    18
package com.nokia.helium.antlint.ant.taskdefs;
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
    19
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
    20
import java.io.File;
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
    21
import java.util.ArrayList;
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
    22
import java.util.List;
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
    23
import java.util.Vector;
628
7c4a911dc066 helium_11.0.0-e00f171ca185
wbernard
parents: 588
diff changeset
    24
587
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
    25
import org.apache.tools.ant.BuildException;
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
    26
import org.apache.tools.ant.DirectoryScanner;
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
    27
import org.apache.tools.ant.Task;
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
    28
import org.apache.tools.ant.types.FileSet;
628
7c4a911dc066 helium_11.0.0-e00f171ca185
wbernard
parents: 588
diff changeset
    29
588
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
    30
import com.nokia.helium.antlint.ant.AntlintException;
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
    31
import com.nokia.helium.antlint.ant.Reporter;
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
    32
import com.nokia.helium.antlint.ant.Severity;
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
    33
import com.nokia.helium.antlint.ant.types.Check;
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
    34
import com.nokia.helium.antlint.ant.types.ConsoleReporter;
587
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
    35
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
    36
/**
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
    37
 * AntLint Task. This task checks for common coding conventions and errors in
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
    38
 * Ant XML script files.
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
    39
 * 
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
    40
 * <p>
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
    41
 * The current checks include:
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
    42
 * <ul>
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
    43
 * <li>CheckAntCall : checks whether antcall is used with no param elements and
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
    44
 * calls target with no dependencies</li>
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
    45
 * <li>CheckDescription : checks for project description</li>
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
    46
 * <li>CheckDuplicateNames : checks for duplicate macros</li>
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
    47
 * <li>CheckFileName : checks the naming convention of ant xml files</li>
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
    48
 * <li>CheckIndentation : checks indentation</li>
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
    49
 * <li>CheckJepJythonScript : checks the coding convention in Jep and Jython
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
    50
 * scripts</li>
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
    51
 * <li>CheckPresetDefMacroDefName : checks the naming convention of presetdef
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
    52
 * and macrodef</li>
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
    53
 * <li>CheckProjectName : checks the naming convention of project</li>
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
    54
 * <li>CheckPropertyName : checks the naming convention of properties</li>
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
    55
 * </li>
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
    56
 * <li>CheckPythonTasks : checks the coding convention of python tasks</li>
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
    57
 * <li>CheckRunTarget : checks whether runtarget calls a target that has
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
    58
 * dependencies</li>
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
    59
 * <li>CheckScriptCondition : checks the coding convention in script condition</li>
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
    60
 * <li>CheckScriptDef : checks the coding convention in scriptdef</li>
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
    61
 * <li>CheckScriptDefNameAttributes - checks the naming convention of scriptdef
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
    62
 * name attributes</li></li>
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
    63
 * <li>CheckScriptDefStyle : checks the coding style of scriptdef</li>
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
    64
 * <li>CheckScriptSize : checks the size of scripts</li>
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
    65
 * <li>CheckTabCharacter : checks for tab characters</li>
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
    66
 * <li>CheckTargetName : checks the naming convention of targets</li>
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
    67
 * <li>CheckUseOfEqualsTask : checks the usage of equals task</li>
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
    68
 * <li>CheckUseOfIfInTargets : checks the usage of if task inside targets</li>
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
    69
 * </ul>
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
    70
 * </pre>
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
    71
 * 
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
    72
 * <p>
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
    73
 * Checks to be added:
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
    74
 * <ul>
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
    75
 * <li>Help target is defined.</li>
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
    76
 * <li>Optional to thrown warnings about deprecated targets (rename, copydir,
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
    77
 * copyfile).</li>
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
    78
 * </ul>
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
    79
 * </p>
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
    80
 * 
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
    81
 * <pre>
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
    82
 * Usage:
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
    83
 * 
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
    84
 *  &lt;antlint&gt;
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
    85
 *       &lt;fileset id=&quot;antlint.files&quot; dir=&quot;${antlint.test.dir}/data&quot;&gt;
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
    86
 *               &lt;include name=&quot;*.ant.xml&quot;/&gt;
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
    87
 *               &lt;include name=&quot;*build.xml&quot;/&gt;
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
    88
 *               &lt;include name=&quot;*.antlib.xml&quot;/&gt;
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
    89
 *       &lt;/fileset&gt;
588
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
    90
 *       &lt;CheckTabCharacter&quot; severity=&quot;error&quot; enabled=&quot;true&quot;/&gt;
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
    91
 *       &lt;CheckTargetName&quot; severity=&quot;warning&amp;quot enabled=&quot;true&quot; regexp=&quot;([a-z0-9[\\d\\-]]*)&quot;/&gt;
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
    92
 *       &lt;CheckScriptDef&quot; severity=&quot;error&quot; enabled=&quot;true&quot; outputDir=&quot;${antlint.test.dir}/output&quot;/&gt;
587
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
    93
 *  &lt;/antlint&gt;
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
    94
 * </pre>
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
    95
 * 
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
    96
 * @ant.task name="antlint" category="AntLint"
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
    97
 * 
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
    98
 */
588
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
    99
public class AntLintTask extends Task implements Reporter {
587
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
   100
588
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   101
    private List<Check> checkerList = new Vector<Check>();
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   102
    private List<FileSet> antFileSetList = new ArrayList<FileSet>();
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   103
    private List<Reporter> reporters = new ArrayList<Reporter>();
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   104
    private int errorCount;
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   105
    private boolean failOnError = true;
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   106
    private ConsoleReporter consoleReporter = new ConsoleReporter();
587
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
   107
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
   108
    /**
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
   109
     * Add a set of files to copy.
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
   110
     * 
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
   111
     * @param set
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
   112
     *            a set of files to AntLintTask.
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
   113
     * @ant.required
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
   114
     */
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
   115
    public void addFileset(FileSet set) {
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
   116
        antFileSetList.add(set);
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
   117
    }
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
   118
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
   119
    /**
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
   120
     * Execute the antlint task.
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
   121
     */
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
   122
    public final void execute() {
588
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   123
        if (checkerList.size() == 0) {
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   124
            throw new BuildException("No antlint checks are defined.");
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   125
        }
587
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
   126
        try {
588
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   127
            // Adding console reported by default if no
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   128
            // other reporter are mentioned.
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   129
            if (reporters.size() == 0) {
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   130
                reporters.add(consoleReporter);
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   131
            }
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   132
            setTask(this);
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   133
            open();
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   134
            doAntLintCheck();
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   135
        } catch (AntlintException e) {
587
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
   136
            throw new BuildException(
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
   137
                    "Exception occured while running AntLint task "
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
   138
                            + e.getMessage());
588
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   139
        } finally {
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   140
            // Closing all reporter session.
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   141
            close();
587
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
   142
        }
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
   143
588
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   144
        if (failOnError && (errorCount > 0)) {
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   145
            throw new BuildException("Build failed because of AntLint errors.");
587
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
   146
        }
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
   147
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
   148
    }
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
   149
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
   150
    /**
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
   151
     * Triggers the antlint checking.
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
   152
     * 
588
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   153
     * @throws AntlintException
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   154
     * 
587
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
   155
     * @throws Exception
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
   156
     *             if the checking fails.
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
   157
     */
588
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   158
    private void doAntLintCheck() throws AntlintException {
587
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
   159
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
   160
        for (FileSet fs : antFileSetList) {
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
   161
            DirectoryScanner ds = fs.getDirectoryScanner(getProject());
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
   162
            String[] srcFiles = ds.getIncludedFiles();
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
   163
            String basedir = ds.getBasedir().getPath();
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
   164
            for (int i = 0; i < srcFiles.length; i++) {
588
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   165
                String antFilename = basedir + File.separator + srcFiles[i];
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   166
                runChecks(new File(antFilename));
587
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
   167
            }
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
   168
        }
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
   169
    }
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
   170
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
   171
    /**
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
   172
     * Runs antlint checks for the given ant file.
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
   173
     * 
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
   174
     * @param antFileName
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
   175
     *            is the name of the ant file to be checked.
588
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   176
     * @throws AntlintException
587
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
   177
     */
588
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   178
    private void runChecks(File antFilename) throws AntlintException {
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   179
        for (Check check : checkerList) {
587
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
   180
            if (check.isEnabled()) {
588
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   181
                check.validateAttributes();
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   182
                check.setReporter(this);
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   183
                check.run(antFilename);
587
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
   184
            }
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
   185
        }
588
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   186
587
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
   187
    }
588
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   188
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   189
    /**
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   190
     * To add Antlint checkers.
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   191
     * 
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   192
     * @param c
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   193
     */
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   194
    public void add(Check c) {
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   195
        checkerList.add(c);
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   196
    }
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   197
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   198
    /**
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   199
     * To add reporters.
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   200
     * 
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   201
     * @param reporter
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   202
     */
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   203
    public void add(Reporter reporter) {
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   204
        reporter.setTask(this);
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   205
        reporters.add(reporter);
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   206
    }
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   207
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   208
    /**
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   209
     * @param failOnError
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   210
     *            the failOnError to set
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   211
     */
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   212
    public void setFailOnError(boolean failOnError) {
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   213
        this.failOnError = failOnError;
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   214
    }
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   215
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   216
    /*
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   217
     * (non-Javadoc)
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   218
     * 
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   219
     * @see
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   220
     * com.nokia.helium.antlint.ant.Reporter#report(com.nokia.helium.antlint
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   221
     * .ant.Severity, java.lang.String, java.io.File, int)
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   222
     */
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   223
    public void report(Severity severity, String message, File filename,
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   224
            int lineNo) {
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   225
        if (severity.getValue().toUpperCase().equals("ERROR")) {
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   226
            errorCount++;
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   227
        }
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   228
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   229
        for (Reporter reporter : reporters) {
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   230
            reporter.report(severity, message, filename, lineNo);
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   231
        }
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   232
    }
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   233
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   234
    /*
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   235
     * (non-Javadoc)
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   236
     * 
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   237
     * @see
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   238
     * com.nokia.helium.antlint.ant.Reporter#setTask(org.apache.tools.ant.Task)
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   239
     */
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   240
    @Override
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   241
    public void setTask(Task task) {
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   242
        for (Reporter reporter : reporters) {
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   243
            reporter.setTask(task);
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   244
        }
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   245
    }
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   246
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   247
    @Override
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   248
    public void close() {
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   249
        for (Reporter reporter : reporters) {
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   250
            reporter.close();
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   251
        }
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   252
    }
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   253
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   254
    @Override
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   255
    public void open() {
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   256
        for (Reporter reporter : reporters) {
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   257
            reporter.open();
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   258
        }
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   259
    }
c7c26511138f helium-10.0.0-bc45d50958fe
wbernard
parents: 587
diff changeset
   260
587
85df38eb4012 helium_9.0-a7879c935424
wbernard
parents:
diff changeset
   261
}