You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by de...@apache.org on 2014/04/14 22:57:32 UTC
svn commit: r906189 [14/15] - in
/websites/production/maven/content/plugins-archives/maven-checkstyle-plugin-LATEST:
./ apidocs/ apidocs/org/apache/maven/plugin/checkstyle/
apidocs/org/apache/maven/plugin/checkstyle/class-use/
apidocs/org/apache/maven/...
Modified: websites/production/maven/content/plugins-archives/maven-checkstyle-plugin-LATEST/xref/org/apache/maven/plugin/checkstyle/DefaultCheckstyleExecutor.html
==============================================================================
--- websites/production/maven/content/plugins-archives/maven-checkstyle-plugin-LATEST/xref/org/apache/maven/plugin/checkstyle/DefaultCheckstyleExecutor.html (original)
+++ websites/production/maven/content/plugins-archives/maven-checkstyle-plugin-LATEST/xref/org/apache/maven/plugin/checkstyle/DefaultCheckstyleExecutor.html Mon Apr 14 20:57:29 2014
@@ -41,692 +41,712 @@
<a class="jxr_linenumber" name="L33" href="#L33">33</a> <strong class="jxr_keyword">import</strong> java.util.Properties;
<a class="jxr_linenumber" name="L34" href="#L34">34</a>
<a class="jxr_linenumber" name="L35" href="#L35">35</a> <strong class="jxr_keyword">import</strong> org.apache.commons.io.IOUtils;
-<a class="jxr_linenumber" name="L36" href="#L36">36</a> <strong class="jxr_keyword">import</strong> org.apache.maven.artifact.DependencyResolutionRequiredException;
-<a class="jxr_linenumber" name="L37" href="#L37">37</a> <strong class="jxr_keyword">import</strong> org.apache.maven.model.Resource;
-<a class="jxr_linenumber" name="L38" href="#L38">38</a> <strong class="jxr_keyword">import</strong> org.apache.maven.project.MavenProject;
-<a class="jxr_linenumber" name="L39" href="#L39">39</a> <strong class="jxr_keyword">import</strong> org.codehaus.plexus.component.annotations.Component;
-<a class="jxr_linenumber" name="L40" href="#L40">40</a> <strong class="jxr_keyword">import</strong> org.codehaus.plexus.component.annotations.Requirement;
-<a class="jxr_linenumber" name="L41" href="#L41">41</a> <strong class="jxr_keyword">import</strong> org.codehaus.plexus.logging.AbstractLogEnabled;
-<a class="jxr_linenumber" name="L42" href="#L42">42</a> <strong class="jxr_keyword">import</strong> org.codehaus.plexus.resource.ResourceManager;
-<a class="jxr_linenumber" name="L43" href="#L43">43</a> <strong class="jxr_keyword">import</strong> org.codehaus.plexus.resource.loader.FileResourceCreationException;
-<a class="jxr_linenumber" name="L44" href="#L44">44</a> <strong class="jxr_keyword">import</strong> org.codehaus.plexus.resource.loader.FileResourceLoader;
-<a class="jxr_linenumber" name="L45" href="#L45">45</a> <strong class="jxr_keyword">import</strong> org.codehaus.plexus.resource.loader.ResourceNotFoundException;
-<a class="jxr_linenumber" name="L46" href="#L46">46</a> <strong class="jxr_keyword">import</strong> org.codehaus.plexus.util.FileUtils;
-<a class="jxr_linenumber" name="L47" href="#L47">47</a> <strong class="jxr_keyword">import</strong> org.codehaus.plexus.util.StringUtils;
-<a class="jxr_linenumber" name="L48" href="#L48">48</a>
-<a class="jxr_linenumber" name="L49" href="#L49">49</a> <strong class="jxr_keyword">import</strong> com.puppycrawl.tools.checkstyle.Checker;
-<a class="jxr_linenumber" name="L50" href="#L50">50</a> <strong class="jxr_keyword">import</strong> com.puppycrawl.tools.checkstyle.ConfigurationLoader;
-<a class="jxr_linenumber" name="L51" href="#L51">51</a> <strong class="jxr_keyword">import</strong> com.puppycrawl.tools.checkstyle.DefaultConfiguration;
-<a class="jxr_linenumber" name="L52" href="#L52">52</a> <strong class="jxr_keyword">import</strong> com.puppycrawl.tools.checkstyle.PackageNamesLoader;
-<a class="jxr_linenumber" name="L53" href="#L53">53</a> <strong class="jxr_keyword">import</strong> com.puppycrawl.tools.checkstyle.PropertiesExpander;
-<a class="jxr_linenumber" name="L54" href="#L54">54</a> <strong class="jxr_keyword">import</strong> com.puppycrawl.tools.checkstyle.api.AuditListener;
-<a class="jxr_linenumber" name="L55" href="#L55">55</a> <strong class="jxr_keyword">import</strong> com.puppycrawl.tools.checkstyle.api.CheckstyleException;
-<a class="jxr_linenumber" name="L56" href="#L56">56</a> <strong class="jxr_keyword">import</strong> com.puppycrawl.tools.checkstyle.api.Configuration;
-<a class="jxr_linenumber" name="L57" href="#L57">57</a> <strong class="jxr_keyword">import</strong> com.puppycrawl.tools.checkstyle.api.FilterSet;
-<a class="jxr_linenumber" name="L58" href="#L58">58</a> <strong class="jxr_keyword">import</strong> com.puppycrawl.tools.checkstyle.filters.SuppressionsLoader;
-<a class="jxr_linenumber" name="L59" href="#L59">59</a>
-<a class="jxr_linenumber" name="L60" href="#L60">60</a> <em class="jxr_javadoccomment">/**</em>
-<a class="jxr_linenumber" name="L61" href="#L61">61</a> <em class="jxr_javadoccomment"> * @author Olivier Lamy</em>
-<a class="jxr_linenumber" name="L62" href="#L62">62</a> <em class="jxr_javadoccomment"> * @since 2.5</em>
-<a class="jxr_linenumber" name="L63" href="#L63">63</a> <em class="jxr_javadoccomment"> * @version $Id: DefaultCheckstyleExecutor.java 1575615 2014-03-08 21:54:26Z dennisl $</em>
-<a class="jxr_linenumber" name="L64" href="#L64">64</a> <em class="jxr_javadoccomment"> */</em>
-<a class="jxr_linenumber" name="L65" href="#L65">65</a> @Component( role = CheckstyleExecutor.<strong class="jxr_keyword">class</strong>, hint = <span class="jxr_string">"default"</span>, instantiationStrategy = <span class="jxr_string">"per-lookup"</span> )
-<a class="jxr_linenumber" name="L66" href="#L66">66</a> <strong class="jxr_keyword">public</strong> <strong class="jxr_keyword">class</strong> <a href="../../../../../org/apache/maven/plugin/checkstyle/DefaultCheckstyleExecutor.html">DefaultCheckstyleExecutor</a>
-<a class="jxr_linenumber" name="L67" href="#L67">67</a> <strong class="jxr_keyword">extends</strong> AbstractLogEnabled
-<a class="jxr_linenumber" name="L68" href="#L68">68</a> <strong class="jxr_keyword">implements</strong> <a href="../../../../../org/apache/maven/plugin/checkstyle/CheckstyleExecutor.html">CheckstyleExecutor</a>
-<a class="jxr_linenumber" name="L69" href="#L69">69</a> {
-<a class="jxr_linenumber" name="L70" href="#L70">70</a> @Requirement( hint = <span class="jxr_string">"default"</span> )
-<a class="jxr_linenumber" name="L71" href="#L71">71</a> <strong class="jxr_keyword">private</strong> ResourceManager locator;
-<a class="jxr_linenumber" name="L72" href="#L72">72</a>
-<a class="jxr_linenumber" name="L73" href="#L73">73</a> @Requirement( hint = <span class="jxr_string">"license"</span> )
-<a class="jxr_linenumber" name="L74" href="#L74">74</a> <strong class="jxr_keyword">private</strong> ResourceManager licenseLocator;
-<a class="jxr_linenumber" name="L75" href="#L75">75</a>
-<a class="jxr_linenumber" name="L76" href="#L76">76</a> <strong class="jxr_keyword">private</strong> <strong class="jxr_keyword">static</strong> <strong class="jxr_keyword">final</strong> File[] EMPTY_FILE_ARRAY = <strong class="jxr_keyword">new</strong> File[0];
-<a class="jxr_linenumber" name="L77" href="#L77">77</a>
-<a class="jxr_linenumber" name="L78" href="#L78">78</a> <strong class="jxr_keyword">public</strong> <a href="../../../../../org/apache/maven/plugin/checkstyle/CheckstyleResults.html">CheckstyleResults</a> executeCheckstyle( CheckstyleExecutorRequest request )
-<a class="jxr_linenumber" name="L79" href="#L79">79</a> <strong class="jxr_keyword">throws</strong> CheckstyleExecutorException, CheckstyleException
-<a class="jxr_linenumber" name="L80" href="#L80">80</a> {
-<a class="jxr_linenumber" name="L81" href="#L81">81</a> <em class="jxr_comment">// checkstyle will always use the context classloader in order</em>
-<a class="jxr_linenumber" name="L82" href="#L82">82</a> <em class="jxr_comment">// to load resources (dtds),</em>
-<a class="jxr_linenumber" name="L83" href="#L83">83</a> <em class="jxr_comment">// so we have to fix it</em>
-<a class="jxr_linenumber" name="L84" href="#L84">84</a> <em class="jxr_comment">// olamy this hack is not anymore needed in maven 3.x</em>
-<a class="jxr_linenumber" name="L85" href="#L85">85</a> ClassLoader checkstyleClassLoader = PackageNamesLoader.<strong class="jxr_keyword">class</strong>.getClassLoader();
-<a class="jxr_linenumber" name="L86" href="#L86">86</a> Thread.currentThread().setContextClassLoader( checkstyleClassLoader );
-<a class="jxr_linenumber" name="L87" href="#L87">87</a>
-<a class="jxr_linenumber" name="L88" href="#L88">88</a> <strong class="jxr_keyword">if</strong> ( getLogger().isDebugEnabled() )
-<a class="jxr_linenumber" name="L89" href="#L89">89</a> {
-<a class="jxr_linenumber" name="L90" href="#L90">90</a> getLogger().debug( <span class="jxr_string">"executeCheckstyle start headerLocation : "</span> + request.getHeaderLocation() );
-<a class="jxr_linenumber" name="L91" href="#L91">91</a> }
-<a class="jxr_linenumber" name="L92" href="#L92">92</a>
-<a class="jxr_linenumber" name="L93" href="#L93">93</a> MavenProject project = request.getProject();
-<a class="jxr_linenumber" name="L94" href="#L94">94</a>
-<a class="jxr_linenumber" name="L95" href="#L95">95</a> configureResourceLocator( locator, request );
-<a class="jxr_linenumber" name="L96" href="#L96">96</a>
-<a class="jxr_linenumber" name="L97" href="#L97">97</a> configureResourceLocator( licenseLocator, request );
-<a class="jxr_linenumber" name="L98" href="#L98">98</a>
-<a class="jxr_linenumber" name="L99" href="#L99">99</a> File[] files;
-<a class="jxr_linenumber" name="L100" href="#L100">100</a> <strong class="jxr_keyword">try</strong>
-<a class="jxr_linenumber" name="L101" href="#L101">101</a> {
-<a class="jxr_linenumber" name="L102" href="#L102">102</a> files = getFilesToProcess( request );
-<a class="jxr_linenumber" name="L103" href="#L103">103</a> }
-<a class="jxr_linenumber" name="L104" href="#L104">104</a> <strong class="jxr_keyword">catch</strong> ( IOException e )
+<a class="jxr_linenumber" name="L36" href="#L36">36</a> <strong class="jxr_keyword">import</strong> org.apache.maven.artifact.Artifact;
+<a class="jxr_linenumber" name="L37" href="#L37">37</a> <strong class="jxr_keyword">import</strong> org.apache.maven.artifact.DependencyResolutionRequiredException;
+<a class="jxr_linenumber" name="L38" href="#L38">38</a> <strong class="jxr_keyword">import</strong> org.apache.maven.model.Resource;
+<a class="jxr_linenumber" name="L39" href="#L39">39</a> <strong class="jxr_keyword">import</strong> org.apache.maven.project.MavenProject;
+<a class="jxr_linenumber" name="L40" href="#L40">40</a> <strong class="jxr_keyword">import</strong> org.codehaus.plexus.component.annotations.Component;
+<a class="jxr_linenumber" name="L41" href="#L41">41</a> <strong class="jxr_keyword">import</strong> org.codehaus.plexus.component.annotations.Requirement;
+<a class="jxr_linenumber" name="L42" href="#L42">42</a> <strong class="jxr_keyword">import</strong> org.codehaus.plexus.logging.AbstractLogEnabled;
+<a class="jxr_linenumber" name="L43" href="#L43">43</a> <strong class="jxr_keyword">import</strong> org.codehaus.plexus.resource.ResourceManager;
+<a class="jxr_linenumber" name="L44" href="#L44">44</a> <strong class="jxr_keyword">import</strong> org.codehaus.plexus.resource.loader.FileResourceCreationException;
+<a class="jxr_linenumber" name="L45" href="#L45">45</a> <strong class="jxr_keyword">import</strong> org.codehaus.plexus.resource.loader.FileResourceLoader;
+<a class="jxr_linenumber" name="L46" href="#L46">46</a> <strong class="jxr_keyword">import</strong> org.codehaus.plexus.resource.loader.ResourceNotFoundException;
+<a class="jxr_linenumber" name="L47" href="#L47">47</a> <strong class="jxr_keyword">import</strong> org.codehaus.plexus.util.FileUtils;
+<a class="jxr_linenumber" name="L48" href="#L48">48</a> <strong class="jxr_keyword">import</strong> org.codehaus.plexus.util.StringUtils;
+<a class="jxr_linenumber" name="L49" href="#L49">49</a>
+<a class="jxr_linenumber" name="L50" href="#L50">50</a> <strong class="jxr_keyword">import</strong> com.puppycrawl.tools.checkstyle.Checker;
+<a class="jxr_linenumber" name="L51" href="#L51">51</a> <strong class="jxr_keyword">import</strong> com.puppycrawl.tools.checkstyle.ConfigurationLoader;
+<a class="jxr_linenumber" name="L52" href="#L52">52</a> <strong class="jxr_keyword">import</strong> com.puppycrawl.tools.checkstyle.DefaultConfiguration;
+<a class="jxr_linenumber" name="L53" href="#L53">53</a> <strong class="jxr_keyword">import</strong> com.puppycrawl.tools.checkstyle.PackageNamesLoader;
+<a class="jxr_linenumber" name="L54" href="#L54">54</a> <strong class="jxr_keyword">import</strong> com.puppycrawl.tools.checkstyle.PropertiesExpander;
+<a class="jxr_linenumber" name="L55" href="#L55">55</a> <strong class="jxr_keyword">import</strong> com.puppycrawl.tools.checkstyle.api.AuditListener;
+<a class="jxr_linenumber" name="L56" href="#L56">56</a> <strong class="jxr_keyword">import</strong> com.puppycrawl.tools.checkstyle.api.CheckstyleException;
+<a class="jxr_linenumber" name="L57" href="#L57">57</a> <strong class="jxr_keyword">import</strong> com.puppycrawl.tools.checkstyle.api.Configuration;
+<a class="jxr_linenumber" name="L58" href="#L58">58</a> <strong class="jxr_keyword">import</strong> com.puppycrawl.tools.checkstyle.api.FilterSet;
+<a class="jxr_linenumber" name="L59" href="#L59">59</a> <strong class="jxr_keyword">import</strong> com.puppycrawl.tools.checkstyle.filters.SuppressionsLoader;
+<a class="jxr_linenumber" name="L60" href="#L60">60</a>
+<a class="jxr_linenumber" name="L61" href="#L61">61</a> <em class="jxr_javadoccomment">/**</em>
+<a class="jxr_linenumber" name="L62" href="#L62">62</a> <em class="jxr_javadoccomment"> * @author Olivier Lamy</em>
+<a class="jxr_linenumber" name="L63" href="#L63">63</a> <em class="jxr_javadoccomment"> * @since 2.5</em>
+<a class="jxr_linenumber" name="L64" href="#L64">64</a> <em class="jxr_javadoccomment"> * @version $Id: DefaultCheckstyleExecutor.java 1586844 2014-04-12 09:53:39Z rfscholte $</em>
+<a class="jxr_linenumber" name="L65" href="#L65">65</a> <em class="jxr_javadoccomment"> */</em>
+<a class="jxr_linenumber" name="L66" href="#L66">66</a> @Component( role = CheckstyleExecutor.<strong class="jxr_keyword">class</strong>, hint = <span class="jxr_string">"default"</span>, instantiationStrategy = <span class="jxr_string">"per-lookup"</span> )
+<a class="jxr_linenumber" name="L67" href="#L67">67</a> <strong class="jxr_keyword">public</strong> <strong class="jxr_keyword">class</strong> <a href="../../../../../org/apache/maven/plugin/checkstyle/DefaultCheckstyleExecutor.html">DefaultCheckstyleExecutor</a>
+<a class="jxr_linenumber" name="L68" href="#L68">68</a> <strong class="jxr_keyword">extends</strong> AbstractLogEnabled
+<a class="jxr_linenumber" name="L69" href="#L69">69</a> <strong class="jxr_keyword">implements</strong> <a href="../../../../../org/apache/maven/plugin/checkstyle/CheckstyleExecutor.html">CheckstyleExecutor</a>
+<a class="jxr_linenumber" name="L70" href="#L70">70</a> {
+<a class="jxr_linenumber" name="L71" href="#L71">71</a> @Requirement( hint = <span class="jxr_string">"default"</span> )
+<a class="jxr_linenumber" name="L72" href="#L72">72</a> <strong class="jxr_keyword">private</strong> ResourceManager locator;
+<a class="jxr_linenumber" name="L73" href="#L73">73</a>
+<a class="jxr_linenumber" name="L74" href="#L74">74</a> @Requirement( hint = <span class="jxr_string">"license"</span> )
+<a class="jxr_linenumber" name="L75" href="#L75">75</a> <strong class="jxr_keyword">private</strong> ResourceManager licenseLocator;
+<a class="jxr_linenumber" name="L76" href="#L76">76</a>
+<a class="jxr_linenumber" name="L77" href="#L77">77</a> <strong class="jxr_keyword">private</strong> <strong class="jxr_keyword">static</strong> <strong class="jxr_keyword">final</strong> File[] EMPTY_FILE_ARRAY = <strong class="jxr_keyword">new</strong> File[0];
+<a class="jxr_linenumber" name="L78" href="#L78">78</a>
+<a class="jxr_linenumber" name="L79" href="#L79">79</a> <strong class="jxr_keyword">public</strong> <a href="../../../../../org/apache/maven/plugin/checkstyle/CheckstyleResults.html">CheckstyleResults</a> executeCheckstyle( CheckstyleExecutorRequest request )
+<a class="jxr_linenumber" name="L80" href="#L80">80</a> <strong class="jxr_keyword">throws</strong> CheckstyleExecutorException, CheckstyleException
+<a class="jxr_linenumber" name="L81" href="#L81">81</a> {
+<a class="jxr_linenumber" name="L82" href="#L82">82</a> <em class="jxr_comment">// checkstyle will always use the context classloader in order</em>
+<a class="jxr_linenumber" name="L83" href="#L83">83</a> <em class="jxr_comment">// to load resources (dtds),</em>
+<a class="jxr_linenumber" name="L84" href="#L84">84</a> <em class="jxr_comment">// so we have to fix it</em>
+<a class="jxr_linenumber" name="L85" href="#L85">85</a> <em class="jxr_comment">// olamy this hack is not anymore needed in maven 3.x</em>
+<a class="jxr_linenumber" name="L86" href="#L86">86</a> ClassLoader checkstyleClassLoader = PackageNamesLoader.<strong class="jxr_keyword">class</strong>.getClassLoader();
+<a class="jxr_linenumber" name="L87" href="#L87">87</a> Thread.currentThread().setContextClassLoader( checkstyleClassLoader );
+<a class="jxr_linenumber" name="L88" href="#L88">88</a>
+<a class="jxr_linenumber" name="L89" href="#L89">89</a> <strong class="jxr_keyword">if</strong> ( getLogger().isDebugEnabled() )
+<a class="jxr_linenumber" name="L90" href="#L90">90</a> {
+<a class="jxr_linenumber" name="L91" href="#L91">91</a> getLogger().debug( <span class="jxr_string">"executeCheckstyle start headerLocation : "</span> + request.getHeaderLocation() );
+<a class="jxr_linenumber" name="L92" href="#L92">92</a> }
+<a class="jxr_linenumber" name="L93" href="#L93">93</a>
+<a class="jxr_linenumber" name="L94" href="#L94">94</a> MavenProject project = request.getProject();
+<a class="jxr_linenumber" name="L95" href="#L95">95</a>
+<a class="jxr_linenumber" name="L96" href="#L96">96</a> configureResourceLocator( locator, request, <strong class="jxr_keyword">null</strong> );
+<a class="jxr_linenumber" name="L97" href="#L97">97</a>
+<a class="jxr_linenumber" name="L98" href="#L98">98</a> configureResourceLocator( licenseLocator, request, request.getLicenseArtifacts() );
+<a class="jxr_linenumber" name="L99" href="#L99">99</a>
+<a class="jxr_linenumber" name="L100" href="#L100">100</a> <em class="jxr_comment">// Config is less critical than License, locator can still be used.</em>
+<a class="jxr_linenumber" name="L101" href="#L101">101</a> <em class="jxr_comment">// configureResourceLocator( configurationLocator, request, request.getConfigurationArtifacts() );</em>
+<a class="jxr_linenumber" name="L102" href="#L102">102</a>
+<a class="jxr_linenumber" name="L103" href="#L103">103</a> File[] files;
+<a class="jxr_linenumber" name="L104" href="#L104">104</a> <strong class="jxr_keyword">try</strong>
<a class="jxr_linenumber" name="L105" href="#L105">105</a> {
-<a class="jxr_linenumber" name="L106" href="#L106">106</a> <strong class="jxr_keyword">throw</strong> <strong class="jxr_keyword">new</strong> <a href="../../../../../org/apache/maven/plugin/checkstyle/CheckstyleExecutorException.html">CheckstyleExecutorException</a>( <span class="jxr_string">"Error getting files to process"</span>, e );
+<a class="jxr_linenumber" name="L106" href="#L106">106</a> files = getFilesToProcess( request );
<a class="jxr_linenumber" name="L107" href="#L107">107</a> }
-<a class="jxr_linenumber" name="L108" href="#L108">108</a>
-<a class="jxr_linenumber" name="L109" href="#L109">109</a> <strong class="jxr_keyword">final</strong> String suppressionsFilePath = getSuppressionsFilePath( request );
-<a class="jxr_linenumber" name="L110" href="#L110">110</a> FilterSet filterSet = getSuppressionsFilterSet( suppressionsFilePath );
-<a class="jxr_linenumber" name="L111" href="#L111">111</a>
-<a class="jxr_linenumber" name="L112" href="#L112">112</a> Checker checker = <strong class="jxr_keyword">new</strong> Checker();
-<a class="jxr_linenumber" name="L113" href="#L113">113</a>
-<a class="jxr_linenumber" name="L114" href="#L114">114</a> <em class="jxr_comment">// setup classloader, needed to avoid "Unable to get class information</em>
-<a class="jxr_linenumber" name="L115" href="#L115">115</a> <em class="jxr_comment">// for ..." errors</em>
-<a class="jxr_linenumber" name="L116" href="#L116">116</a> List<String> classPathStrings = <strong class="jxr_keyword">new</strong> ArrayList<String>();
-<a class="jxr_linenumber" name="L117" href="#L117">117</a> List<String> outputDirectories = <strong class="jxr_keyword">new</strong> ArrayList<String>();
-<a class="jxr_linenumber" name="L118" href="#L118">118</a> File sourceDirectory = request.getSourceDirectory();
-<a class="jxr_linenumber" name="L119" href="#L119">119</a> File testSourceDirectory = request.getTestSourceDirectory();
-<a class="jxr_linenumber" name="L120" href="#L120">120</a> <strong class="jxr_keyword">if</strong> ( request.isAggregate() )
-<a class="jxr_linenumber" name="L121" href="#L121">121</a> {
-<a class="jxr_linenumber" name="L122" href="#L122">122</a> <strong class="jxr_keyword">for</strong> ( MavenProject childProject : request.getReactorProjects() )
-<a class="jxr_linenumber" name="L123" href="#L123">123</a> {
-<a class="jxr_linenumber" name="L124" href="#L124">124</a> prepareCheckstylePaths( request, childProject, classPathStrings, outputDirectories,
-<a class="jxr_linenumber" name="L125" href="#L125">125</a> <strong class="jxr_keyword">new</strong> File( childProject.getBuild().getSourceDirectory() ),
-<a class="jxr_linenumber" name="L126" href="#L126">126</a> <strong class="jxr_keyword">new</strong> File( childProject.getBuild().getTestSourceDirectory() ) );
-<a class="jxr_linenumber" name="L127" href="#L127">127</a> }
-<a class="jxr_linenumber" name="L128" href="#L128">128</a> }
-<a class="jxr_linenumber" name="L129" href="#L129">129</a> <strong class="jxr_keyword">else</strong>
-<a class="jxr_linenumber" name="L130" href="#L130">130</a> {
-<a class="jxr_linenumber" name="L131" href="#L131">131</a> prepareCheckstylePaths( request, project, classPathStrings, outputDirectories, sourceDirectory,
-<a class="jxr_linenumber" name="L132" href="#L132">132</a> testSourceDirectory );
-<a class="jxr_linenumber" name="L133" href="#L133">133</a> }
-<a class="jxr_linenumber" name="L134" href="#L134">134</a>
-<a class="jxr_linenumber" name="L135" href="#L135">135</a> List<URL> urls = <strong class="jxr_keyword">new</strong> ArrayList<URL>( classPathStrings.size() );
-<a class="jxr_linenumber" name="L136" href="#L136">136</a>
-<a class="jxr_linenumber" name="L137" href="#L137">137</a> <strong class="jxr_keyword">for</strong> ( String path : classPathStrings )
-<a class="jxr_linenumber" name="L138" href="#L138">138</a> {
-<a class="jxr_linenumber" name="L139" href="#L139">139</a> <strong class="jxr_keyword">try</strong>
-<a class="jxr_linenumber" name="L140" href="#L140">140</a> {
-<a class="jxr_linenumber" name="L141" href="#L141">141</a> urls.add( <strong class="jxr_keyword">new</strong> File( path ).toURL() );
-<a class="jxr_linenumber" name="L142" href="#L142">142</a> }
-<a class="jxr_linenumber" name="L143" href="#L143">143</a> <strong class="jxr_keyword">catch</strong> ( MalformedURLException e )
+<a class="jxr_linenumber" name="L108" href="#L108">108</a> <strong class="jxr_keyword">catch</strong> ( IOException e )
+<a class="jxr_linenumber" name="L109" href="#L109">109</a> {
+<a class="jxr_linenumber" name="L110" href="#L110">110</a> <strong class="jxr_keyword">throw</strong> <strong class="jxr_keyword">new</strong> <a href="../../../../../org/apache/maven/plugin/checkstyle/CheckstyleExecutorException.html">CheckstyleExecutorException</a>( <span class="jxr_string">"Error getting files to process"</span>, e );
+<a class="jxr_linenumber" name="L111" href="#L111">111</a> }
+<a class="jxr_linenumber" name="L112" href="#L112">112</a>
+<a class="jxr_linenumber" name="L113" href="#L113">113</a> <strong class="jxr_keyword">final</strong> String suppressionsFilePath = getSuppressionsFilePath( request );
+<a class="jxr_linenumber" name="L114" href="#L114">114</a> FilterSet filterSet = getSuppressionsFilterSet( suppressionsFilePath );
+<a class="jxr_linenumber" name="L115" href="#L115">115</a>
+<a class="jxr_linenumber" name="L116" href="#L116">116</a> Checker checker = <strong class="jxr_keyword">new</strong> Checker();
+<a class="jxr_linenumber" name="L117" href="#L117">117</a>
+<a class="jxr_linenumber" name="L118" href="#L118">118</a> <em class="jxr_comment">// setup classloader, needed to avoid "Unable to get class information</em>
+<a class="jxr_linenumber" name="L119" href="#L119">119</a> <em class="jxr_comment">// for ..." errors</em>
+<a class="jxr_linenumber" name="L120" href="#L120">120</a> List<String> classPathStrings = <strong class="jxr_keyword">new</strong> ArrayList<String>();
+<a class="jxr_linenumber" name="L121" href="#L121">121</a> List<String> outputDirectories = <strong class="jxr_keyword">new</strong> ArrayList<String>();
+<a class="jxr_linenumber" name="L122" href="#L122">122</a> File sourceDirectory = request.getSourceDirectory();
+<a class="jxr_linenumber" name="L123" href="#L123">123</a> File testSourceDirectory = request.getTestSourceDirectory();
+<a class="jxr_linenumber" name="L124" href="#L124">124</a> <strong class="jxr_keyword">if</strong> ( request.isAggregate() )
+<a class="jxr_linenumber" name="L125" href="#L125">125</a> {
+<a class="jxr_linenumber" name="L126" href="#L126">126</a> <strong class="jxr_keyword">for</strong> ( MavenProject childProject : request.getReactorProjects() )
+<a class="jxr_linenumber" name="L127" href="#L127">127</a> {
+<a class="jxr_linenumber" name="L128" href="#L128">128</a> prepareCheckstylePaths( request, childProject, classPathStrings, outputDirectories,
+<a class="jxr_linenumber" name="L129" href="#L129">129</a> <strong class="jxr_keyword">new</strong> File( childProject.getBuild().getSourceDirectory() ),
+<a class="jxr_linenumber" name="L130" href="#L130">130</a> <strong class="jxr_keyword">new</strong> File( childProject.getBuild().getTestSourceDirectory() ) );
+<a class="jxr_linenumber" name="L131" href="#L131">131</a> }
+<a class="jxr_linenumber" name="L132" href="#L132">132</a> }
+<a class="jxr_linenumber" name="L133" href="#L133">133</a> <strong class="jxr_keyword">else</strong>
+<a class="jxr_linenumber" name="L134" href="#L134">134</a> {
+<a class="jxr_linenumber" name="L135" href="#L135">135</a> prepareCheckstylePaths( request, project, classPathStrings, outputDirectories, sourceDirectory,
+<a class="jxr_linenumber" name="L136" href="#L136">136</a> testSourceDirectory );
+<a class="jxr_linenumber" name="L137" href="#L137">137</a> }
+<a class="jxr_linenumber" name="L138" href="#L138">138</a>
+<a class="jxr_linenumber" name="L139" href="#L139">139</a> List<URL> urls = <strong class="jxr_keyword">new</strong> ArrayList<URL>( classPathStrings.size() );
+<a class="jxr_linenumber" name="L140" href="#L140">140</a>
+<a class="jxr_linenumber" name="L141" href="#L141">141</a> <strong class="jxr_keyword">for</strong> ( String path : classPathStrings )
+<a class="jxr_linenumber" name="L142" href="#L142">142</a> {
+<a class="jxr_linenumber" name="L143" href="#L143">143</a> <strong class="jxr_keyword">try</strong>
<a class="jxr_linenumber" name="L144" href="#L144">144</a> {
-<a class="jxr_linenumber" name="L145" href="#L145">145</a> <strong class="jxr_keyword">throw</strong> <strong class="jxr_keyword">new</strong> <a href="../../../../../org/apache/maven/plugin/checkstyle/CheckstyleExecutorException.html">CheckstyleExecutorException</a>( e.getMessage(), e );
+<a class="jxr_linenumber" name="L145" href="#L145">145</a> urls.add( <strong class="jxr_keyword">new</strong> File( path ).toURL() );
<a class="jxr_linenumber" name="L146" href="#L146">146</a> }
-<a class="jxr_linenumber" name="L147" href="#L147">147</a> }
-<a class="jxr_linenumber" name="L148" href="#L148">148</a>
-<a class="jxr_linenumber" name="L149" href="#L149">149</a> <strong class="jxr_keyword">for</strong> ( String outputDirectoryString : outputDirectories )
-<a class="jxr_linenumber" name="L150" href="#L150">150</a> {
-<a class="jxr_linenumber" name="L151" href="#L151">151</a> <strong class="jxr_keyword">try</strong>
-<a class="jxr_linenumber" name="L152" href="#L152">152</a> {
-<a class="jxr_linenumber" name="L153" href="#L153">153</a> <strong class="jxr_keyword">if</strong> ( outputDirectoryString != <strong class="jxr_keyword">null</strong> )
-<a class="jxr_linenumber" name="L154" href="#L154">154</a> {
-<a class="jxr_linenumber" name="L155" href="#L155">155</a> File outputDirectoryFile = <strong class="jxr_keyword">new</strong> File( outputDirectoryString );
-<a class="jxr_linenumber" name="L156" href="#L156">156</a> <strong class="jxr_keyword">if</strong> ( outputDirectoryFile.exists() )
-<a class="jxr_linenumber" name="L157" href="#L157">157</a> {
-<a class="jxr_linenumber" name="L158" href="#L158">158</a> URL outputDirectoryUrl = outputDirectoryFile.toURL();
-<a class="jxr_linenumber" name="L159" href="#L159">159</a> request.getLog().debug(
-<a class="jxr_linenumber" name="L160" href="#L160">160</a> <span class="jxr_string">"Adding the outputDirectory "</span> + outputDirectoryUrl.toString()
-<a class="jxr_linenumber" name="L161" href="#L161">161</a> + <span class="jxr_string">" to the Checkstyle class path"</span> );
-<a class="jxr_linenumber" name="L162" href="#L162">162</a> urls.add( outputDirectoryUrl );
-<a class="jxr_linenumber" name="L163" href="#L163">163</a> }
-<a class="jxr_linenumber" name="L164" href="#L164">164</a> }
-<a class="jxr_linenumber" name="L165" href="#L165">165</a> }
-<a class="jxr_linenumber" name="L166" href="#L166">166</a> <strong class="jxr_keyword">catch</strong> ( MalformedURLException e )
-<a class="jxr_linenumber" name="L167" href="#L167">167</a> {
-<a class="jxr_linenumber" name="L168" href="#L168">168</a> <strong class="jxr_keyword">throw</strong> <strong class="jxr_keyword">new</strong> <a href="../../../../../org/apache/maven/plugin/checkstyle/CheckstyleExecutorException.html">CheckstyleExecutorException</a>( e.getMessage(), e );
+<a class="jxr_linenumber" name="L147" href="#L147">147</a> <strong class="jxr_keyword">catch</strong> ( MalformedURLException e )
+<a class="jxr_linenumber" name="L148" href="#L148">148</a> {
+<a class="jxr_linenumber" name="L149" href="#L149">149</a> <strong class="jxr_keyword">throw</strong> <strong class="jxr_keyword">new</strong> <a href="../../../../../org/apache/maven/plugin/checkstyle/CheckstyleExecutorException.html">CheckstyleExecutorException</a>( e.getMessage(), e );
+<a class="jxr_linenumber" name="L150" href="#L150">150</a> }
+<a class="jxr_linenumber" name="L151" href="#L151">151</a> }
+<a class="jxr_linenumber" name="L152" href="#L152">152</a>
+<a class="jxr_linenumber" name="L153" href="#L153">153</a> <strong class="jxr_keyword">for</strong> ( String outputDirectoryString : outputDirectories )
+<a class="jxr_linenumber" name="L154" href="#L154">154</a> {
+<a class="jxr_linenumber" name="L155" href="#L155">155</a> <strong class="jxr_keyword">try</strong>
+<a class="jxr_linenumber" name="L156" href="#L156">156</a> {
+<a class="jxr_linenumber" name="L157" href="#L157">157</a> <strong class="jxr_keyword">if</strong> ( outputDirectoryString != <strong class="jxr_keyword">null</strong> )
+<a class="jxr_linenumber" name="L158" href="#L158">158</a> {
+<a class="jxr_linenumber" name="L159" href="#L159">159</a> File outputDirectoryFile = <strong class="jxr_keyword">new</strong> File( outputDirectoryString );
+<a class="jxr_linenumber" name="L160" href="#L160">160</a> <strong class="jxr_keyword">if</strong> ( outputDirectoryFile.exists() )
+<a class="jxr_linenumber" name="L161" href="#L161">161</a> {
+<a class="jxr_linenumber" name="L162" href="#L162">162</a> URL outputDirectoryUrl = outputDirectoryFile.toURL();
+<a class="jxr_linenumber" name="L163" href="#L163">163</a> request.getLog().debug(
+<a class="jxr_linenumber" name="L164" href="#L164">164</a> <span class="jxr_string">"Adding the outputDirectory "</span> + outputDirectoryUrl.toString()
+<a class="jxr_linenumber" name="L165" href="#L165">165</a> + <span class="jxr_string">" to the Checkstyle class path"</span> );
+<a class="jxr_linenumber" name="L166" href="#L166">166</a> urls.add( outputDirectoryUrl );
+<a class="jxr_linenumber" name="L167" href="#L167">167</a> }
+<a class="jxr_linenumber" name="L168" href="#L168">168</a> }
<a class="jxr_linenumber" name="L169" href="#L169">169</a> }
-<a class="jxr_linenumber" name="L170" href="#L170">170</a> }
-<a class="jxr_linenumber" name="L171" href="#L171">171</a>
-<a class="jxr_linenumber" name="L172" href="#L172">172</a> URLClassLoader projectClassLoader = <strong class="jxr_keyword">new</strong> URLClassLoader( urls.toArray( <strong class="jxr_keyword">new</strong> URL[urls.size()] ), <strong class="jxr_keyword">null</strong> );
-<a class="jxr_linenumber" name="L173" href="#L173">173</a> checker.setClassloader( projectClassLoader );
-<a class="jxr_linenumber" name="L174" href="#L174">174</a>
-<a class="jxr_linenumber" name="L175" href="#L175">175</a> checker.setModuleClassLoader( Thread.currentThread().getContextClassLoader() );
-<a class="jxr_linenumber" name="L176" href="#L176">176</a>
-<a class="jxr_linenumber" name="L177" href="#L177">177</a> <strong class="jxr_keyword">if</strong> ( filterSet != <strong class="jxr_keyword">null</strong> )
-<a class="jxr_linenumber" name="L178" href="#L178">178</a> {
-<a class="jxr_linenumber" name="L179" href="#L179">179</a> checker.addFilter( filterSet );
-<a class="jxr_linenumber" name="L180" href="#L180">180</a> }
-<a class="jxr_linenumber" name="L181" href="#L181">181</a> Configuration configuration = getConfiguration( request );
-<a class="jxr_linenumber" name="L182" href="#L182">182</a> checker.configure( configuration );
-<a class="jxr_linenumber" name="L183" href="#L183">183</a>
-<a class="jxr_linenumber" name="L184" href="#L184">184</a> AuditListener listener = request.getListener();
-<a class="jxr_linenumber" name="L185" href="#L185">185</a>
-<a class="jxr_linenumber" name="L186" href="#L186">186</a> <strong class="jxr_keyword">if</strong> ( listener != <strong class="jxr_keyword">null</strong> )
-<a class="jxr_linenumber" name="L187" href="#L187">187</a> {
-<a class="jxr_linenumber" name="L188" href="#L188">188</a> checker.addListener( listener );
-<a class="jxr_linenumber" name="L189" href="#L189">189</a> }
-<a class="jxr_linenumber" name="L190" href="#L190">190</a>
-<a class="jxr_linenumber" name="L191" href="#L191">191</a> <strong class="jxr_keyword">if</strong> ( request.isConsoleOutput() )
-<a class="jxr_linenumber" name="L192" href="#L192">192</a> {
-<a class="jxr_linenumber" name="L193" href="#L193">193</a> checker.addListener( request.getConsoleListener() );
-<a class="jxr_linenumber" name="L194" href="#L194">194</a> }
-<a class="jxr_linenumber" name="L195" href="#L195">195</a>
-<a class="jxr_linenumber" name="L196" href="#L196">196</a> <a href="../../../../../org/apache/maven/plugin/checkstyle/CheckstyleReportListener.html">CheckstyleReportListener</a> sinkListener = <strong class="jxr_keyword">new</strong> <a href="../../../../../org/apache/maven/plugin/checkstyle/CheckstyleReportListener.html">CheckstyleReportListener</a>( configuration );
-<a class="jxr_linenumber" name="L197" href="#L197">197</a> <strong class="jxr_keyword">if</strong> ( request.isAggregate() )
-<a class="jxr_linenumber" name="L198" href="#L198">198</a> {
-<a class="jxr_linenumber" name="L199" href="#L199">199</a> <strong class="jxr_keyword">for</strong> ( MavenProject childProject : request.getReactorProjects() )
-<a class="jxr_linenumber" name="L200" href="#L200">200</a> {
-<a class="jxr_linenumber" name="L201" href="#L201">201</a> addSourceDirectory( sinkListener, <strong class="jxr_keyword">new</strong> File( childProject.getBuild().getSourceDirectory() ),
-<a class="jxr_linenumber" name="L202" href="#L202">202</a> <strong class="jxr_keyword">new</strong> File( childProject.getBuild().getTestSourceDirectory() ),
-<a class="jxr_linenumber" name="L203" href="#L203">203</a> childProject.getResources(), request );
-<a class="jxr_linenumber" name="L204" href="#L204">204</a> }
-<a class="jxr_linenumber" name="L205" href="#L205">205</a> }
-<a class="jxr_linenumber" name="L206" href="#L206">206</a> <strong class="jxr_keyword">else</strong>
-<a class="jxr_linenumber" name="L207" href="#L207">207</a> {
-<a class="jxr_linenumber" name="L208" href="#L208">208</a> addSourceDirectory( sinkListener, sourceDirectory, testSourceDirectory, request.getResources(), request );
+<a class="jxr_linenumber" name="L170" href="#L170">170</a> <strong class="jxr_keyword">catch</strong> ( MalformedURLException e )
+<a class="jxr_linenumber" name="L171" href="#L171">171</a> {
+<a class="jxr_linenumber" name="L172" href="#L172">172</a> <strong class="jxr_keyword">throw</strong> <strong class="jxr_keyword">new</strong> <a href="../../../../../org/apache/maven/plugin/checkstyle/CheckstyleExecutorException.html">CheckstyleExecutorException</a>( e.getMessage(), e );
+<a class="jxr_linenumber" name="L173" href="#L173">173</a> }
+<a class="jxr_linenumber" name="L174" href="#L174">174</a> }
+<a class="jxr_linenumber" name="L175" href="#L175">175</a>
+<a class="jxr_linenumber" name="L176" href="#L176">176</a> URLClassLoader projectClassLoader = <strong class="jxr_keyword">new</strong> URLClassLoader( urls.toArray( <strong class="jxr_keyword">new</strong> URL[urls.size()] ), <strong class="jxr_keyword">null</strong> );
+<a class="jxr_linenumber" name="L177" href="#L177">177</a> checker.setClassloader( projectClassLoader );
+<a class="jxr_linenumber" name="L178" href="#L178">178</a>
+<a class="jxr_linenumber" name="L179" href="#L179">179</a> checker.setModuleClassLoader( Thread.currentThread().getContextClassLoader() );
+<a class="jxr_linenumber" name="L180" href="#L180">180</a>
+<a class="jxr_linenumber" name="L181" href="#L181">181</a> <strong class="jxr_keyword">if</strong> ( filterSet != <strong class="jxr_keyword">null</strong> )
+<a class="jxr_linenumber" name="L182" href="#L182">182</a> {
+<a class="jxr_linenumber" name="L183" href="#L183">183</a> checker.addFilter( filterSet );
+<a class="jxr_linenumber" name="L184" href="#L184">184</a> }
+<a class="jxr_linenumber" name="L185" href="#L185">185</a> Configuration configuration = getConfiguration( request );
+<a class="jxr_linenumber" name="L186" href="#L186">186</a> checker.configure( configuration );
+<a class="jxr_linenumber" name="L187" href="#L187">187</a>
+<a class="jxr_linenumber" name="L188" href="#L188">188</a> AuditListener listener = request.getListener();
+<a class="jxr_linenumber" name="L189" href="#L189">189</a>
+<a class="jxr_linenumber" name="L190" href="#L190">190</a> <strong class="jxr_keyword">if</strong> ( listener != <strong class="jxr_keyword">null</strong> )
+<a class="jxr_linenumber" name="L191" href="#L191">191</a> {
+<a class="jxr_linenumber" name="L192" href="#L192">192</a> checker.addListener( listener );
+<a class="jxr_linenumber" name="L193" href="#L193">193</a> }
+<a class="jxr_linenumber" name="L194" href="#L194">194</a>
+<a class="jxr_linenumber" name="L195" href="#L195">195</a> <strong class="jxr_keyword">if</strong> ( request.isConsoleOutput() )
+<a class="jxr_linenumber" name="L196" href="#L196">196</a> {
+<a class="jxr_linenumber" name="L197" href="#L197">197</a> checker.addListener( request.getConsoleListener() );
+<a class="jxr_linenumber" name="L198" href="#L198">198</a> }
+<a class="jxr_linenumber" name="L199" href="#L199">199</a>
+<a class="jxr_linenumber" name="L200" href="#L200">200</a> <a href="../../../../../org/apache/maven/plugin/checkstyle/CheckstyleReportListener.html">CheckstyleReportListener</a> sinkListener = <strong class="jxr_keyword">new</strong> <a href="../../../../../org/apache/maven/plugin/checkstyle/CheckstyleReportListener.html">CheckstyleReportListener</a>( configuration );
+<a class="jxr_linenumber" name="L201" href="#L201">201</a> <strong class="jxr_keyword">if</strong> ( request.isAggregate() )
+<a class="jxr_linenumber" name="L202" href="#L202">202</a> {
+<a class="jxr_linenumber" name="L203" href="#L203">203</a> <strong class="jxr_keyword">for</strong> ( MavenProject childProject : request.getReactorProjects() )
+<a class="jxr_linenumber" name="L204" href="#L204">204</a> {
+<a class="jxr_linenumber" name="L205" href="#L205">205</a> addSourceDirectory( sinkListener, <strong class="jxr_keyword">new</strong> File( childProject.getBuild().getSourceDirectory() ),
+<a class="jxr_linenumber" name="L206" href="#L206">206</a> <strong class="jxr_keyword">new</strong> File( childProject.getBuild().getTestSourceDirectory() ),
+<a class="jxr_linenumber" name="L207" href="#L207">207</a> childProject.getResources(), request );
+<a class="jxr_linenumber" name="L208" href="#L208">208</a> }
<a class="jxr_linenumber" name="L209" href="#L209">209</a> }
-<a class="jxr_linenumber" name="L210" href="#L210">210</a>
-<a class="jxr_linenumber" name="L211" href="#L211">211</a> checker.addListener( sinkListener );
-<a class="jxr_linenumber" name="L212" href="#L212">212</a>
-<a class="jxr_linenumber" name="L213" href="#L213">213</a> List<File> filesList = Arrays.asList( files );
-<a class="jxr_linenumber" name="L214" href="#L214">214</a> <strong class="jxr_keyword">int</strong> nbErrors = checker.process( filesList );
-<a class="jxr_linenumber" name="L215" href="#L215">215</a>
-<a class="jxr_linenumber" name="L216" href="#L216">216</a> checker.destroy();
-<a class="jxr_linenumber" name="L217" href="#L217">217</a>
-<a class="jxr_linenumber" name="L218" href="#L218">218</a> <strong class="jxr_keyword">if</strong> ( projectClassLoader instanceof Closeable )
-<a class="jxr_linenumber" name="L219" href="#L219">219</a> {
-<a class="jxr_linenumber" name="L220" href="#L220">220</a> <strong class="jxr_keyword">try</strong>
-<a class="jxr_linenumber" name="L221" href="#L221">221</a> {
-<a class="jxr_linenumber" name="L222" href="#L222">222</a> ( ( Closeable ) projectClassLoader ).close();
-<a class="jxr_linenumber" name="L223" href="#L223">223</a> }
-<a class="jxr_linenumber" name="L224" href="#L224">224</a> <strong class="jxr_keyword">catch</strong> ( IOException ex )
+<a class="jxr_linenumber" name="L210" href="#L210">210</a> <strong class="jxr_keyword">else</strong>
+<a class="jxr_linenumber" name="L211" href="#L211">211</a> {
+<a class="jxr_linenumber" name="L212" href="#L212">212</a> addSourceDirectory( sinkListener, sourceDirectory, testSourceDirectory, request.getResources(), request );
+<a class="jxr_linenumber" name="L213" href="#L213">213</a> }
+<a class="jxr_linenumber" name="L214" href="#L214">214</a>
+<a class="jxr_linenumber" name="L215" href="#L215">215</a> checker.addListener( sinkListener );
+<a class="jxr_linenumber" name="L216" href="#L216">216</a>
+<a class="jxr_linenumber" name="L217" href="#L217">217</a> List<File> filesList = Arrays.asList( files );
+<a class="jxr_linenumber" name="L218" href="#L218">218</a> <strong class="jxr_keyword">int</strong> nbErrors = checker.process( filesList );
+<a class="jxr_linenumber" name="L219" href="#L219">219</a>
+<a class="jxr_linenumber" name="L220" href="#L220">220</a> checker.destroy();
+<a class="jxr_linenumber" name="L221" href="#L221">221</a>
+<a class="jxr_linenumber" name="L222" href="#L222">222</a> <strong class="jxr_keyword">if</strong> ( projectClassLoader instanceof Closeable )
+<a class="jxr_linenumber" name="L223" href="#L223">223</a> {
+<a class="jxr_linenumber" name="L224" href="#L224">224</a> <strong class="jxr_keyword">try</strong>
<a class="jxr_linenumber" name="L225" href="#L225">225</a> {
-<a class="jxr_linenumber" name="L226" href="#L226">226</a> <em class="jxr_comment">// Nothing we can do - and not detrimental to the build (save running out of file handles).</em>
-<a class="jxr_linenumber" name="L227" href="#L227">227</a> getLogger().info( <span class="jxr_string">"Failed to close custom Classloader - this indicated a bug in the code."</span>, ex );
-<a class="jxr_linenumber" name="L228" href="#L228">228</a> }
-<a class="jxr_linenumber" name="L229" href="#L229">229</a> }
-<a class="jxr_linenumber" name="L230" href="#L230">230</a>
-<a class="jxr_linenumber" name="L231" href="#L231">231</a> <strong class="jxr_keyword">if</strong> ( request.getStringOutputStream() != <strong class="jxr_keyword">null</strong> )
-<a class="jxr_linenumber" name="L232" href="#L232">232</a> {
-<a class="jxr_linenumber" name="L233" href="#L233">233</a> request.getLog().info( request.getStringOutputStream().toString() );
-<a class="jxr_linenumber" name="L234" href="#L234">234</a> }
-<a class="jxr_linenumber" name="L235" href="#L235">235</a>
-<a class="jxr_linenumber" name="L236" href="#L236">236</a> <strong class="jxr_keyword">if</strong> ( request.isFailsOnError() && nbErrors > 0 )
-<a class="jxr_linenumber" name="L237" href="#L237">237</a> {
-<a class="jxr_linenumber" name="L238" href="#L238">238</a> <em class="jxr_comment">// TODO: should be a failure, not an error. Report is not meant to</em>
-<a class="jxr_linenumber" name="L239" href="#L239">239</a> <em class="jxr_comment">// throw an exception here (so site would</em>
-<a class="jxr_linenumber" name="L240" href="#L240">240</a> <em class="jxr_comment">// work regardless of config), but should record this information</em>
-<a class="jxr_linenumber" name="L241" href="#L241">241</a> <strong class="jxr_keyword">throw</strong> <strong class="jxr_keyword">new</strong> <a href="../../../../../org/apache/maven/plugin/checkstyle/CheckstyleExecutorException.html">CheckstyleExecutorException</a>( <span class="jxr_string">"There are "</span> + nbErrors + <span class="jxr_string">" checkstyle errors."</span> );
-<a class="jxr_linenumber" name="L242" href="#L242">242</a> }
-<a class="jxr_linenumber" name="L243" href="#L243">243</a> <strong class="jxr_keyword">else</strong> <strong class="jxr_keyword">if</strong> ( nbErrors > 0 )
-<a class="jxr_linenumber" name="L244" href="#L244">244</a> {
-<a class="jxr_linenumber" name="L245" href="#L245">245</a> request.getLog().info( <span class="jxr_string">"There are "</span> + nbErrors + <span class="jxr_string">" checkstyle errors."</span> );
+<a class="jxr_linenumber" name="L226" href="#L226">226</a> ( ( Closeable ) projectClassLoader ).close();
+<a class="jxr_linenumber" name="L227" href="#L227">227</a> }
+<a class="jxr_linenumber" name="L228" href="#L228">228</a> <strong class="jxr_keyword">catch</strong> ( IOException ex )
+<a class="jxr_linenumber" name="L229" href="#L229">229</a> {
+<a class="jxr_linenumber" name="L230" href="#L230">230</a> <em class="jxr_comment">// Nothing we can do - and not detrimental to the build (save running out of file handles).</em>
+<a class="jxr_linenumber" name="L231" href="#L231">231</a> getLogger().info( <span class="jxr_string">"Failed to close custom Classloader - this indicated a bug in the code."</span>, ex );
+<a class="jxr_linenumber" name="L232" href="#L232">232</a> }
+<a class="jxr_linenumber" name="L233" href="#L233">233</a> }
+<a class="jxr_linenumber" name="L234" href="#L234">234</a>
+<a class="jxr_linenumber" name="L235" href="#L235">235</a> <strong class="jxr_keyword">if</strong> ( request.getStringOutputStream() != <strong class="jxr_keyword">null</strong> )
+<a class="jxr_linenumber" name="L236" href="#L236">236</a> {
+<a class="jxr_linenumber" name="L237" href="#L237">237</a> request.getLog().info( request.getStringOutputStream().toString() );
+<a class="jxr_linenumber" name="L238" href="#L238">238</a> }
+<a class="jxr_linenumber" name="L239" href="#L239">239</a>
+<a class="jxr_linenumber" name="L240" href="#L240">240</a> <strong class="jxr_keyword">if</strong> ( request.isFailsOnError() && nbErrors > 0 )
+<a class="jxr_linenumber" name="L241" href="#L241">241</a> {
+<a class="jxr_linenumber" name="L242" href="#L242">242</a> <em class="jxr_comment">// TODO: should be a failure, not an error. Report is not meant to</em>
+<a class="jxr_linenumber" name="L243" href="#L243">243</a> <em class="jxr_comment">// throw an exception here (so site would</em>
+<a class="jxr_linenumber" name="L244" href="#L244">244</a> <em class="jxr_comment">// work regardless of config), but should record this information</em>
+<a class="jxr_linenumber" name="L245" href="#L245">245</a> <strong class="jxr_keyword">throw</strong> <strong class="jxr_keyword">new</strong> <a href="../../../../../org/apache/maven/plugin/checkstyle/CheckstyleExecutorException.html">CheckstyleExecutorException</a>( <span class="jxr_string">"There are "</span> + nbErrors + <span class="jxr_string">" checkstyle errors."</span> );
<a class="jxr_linenumber" name="L246" href="#L246">246</a> }
-<a class="jxr_linenumber" name="L247" href="#L247">247</a>
-<a class="jxr_linenumber" name="L248" href="#L248">248</a> <strong class="jxr_keyword">return</strong> sinkListener.getResults();
-<a class="jxr_linenumber" name="L249" href="#L249">249</a> }
-<a class="jxr_linenumber" name="L250" href="#L250">250</a>
-<a class="jxr_linenumber" name="L251" href="#L251">251</a> <strong class="jxr_keyword">protected</strong> <strong class="jxr_keyword">void</strong> addSourceDirectory( CheckstyleReportListener sinkListener, File sourceDirectory,
-<a class="jxr_linenumber" name="L252" href="#L252">252</a> File testSourceDirectory, List<Resource> resources,
-<a class="jxr_linenumber" name="L253" href="#L253">253</a> <a href="../../../../../org/apache/maven/plugin/checkstyle/CheckstyleExecutorRequest.html">CheckstyleExecutorRequest</a> request )
-<a class="jxr_linenumber" name="L254" href="#L254">254</a> {
-<a class="jxr_linenumber" name="L255" href="#L255">255</a> <strong class="jxr_keyword">if</strong> ( sourceDirectory != <strong class="jxr_keyword">null</strong> )
-<a class="jxr_linenumber" name="L256" href="#L256">256</a> {
-<a class="jxr_linenumber" name="L257" href="#L257">257</a> sinkListener.addSourceDirectory( sourceDirectory );
-<a class="jxr_linenumber" name="L258" href="#L258">258</a> }
-<a class="jxr_linenumber" name="L259" href="#L259">259</a>
-<a class="jxr_linenumber" name="L260" href="#L260">260</a> <strong class="jxr_keyword">if</strong> ( request.isIncludeTestSourceDirectory() && ( testSourceDirectory != <strong class="jxr_keyword">null</strong> )
-<a class="jxr_linenumber" name="L261" href="#L261">261</a> && ( testSourceDirectory.exists() ) && ( testSourceDirectory.isDirectory() ) )
-<a class="jxr_linenumber" name="L262" href="#L262">262</a> {
-<a class="jxr_linenumber" name="L263" href="#L263">263</a> sinkListener.addSourceDirectory( testSourceDirectory );
-<a class="jxr_linenumber" name="L264" href="#L264">264</a> }
-<a class="jxr_linenumber" name="L265" href="#L265">265</a>
-<a class="jxr_linenumber" name="L266" href="#L266">266</a> <strong class="jxr_keyword">if</strong> ( resources != <strong class="jxr_keyword">null</strong> )
-<a class="jxr_linenumber" name="L267" href="#L267">267</a> {
-<a class="jxr_linenumber" name="L268" href="#L268">268</a> <strong class="jxr_keyword">for</strong> ( Resource resource : resources )
-<a class="jxr_linenumber" name="L269" href="#L269">269</a> {
-<a class="jxr_linenumber" name="L270" href="#L270">270</a> <strong class="jxr_keyword">if</strong> ( resource.getDirectory() != <strong class="jxr_keyword">null</strong> )
-<a class="jxr_linenumber" name="L271" href="#L271">271</a> {
-<a class="jxr_linenumber" name="L272" href="#L272">272</a> File resourcesDirectory = <strong class="jxr_keyword">new</strong> File( resource.getDirectory() );
-<a class="jxr_linenumber" name="L273" href="#L273">273</a> <strong class="jxr_keyword">if</strong> ( resourcesDirectory.exists() && resourcesDirectory.isDirectory() )
-<a class="jxr_linenumber" name="L274" href="#L274">274</a> {
-<a class="jxr_linenumber" name="L275" href="#L275">275</a> sinkListener.addSourceDirectory( resourcesDirectory );
-<a class="jxr_linenumber" name="L276" href="#L276">276</a> getLogger().debug( <span class="jxr_string">"Added '"</span> + resourcesDirectory.getAbsolutePath()
-<a class="jxr_linenumber" name="L277" href="#L277">277</a> + <span class="jxr_string">"' as a source directory."</span> );
-<a class="jxr_linenumber" name="L278" href="#L278">278</a> }
-<a class="jxr_linenumber" name="L279" href="#L279">279</a> }
-<a class="jxr_linenumber" name="L280" href="#L280">280</a> }
-<a class="jxr_linenumber" name="L281" href="#L281">281</a> }
-<a class="jxr_linenumber" name="L282" href="#L282">282</a> }
-<a class="jxr_linenumber" name="L283" href="#L283">283</a>
-<a class="jxr_linenumber" name="L284" href="#L284">284</a> <strong class="jxr_keyword">public</strong> Configuration getConfiguration( CheckstyleExecutorRequest request )
-<a class="jxr_linenumber" name="L285" href="#L285">285</a> <strong class="jxr_keyword">throws</strong> <a href="../../../../../org/apache/maven/plugin/checkstyle/CheckstyleExecutorException.html">CheckstyleExecutorException</a>
-<a class="jxr_linenumber" name="L286" href="#L286">286</a> {
-<a class="jxr_linenumber" name="L287" href="#L287">287</a> <strong class="jxr_keyword">try</strong>
-<a class="jxr_linenumber" name="L288" href="#L288">288</a> {
-<a class="jxr_linenumber" name="L289" href="#L289">289</a> <em class="jxr_comment">// checkstyle will always use the context classloader in order</em>
-<a class="jxr_linenumber" name="L290" href="#L290">290</a> <em class="jxr_comment">// to load resources (dtds),</em>
-<a class="jxr_linenumber" name="L291" href="#L291">291</a> <em class="jxr_comment">// so we have to fix it</em>
-<a class="jxr_linenumber" name="L292" href="#L292">292</a> ClassLoader checkstyleClassLoader = PackageNamesLoader.<strong class="jxr_keyword">class</strong>.getClassLoader();
-<a class="jxr_linenumber" name="L293" href="#L293">293</a> Thread.currentThread().setContextClassLoader( checkstyleClassLoader );
-<a class="jxr_linenumber" name="L294" href="#L294">294</a> String configFile = getConfigFile( request );
-<a class="jxr_linenumber" name="L295" href="#L295">295</a> Properties overridingProperties = getOverridingProperties( request );
-<a class="jxr_linenumber" name="L296" href="#L296">296</a> Configuration config = ConfigurationLoader
-<a class="jxr_linenumber" name="L297" href="#L297">297</a> .loadConfiguration( configFile, <strong class="jxr_keyword">new</strong> PropertiesExpander( overridingProperties ) );
-<a class="jxr_linenumber" name="L298" href="#L298">298</a> String effectiveEncoding = StringUtils.isNotEmpty( request.getEncoding() ) ? request.getEncoding() : System
-<a class="jxr_linenumber" name="L299" href="#L299">299</a> .getProperty( <span class="jxr_string">"file.encoding"</span>, <span class="jxr_string">"UTF-8"</span> );
-<a class="jxr_linenumber" name="L300" href="#L300">300</a>
-<a class="jxr_linenumber" name="L301" href="#L301">301</a> <strong class="jxr_keyword">if</strong> ( StringUtils.isEmpty( request.getEncoding() ) )
-<a class="jxr_linenumber" name="L302" href="#L302">302</a> {
-<a class="jxr_linenumber" name="L303" href="#L303">303</a> request.getLog().warn(
-<a class="jxr_linenumber" name="L304" href="#L304">304</a> <span class="jxr_string">"File encoding has not been set, using platform encoding "</span> + effectiveEncoding
-<a class="jxr_linenumber" name="L305" href="#L305">305</a> + <span class="jxr_string">", i.e. build is platform dependent!"</span> );
-<a class="jxr_linenumber" name="L306" href="#L306">306</a> }
-<a class="jxr_linenumber" name="L307" href="#L307">307</a>
-<a class="jxr_linenumber" name="L308" href="#L308">308</a> <strong class="jxr_keyword">if</strong> ( <span class="jxr_string">"Checker"</span>.equals( config.getName() )
-<a class="jxr_linenumber" name="L309" href="#L309">309</a> || <span class="jxr_string">"com.puppycrawl.tools.checkstyle.Checker"</span>.equals( config.getName() ) )
-<a class="jxr_linenumber" name="L310" href="#L310">310</a> {
-<a class="jxr_linenumber" name="L311" href="#L311">311</a> <strong class="jxr_keyword">if</strong> ( config instanceof DefaultConfiguration )
-<a class="jxr_linenumber" name="L312" href="#L312">312</a> {
-<a class="jxr_linenumber" name="L313" href="#L313">313</a> <em class="jxr_comment">// MCHECKSTYLE-173 Only add the "charset" attribute if it has not been set</em>
-<a class="jxr_linenumber" name="L314" href="#L314">314</a> <strong class="jxr_keyword">try</strong>
-<a class="jxr_linenumber" name="L315" href="#L315">315</a> {
-<a class="jxr_linenumber" name="L316" href="#L316">316</a> <strong class="jxr_keyword">if</strong> ( config.getAttribute( <span class="jxr_string">"charset"</span> ) == <strong class="jxr_keyword">null</strong> )
-<a class="jxr_linenumber" name="L317" href="#L317">317</a> {
-<a class="jxr_linenumber" name="L318" href="#L318">318</a> ( (DefaultConfiguration) config ).addAttribute( <span class="jxr_string">"charset"</span>, effectiveEncoding );
-<a class="jxr_linenumber" name="L319" href="#L319">319</a> }
-<a class="jxr_linenumber" name="L320" href="#L320">320</a> }
-<a class="jxr_linenumber" name="L321" href="#L321">321</a> <strong class="jxr_keyword">catch</strong> ( CheckstyleException ex )
-<a class="jxr_linenumber" name="L322" href="#L322">322</a> {
-<a class="jxr_linenumber" name="L323" href="#L323">323</a> <em class="jxr_comment">// Checkstyle 5.4+ throws an exception when trying to access an attribute that doesn't exist</em>
-<a class="jxr_linenumber" name="L324" href="#L324">324</a> ( (DefaultConfiguration) config ).addAttribute( <span class="jxr_string">"charset"</span>, effectiveEncoding );
-<a class="jxr_linenumber" name="L325" href="#L325">325</a> }
-<a class="jxr_linenumber" name="L326" href="#L326">326</a> }
-<a class="jxr_linenumber" name="L327" href="#L327">327</a> <strong class="jxr_keyword">else</strong>
-<a class="jxr_linenumber" name="L328" href="#L328">328</a> {
-<a class="jxr_linenumber" name="L329" href="#L329">329</a> request.getLog().warn( <span class="jxr_string">"Failed to configure file encoding on module "</span> + config );
+<a class="jxr_linenumber" name="L247" href="#L247">247</a> <strong class="jxr_keyword">else</strong> <strong class="jxr_keyword">if</strong> ( nbErrors > 0 )
+<a class="jxr_linenumber" name="L248" href="#L248">248</a> {
+<a class="jxr_linenumber" name="L249" href="#L249">249</a> request.getLog().info( <span class="jxr_string">"There are "</span> + nbErrors + <span class="jxr_string">" checkstyle errors."</span> );
+<a class="jxr_linenumber" name="L250" href="#L250">250</a> }
+<a class="jxr_linenumber" name="L251" href="#L251">251</a>
+<a class="jxr_linenumber" name="L252" href="#L252">252</a> <strong class="jxr_keyword">return</strong> sinkListener.getResults();
+<a class="jxr_linenumber" name="L253" href="#L253">253</a> }
+<a class="jxr_linenumber" name="L254" href="#L254">254</a>
+<a class="jxr_linenumber" name="L255" href="#L255">255</a> <strong class="jxr_keyword">protected</strong> <strong class="jxr_keyword">void</strong> addSourceDirectory( CheckstyleReportListener sinkListener, File sourceDirectory,
+<a class="jxr_linenumber" name="L256" href="#L256">256</a> File testSourceDirectory, List<Resource> resources,
+<a class="jxr_linenumber" name="L257" href="#L257">257</a> <a href="../../../../../org/apache/maven/plugin/checkstyle/CheckstyleExecutorRequest.html">CheckstyleExecutorRequest</a> request )
+<a class="jxr_linenumber" name="L258" href="#L258">258</a> {
+<a class="jxr_linenumber" name="L259" href="#L259">259</a> <strong class="jxr_keyword">if</strong> ( sourceDirectory != <strong class="jxr_keyword">null</strong> )
+<a class="jxr_linenumber" name="L260" href="#L260">260</a> {
+<a class="jxr_linenumber" name="L261" href="#L261">261</a> sinkListener.addSourceDirectory( sourceDirectory );
+<a class="jxr_linenumber" name="L262" href="#L262">262</a> }
+<a class="jxr_linenumber" name="L263" href="#L263">263</a>
+<a class="jxr_linenumber" name="L264" href="#L264">264</a> <strong class="jxr_keyword">if</strong> ( request.isIncludeTestSourceDirectory() && ( testSourceDirectory != <strong class="jxr_keyword">null</strong> )
+<a class="jxr_linenumber" name="L265" href="#L265">265</a> && ( testSourceDirectory.exists() ) && ( testSourceDirectory.isDirectory() ) )
+<a class="jxr_linenumber" name="L266" href="#L266">266</a> {
+<a class="jxr_linenumber" name="L267" href="#L267">267</a> sinkListener.addSourceDirectory( testSourceDirectory );
+<a class="jxr_linenumber" name="L268" href="#L268">268</a> }
+<a class="jxr_linenumber" name="L269" href="#L269">269</a>
+<a class="jxr_linenumber" name="L270" href="#L270">270</a> <strong class="jxr_keyword">if</strong> ( resources != <strong class="jxr_keyword">null</strong> )
+<a class="jxr_linenumber" name="L271" href="#L271">271</a> {
+<a class="jxr_linenumber" name="L272" href="#L272">272</a> <strong class="jxr_keyword">for</strong> ( Resource resource : resources )
+<a class="jxr_linenumber" name="L273" href="#L273">273</a> {
+<a class="jxr_linenumber" name="L274" href="#L274">274</a> <strong class="jxr_keyword">if</strong> ( resource.getDirectory() != <strong class="jxr_keyword">null</strong> )
+<a class="jxr_linenumber" name="L275" href="#L275">275</a> {
+<a class="jxr_linenumber" name="L276" href="#L276">276</a> File resourcesDirectory = <strong class="jxr_keyword">new</strong> File( resource.getDirectory() );
+<a class="jxr_linenumber" name="L277" href="#L277">277</a> <strong class="jxr_keyword">if</strong> ( resourcesDirectory.exists() && resourcesDirectory.isDirectory() )
+<a class="jxr_linenumber" name="L278" href="#L278">278</a> {
+<a class="jxr_linenumber" name="L279" href="#L279">279</a> sinkListener.addSourceDirectory( resourcesDirectory );
+<a class="jxr_linenumber" name="L280" href="#L280">280</a> getLogger().debug( <span class="jxr_string">"Added '"</span> + resourcesDirectory.getAbsolutePath()
+<a class="jxr_linenumber" name="L281" href="#L281">281</a> + <span class="jxr_string">"' as a source directory."</span> );
+<a class="jxr_linenumber" name="L282" href="#L282">282</a> }
+<a class="jxr_linenumber" name="L283" href="#L283">283</a> }
+<a class="jxr_linenumber" name="L284" href="#L284">284</a> }
+<a class="jxr_linenumber" name="L285" href="#L285">285</a> }
+<a class="jxr_linenumber" name="L286" href="#L286">286</a> }
+<a class="jxr_linenumber" name="L287" href="#L287">287</a>
+<a class="jxr_linenumber" name="L288" href="#L288">288</a> <strong class="jxr_keyword">public</strong> Configuration getConfiguration( CheckstyleExecutorRequest request )
+<a class="jxr_linenumber" name="L289" href="#L289">289</a> <strong class="jxr_keyword">throws</strong> <a href="../../../../../org/apache/maven/plugin/checkstyle/CheckstyleExecutorException.html">CheckstyleExecutorException</a>
+<a class="jxr_linenumber" name="L290" href="#L290">290</a> {
+<a class="jxr_linenumber" name="L291" href="#L291">291</a> <strong class="jxr_keyword">try</strong>
+<a class="jxr_linenumber" name="L292" href="#L292">292</a> {
+<a class="jxr_linenumber" name="L293" href="#L293">293</a> <em class="jxr_comment">// checkstyle will always use the context classloader in order</em>
+<a class="jxr_linenumber" name="L294" href="#L294">294</a> <em class="jxr_comment">// to load resources (dtds),</em>
+<a class="jxr_linenumber" name="L295" href="#L295">295</a> <em class="jxr_comment">// so we have to fix it</em>
+<a class="jxr_linenumber" name="L296" href="#L296">296</a> ClassLoader checkstyleClassLoader = PackageNamesLoader.<strong class="jxr_keyword">class</strong>.getClassLoader();
+<a class="jxr_linenumber" name="L297" href="#L297">297</a> Thread.currentThread().setContextClassLoader( checkstyleClassLoader );
+<a class="jxr_linenumber" name="L298" href="#L298">298</a> String configFile = getConfigFile( request );
+<a class="jxr_linenumber" name="L299" href="#L299">299</a> Properties overridingProperties = getOverridingProperties( request );
+<a class="jxr_linenumber" name="L300" href="#L300">300</a> Configuration config = ConfigurationLoader
+<a class="jxr_linenumber" name="L301" href="#L301">301</a> .loadConfiguration( configFile, <strong class="jxr_keyword">new</strong> PropertiesExpander( overridingProperties ) );
+<a class="jxr_linenumber" name="L302" href="#L302">302</a> String effectiveEncoding = StringUtils.isNotEmpty( request.getEncoding() ) ? request.getEncoding() : System
+<a class="jxr_linenumber" name="L303" href="#L303">303</a> .getProperty( <span class="jxr_string">"file.encoding"</span>, <span class="jxr_string">"UTF-8"</span> );
+<a class="jxr_linenumber" name="L304" href="#L304">304</a>
+<a class="jxr_linenumber" name="L305" href="#L305">305</a> <strong class="jxr_keyword">if</strong> ( StringUtils.isEmpty( request.getEncoding() ) )
+<a class="jxr_linenumber" name="L306" href="#L306">306</a> {
+<a class="jxr_linenumber" name="L307" href="#L307">307</a> request.getLog().warn(
+<a class="jxr_linenumber" name="L308" href="#L308">308</a> <span class="jxr_string">"File encoding has not been set, using platform encoding "</span> + effectiveEncoding
+<a class="jxr_linenumber" name="L309" href="#L309">309</a> + <span class="jxr_string">", i.e. build is platform dependent!"</span> );
+<a class="jxr_linenumber" name="L310" href="#L310">310</a> }
+<a class="jxr_linenumber" name="L311" href="#L311">311</a>
+<a class="jxr_linenumber" name="L312" href="#L312">312</a> <strong class="jxr_keyword">if</strong> ( <span class="jxr_string">"Checker"</span>.equals( config.getName() )
+<a class="jxr_linenumber" name="L313" href="#L313">313</a> || <span class="jxr_string">"com.puppycrawl.tools.checkstyle.Checker"</span>.equals( config.getName() ) )
+<a class="jxr_linenumber" name="L314" href="#L314">314</a> {
+<a class="jxr_linenumber" name="L315" href="#L315">315</a> <strong class="jxr_keyword">if</strong> ( config instanceof DefaultConfiguration )
+<a class="jxr_linenumber" name="L316" href="#L316">316</a> {
+<a class="jxr_linenumber" name="L317" href="#L317">317</a> <em class="jxr_comment">// MCHECKSTYLE-173 Only add the "charset" attribute if it has not been set</em>
+<a class="jxr_linenumber" name="L318" href="#L318">318</a> <strong class="jxr_keyword">try</strong>
+<a class="jxr_linenumber" name="L319" href="#L319">319</a> {
+<a class="jxr_linenumber" name="L320" href="#L320">320</a> <strong class="jxr_keyword">if</strong> ( config.getAttribute( <span class="jxr_string">"charset"</span> ) == <strong class="jxr_keyword">null</strong> )
+<a class="jxr_linenumber" name="L321" href="#L321">321</a> {
+<a class="jxr_linenumber" name="L322" href="#L322">322</a> ( (DefaultConfiguration) config ).addAttribute( <span class="jxr_string">"charset"</span>, effectiveEncoding );
+<a class="jxr_linenumber" name="L323" href="#L323">323</a> }
+<a class="jxr_linenumber" name="L324" href="#L324">324</a> }
+<a class="jxr_linenumber" name="L325" href="#L325">325</a> <strong class="jxr_keyword">catch</strong> ( CheckstyleException ex )
+<a class="jxr_linenumber" name="L326" href="#L326">326</a> {
+<a class="jxr_linenumber" name="L327" href="#L327">327</a> <em class="jxr_comment">// Checkstyle 5.4+ throws an exception when trying to access an attribute that doesn't exist</em>
+<a class="jxr_linenumber" name="L328" href="#L328">328</a> ( (DefaultConfiguration) config ).addAttribute( <span class="jxr_string">"charset"</span>, effectiveEncoding );
+<a class="jxr_linenumber" name="L329" href="#L329">329</a> }
<a class="jxr_linenumber" name="L330" href="#L330">330</a> }
-<a class="jxr_linenumber" name="L331" href="#L331">331</a> }
-<a class="jxr_linenumber" name="L332" href="#L332">332</a> Configuration[] modules = config.getChildren();
-<a class="jxr_linenumber" name="L333" href="#L333">333</a> <strong class="jxr_keyword">for</strong> ( Configuration module : modules )
-<a class="jxr_linenumber" name="L334" href="#L334">334</a> {
-<a class="jxr_linenumber" name="L335" href="#L335">335</a> <strong class="jxr_keyword">if</strong> ( <span class="jxr_string">"TreeWalker"</span>.equals( module.getName() )
-<a class="jxr_linenumber" name="L336" href="#L336">336</a> || <span class="jxr_string">"com.puppycrawl.tools.checkstyle.TreeWalker"</span>.equals( module.getName() ) )
-<a class="jxr_linenumber" name="L337" href="#L337">337</a> {
-<a class="jxr_linenumber" name="L338" href="#L338">338</a> <strong class="jxr_keyword">if</strong> ( module instanceof DefaultConfiguration )
-<a class="jxr_linenumber" name="L339" href="#L339">339</a> {
-<a class="jxr_linenumber" name="L340" href="#L340">340</a> <em class="jxr_comment">// MCHECKSTYLE-132 DefaultConfiguration addAttribute has changed in checkstyle 5.3</em>
-<a class="jxr_linenumber" name="L341" href="#L341">341</a> <strong class="jxr_keyword">try</strong>
-<a class="jxr_linenumber" name="L342" href="#L342">342</a> {
-<a class="jxr_linenumber" name="L343" href="#L343">343</a> <strong class="jxr_keyword">if</strong> ( module.getAttribute( <span class="jxr_string">"cacheFile"</span> ) == <strong class="jxr_keyword">null</strong> )
-<a class="jxr_linenumber" name="L344" href="#L344">344</a> {
-<a class="jxr_linenumber" name="L345" href="#L345">345</a> ( (DefaultConfiguration) module ).addAttribute( <span class="jxr_string">"cacheFile"</span>, request.getCacheFile() );
-<a class="jxr_linenumber" name="L346" href="#L346">346</a> }
-<a class="jxr_linenumber" name="L347" href="#L347">347</a> }
-<a class="jxr_linenumber" name="L348" href="#L348">348</a> <strong class="jxr_keyword">catch</strong> ( CheckstyleException ex )
-<a class="jxr_linenumber" name="L349" href="#L349">349</a> {
-<a class="jxr_linenumber" name="L350" href="#L350">350</a> <em class="jxr_comment">// MCHECKSTYLE-159 - checkstyle 5.4 throws an exception instead of return null if</em>
-<a class="jxr_linenumber" name="L351" href="#L351">351</a> <em class="jxr_comment">// "cacheFile"</em>
-<a class="jxr_linenumber" name="L352" href="#L352">352</a> <em class="jxr_comment">// doesn't exist</em>
-<a class="jxr_linenumber" name="L353" href="#L353">353</a> ( (DefaultConfiguration) module ).addAttribute( <span class="jxr_string">"cacheFile"</span>, request.getCacheFile() );
-<a class="jxr_linenumber" name="L354" href="#L354">354</a> }
-<a class="jxr_linenumber" name="L355" href="#L355">355</a> }
-<a class="jxr_linenumber" name="L356" href="#L356">356</a> <strong class="jxr_keyword">else</strong>
-<a class="jxr_linenumber" name="L357" href="#L357">357</a> {
-<a class="jxr_linenumber" name="L358" href="#L358">358</a> request.getLog().warn( <span class="jxr_string">"Failed to configure cache file on module "</span> + module );
+<a class="jxr_linenumber" name="L331" href="#L331">331</a> <strong class="jxr_keyword">else</strong>
+<a class="jxr_linenumber" name="L332" href="#L332">332</a> {
+<a class="jxr_linenumber" name="L333" href="#L333">333</a> request.getLog().warn( <span class="jxr_string">"Failed to configure file encoding on module "</span> + config );
+<a class="jxr_linenumber" name="L334" href="#L334">334</a> }
+<a class="jxr_linenumber" name="L335" href="#L335">335</a> }
+<a class="jxr_linenumber" name="L336" href="#L336">336</a> Configuration[] modules = config.getChildren();
+<a class="jxr_linenumber" name="L337" href="#L337">337</a> <strong class="jxr_keyword">for</strong> ( Configuration module : modules )
+<a class="jxr_linenumber" name="L338" href="#L338">338</a> {
+<a class="jxr_linenumber" name="L339" href="#L339">339</a> <strong class="jxr_keyword">if</strong> ( <span class="jxr_string">"TreeWalker"</span>.equals( module.getName() )
+<a class="jxr_linenumber" name="L340" href="#L340">340</a> || <span class="jxr_string">"com.puppycrawl.tools.checkstyle.TreeWalker"</span>.equals( module.getName() ) )
+<a class="jxr_linenumber" name="L341" href="#L341">341</a> {
+<a class="jxr_linenumber" name="L342" href="#L342">342</a> <strong class="jxr_keyword">if</strong> ( module instanceof DefaultConfiguration )
+<a class="jxr_linenumber" name="L343" href="#L343">343</a> {
+<a class="jxr_linenumber" name="L344" href="#L344">344</a> <em class="jxr_comment">// MCHECKSTYLE-132 DefaultConfiguration addAttribute has changed in checkstyle 5.3</em>
+<a class="jxr_linenumber" name="L345" href="#L345">345</a> <strong class="jxr_keyword">try</strong>
+<a class="jxr_linenumber" name="L346" href="#L346">346</a> {
+<a class="jxr_linenumber" name="L347" href="#L347">347</a> <strong class="jxr_keyword">if</strong> ( module.getAttribute( <span class="jxr_string">"cacheFile"</span> ) == <strong class="jxr_keyword">null</strong> )
+<a class="jxr_linenumber" name="L348" href="#L348">348</a> {
+<a class="jxr_linenumber" name="L349" href="#L349">349</a> ( (DefaultConfiguration) module ).addAttribute( <span class="jxr_string">"cacheFile"</span>, request.getCacheFile() );
+<a class="jxr_linenumber" name="L350" href="#L350">350</a> }
+<a class="jxr_linenumber" name="L351" href="#L351">351</a> }
+<a class="jxr_linenumber" name="L352" href="#L352">352</a> <strong class="jxr_keyword">catch</strong> ( CheckstyleException ex )
+<a class="jxr_linenumber" name="L353" href="#L353">353</a> {
+<a class="jxr_linenumber" name="L354" href="#L354">354</a> <em class="jxr_comment">// MCHECKSTYLE-159 - checkstyle 5.4 throws an exception instead of return null if</em>
+<a class="jxr_linenumber" name="L355" href="#L355">355</a> <em class="jxr_comment">// "cacheFile"</em>
+<a class="jxr_linenumber" name="L356" href="#L356">356</a> <em class="jxr_comment">// doesn't exist</em>
+<a class="jxr_linenumber" name="L357" href="#L357">357</a> ( (DefaultConfiguration) module ).addAttribute( <span class="jxr_string">"cacheFile"</span>, request.getCacheFile() );
+<a class="jxr_linenumber" name="L358" href="#L358">358</a> }
<a class="jxr_linenumber" name="L359" href="#L359">359</a> }
-<a class="jxr_linenumber" name="L360" href="#L360">360</a> }
-<a class="jxr_linenumber" name="L361" href="#L361">361</a> }
-<a class="jxr_linenumber" name="L362" href="#L362">362</a> <strong class="jxr_keyword">return</strong> config;
-<a class="jxr_linenumber" name="L363" href="#L363">363</a> }
-<a class="jxr_linenumber" name="L364" href="#L364">364</a> <strong class="jxr_keyword">catch</strong> ( CheckstyleException e )
-<a class="jxr_linenumber" name="L365" href="#L365">365</a> {
-<a class="jxr_linenumber" name="L366" href="#L366">366</a> <strong class="jxr_keyword">throw</strong> <strong class="jxr_keyword">new</strong> <a href="../../../../../org/apache/maven/plugin/checkstyle/CheckstyleExecutorException.html">CheckstyleExecutorException</a>( <span class="jxr_string">"Failed during checkstyle configuration"</span>, e );
+<a class="jxr_linenumber" name="L360" href="#L360">360</a> <strong class="jxr_keyword">else</strong>
+<a class="jxr_linenumber" name="L361" href="#L361">361</a> {
+<a class="jxr_linenumber" name="L362" href="#L362">362</a> request.getLog().warn( <span class="jxr_string">"Failed to configure cache file on module "</span> + module );
+<a class="jxr_linenumber" name="L363" href="#L363">363</a> }
+<a class="jxr_linenumber" name="L364" href="#L364">364</a> }
+<a class="jxr_linenumber" name="L365" href="#L365">365</a> }
+<a class="jxr_linenumber" name="L366" href="#L366">366</a> <strong class="jxr_keyword">return</strong> config;
<a class="jxr_linenumber" name="L367" href="#L367">367</a> }
-<a class="jxr_linenumber" name="L368" href="#L368">368</a> }
-<a class="jxr_linenumber" name="L369" href="#L369">369</a>
-<a class="jxr_linenumber" name="L370" href="#L370">370</a> <strong class="jxr_keyword">private</strong> <strong class="jxr_keyword">void</strong> prepareCheckstylePaths( CheckstyleExecutorRequest request, MavenProject project,
-<a class="jxr_linenumber" name="L371" href="#L371">371</a> List<String> classPathStrings, List<String> outputDirectories,
-<a class="jxr_linenumber" name="L372" href="#L372">372</a> File sourceDirectory, File testSourceDirectory )
-<a class="jxr_linenumber" name="L373" href="#L373">373</a> <strong class="jxr_keyword">throws</strong> <a href="../../../../../org/apache/maven/plugin/checkstyle/CheckstyleExecutorException.html">CheckstyleExecutorException</a>
-<a class="jxr_linenumber" name="L374" href="#L374">374</a> {
-<a class="jxr_linenumber" name="L375" href="#L375">375</a> <strong class="jxr_keyword">try</strong>
-<a class="jxr_linenumber" name="L376" href="#L376">376</a> {
-<a class="jxr_linenumber" name="L377" href="#L377">377</a> outputDirectories.add( project.getBuild().getOutputDirectory() );
-<a class="jxr_linenumber" name="L378" href="#L378">378</a>
-<a class="jxr_linenumber" name="L379" href="#L379">379</a> <strong class="jxr_keyword">if</strong> ( request.isIncludeTestSourceDirectory() && ( testSourceDirectory != <strong class="jxr_keyword">null</strong> )
-<a class="jxr_linenumber" name="L380" href="#L380">380</a> && ( testSourceDirectory.exists() ) && ( testSourceDirectory.isDirectory() ) )
-<a class="jxr_linenumber" name="L381" href="#L381">381</a> {
-<a class="jxr_linenumber" name="L382" href="#L382">382</a> classPathStrings.addAll( project.getTestClasspathElements() );
-<a class="jxr_linenumber" name="L383" href="#L383">383</a> outputDirectories.add( project.getBuild().getTestOutputDirectory() );
-<a class="jxr_linenumber" name="L384" href="#L384">384</a> }
-<a class="jxr_linenumber" name="L385" href="#L385">385</a> <strong class="jxr_keyword">else</strong>
-<a class="jxr_linenumber" name="L386" href="#L386">386</a> {
-<a class="jxr_linenumber" name="L387" href="#L387">387</a> classPathStrings.addAll( project.getCompileClasspathElements() );
+<a class="jxr_linenumber" name="L368" href="#L368">368</a> <strong class="jxr_keyword">catch</strong> ( CheckstyleException e )
+<a class="jxr_linenumber" name="L369" href="#L369">369</a> {
+<a class="jxr_linenumber" name="L370" href="#L370">370</a> <strong class="jxr_keyword">throw</strong> <strong class="jxr_keyword">new</strong> <a href="../../../../../org/apache/maven/plugin/checkstyle/CheckstyleExecutorException.html">CheckstyleExecutorException</a>( <span class="jxr_string">"Failed during checkstyle configuration"</span>, e );
+<a class="jxr_linenumber" name="L371" href="#L371">371</a> }
+<a class="jxr_linenumber" name="L372" href="#L372">372</a> }
+<a class="jxr_linenumber" name="L373" href="#L373">373</a>
+<a class="jxr_linenumber" name="L374" href="#L374">374</a> <strong class="jxr_keyword">private</strong> <strong class="jxr_keyword">void</strong> prepareCheckstylePaths( CheckstyleExecutorRequest request, MavenProject project,
+<a class="jxr_linenumber" name="L375" href="#L375">375</a> List<String> classPathStrings, List<String> outputDirectories,
+<a class="jxr_linenumber" name="L376" href="#L376">376</a> File sourceDirectory, File testSourceDirectory )
+<a class="jxr_linenumber" name="L377" href="#L377">377</a> <strong class="jxr_keyword">throws</strong> <a href="../../../../../org/apache/maven/plugin/checkstyle/CheckstyleExecutorException.html">CheckstyleExecutorException</a>
+<a class="jxr_linenumber" name="L378" href="#L378">378</a> {
+<a class="jxr_linenumber" name="L379" href="#L379">379</a> <strong class="jxr_keyword">try</strong>
+<a class="jxr_linenumber" name="L380" href="#L380">380</a> {
+<a class="jxr_linenumber" name="L381" href="#L381">381</a> outputDirectories.add( project.getBuild().getOutputDirectory() );
+<a class="jxr_linenumber" name="L382" href="#L382">382</a>
+<a class="jxr_linenumber" name="L383" href="#L383">383</a> <strong class="jxr_keyword">if</strong> ( request.isIncludeTestSourceDirectory() && ( testSourceDirectory != <strong class="jxr_keyword">null</strong> )
+<a class="jxr_linenumber" name="L384" href="#L384">384</a> && ( testSourceDirectory.exists() ) && ( testSourceDirectory.isDirectory() ) )
+<a class="jxr_linenumber" name="L385" href="#L385">385</a> {
+<a class="jxr_linenumber" name="L386" href="#L386">386</a> classPathStrings.addAll( project.getTestClasspathElements() );
+<a class="jxr_linenumber" name="L387" href="#L387">387</a> outputDirectories.add( project.getBuild().getTestOutputDirectory() );
<a class="jxr_linenumber" name="L388" href="#L388">388</a> }
-<a class="jxr_linenumber" name="L389" href="#L389">389</a> }
-<a class="jxr_linenumber" name="L390" href="#L390">390</a> <strong class="jxr_keyword">catch</strong> ( DependencyResolutionRequiredException e )
-<a class="jxr_linenumber" name="L391" href="#L391">391</a> {
-<a class="jxr_linenumber" name="L392" href="#L392">392</a> <strong class="jxr_keyword">throw</strong> <strong class="jxr_keyword">new</strong> <a href="../../../../../org/apache/maven/plugin/checkstyle/CheckstyleExecutorException.html">CheckstyleExecutorException</a>( e.getMessage(), e );
+<a class="jxr_linenumber" name="L389" href="#L389">389</a> <strong class="jxr_keyword">else</strong>
+<a class="jxr_linenumber" name="L390" href="#L390">390</a> {
+<a class="jxr_linenumber" name="L391" href="#L391">391</a> classPathStrings.addAll( project.getCompileClasspathElements() );
+<a class="jxr_linenumber" name="L392" href="#L392">392</a> }
[... 665 lines stripped ...]