You are viewing a plain text version of this content. The canonical link for it is here.
Posted to site-commits@maven.apache.org by de...@apache.org on 2015/03/15 20:05:23 UTC
svn commit: r943880 [24/25] - 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/exec/DefaultCheckstyleExecutor.html
==============================================================================
--- websites/production/maven/content/plugins-archives/maven-checkstyle-plugin-LATEST/xref/org/apache/maven/plugin/checkstyle/exec/DefaultCheckstyleExecutor.html (original)
+++ websites/production/maven/content/plugins-archives/maven-checkstyle-plugin-LATEST/xref/org/apache/maven/plugin/checkstyle/exec/DefaultCheckstyleExecutor.html Sun Mar 15 19:05:21 2015
@@ -287,586 +287,624 @@
<a class="jxr_linenumber" name="L279" href="#L279">279</a>
<a class="jxr_linenumber" name="L280" href="#L280">280</a> <strong class="jxr_keyword">if</strong> ( nbErrors > 0 )
<a class="jxr_linenumber" name="L281" href="#L281">281</a> {
-<a class="jxr_linenumber" name="L282" href="#L282">282</a> String message = <span class="jxr_string">"There are "</span> + nbErrors + <span class="jxr_string">" checkstyle errors."</span>;
-<a class="jxr_linenumber" name="L283" href="#L283">283</a>
-<a class="jxr_linenumber" name="L284" href="#L284">284</a> <strong class="jxr_keyword">if</strong> ( request.isFailsOnError() )
-<a class="jxr_linenumber" name="L285" href="#L285">285</a> {
-<a class="jxr_linenumber" name="L286" href="#L286">286</a> <em class="jxr_comment">// TODO: should be a failure, not an error. Report is not meant to</em>
-<a class="jxr_linenumber" name="L287" href="#L287">287</a> <em class="jxr_comment">// throw an exception here (so site would</em>
-<a class="jxr_linenumber" name="L288" href="#L288">288</a> <em class="jxr_comment">// work regardless of config), but should record this information</em>
-<a class="jxr_linenumber" name="L289" href="#L289">289</a> <strong class="jxr_keyword">throw</strong> <strong class="jxr_keyword">new</strong> <a href="../../../../../../org/apache/maven/plugin/checkstyle/exec/CheckstyleExecutorException.html">CheckstyleExecutorException</a>( message );
+<a class="jxr_linenumber" name="L282" href="#L282">282</a> StringBuffer message = <strong class="jxr_keyword">new</strong> StringBuffer( <span class="jxr_string">"There "</span> );
+<a class="jxr_linenumber" name="L283" href="#L283">283</a> <strong class="jxr_keyword">if</strong> ( nbErrors == 1 )
+<a class="jxr_linenumber" name="L284" href="#L284">284</a> {
+<a class="jxr_linenumber" name="L285" href="#L285">285</a> message.append( <span class="jxr_string">"is"</span> );
+<a class="jxr_linenumber" name="L286" href="#L286">286</a> }
+<a class="jxr_linenumber" name="L287" href="#L287">287</a> <strong class="jxr_keyword">else</strong>
+<a class="jxr_linenumber" name="L288" href="#L288">288</a> {
+<a class="jxr_linenumber" name="L289" href="#L289">289</a> message.append( <span class="jxr_string">"are"</span> );
<a class="jxr_linenumber" name="L290" href="#L290">290</a> }
-<a class="jxr_linenumber" name="L291" href="#L291">291</a> <strong class="jxr_keyword">else</strong>
-<a class="jxr_linenumber" name="L292" href="#L292">292</a> {
-<a class="jxr_linenumber" name="L293" href="#L293">293</a> getLogger().info( message );
-<a class="jxr_linenumber" name="L294" href="#L294">294</a> }
-<a class="jxr_linenumber" name="L295" href="#L295">295</a> }
-<a class="jxr_linenumber" name="L296" href="#L296">296</a>
-<a class="jxr_linenumber" name="L297" href="#L297">297</a> <strong class="jxr_keyword">return</strong> checkerListener.getResults();
-<a class="jxr_linenumber" name="L298" href="#L298">298</a> }
-<a class="jxr_linenumber" name="L299" href="#L299">299</a>
-<a class="jxr_linenumber" name="L300" href="#L300">300</a> <strong class="jxr_keyword">protected</strong> <strong class="jxr_keyword">void</strong> addSourceDirectory( CheckstyleCheckerListener sinkListener, Collection<File> sourceDirectories,
-<a class="jxr_linenumber" name="L301" href="#L301">301</a> Collection<File> testSourceDirectories, List<Resource> resources,
-<a class="jxr_linenumber" name="L302" href="#L302">302</a> <a href="../../../../../../org/apache/maven/plugin/checkstyle/exec/CheckstyleExecutorRequest.html">CheckstyleExecutorRequest</a> request )
-<a class="jxr_linenumber" name="L303" href="#L303">303</a> {
-<a class="jxr_linenumber" name="L304" href="#L304">304</a> <strong class="jxr_keyword">if</strong> ( sourceDirectories != <strong class="jxr_keyword">null</strong> )
-<a class="jxr_linenumber" name="L305" href="#L305">305</a> {
-<a class="jxr_linenumber" name="L306" href="#L306">306</a> <strong class="jxr_keyword">for</strong> ( File sourceDirectory : sourceDirectories )
-<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> ( sourceDirectory.exists() )
-<a class="jxr_linenumber" name="L309" href="#L309">309</a> {
-<a class="jxr_linenumber" name="L310" href="#L310">310</a> sinkListener.addSourceDirectory( sourceDirectory );
-<a class="jxr_linenumber" name="L311" href="#L311">311</a> }
-<a class="jxr_linenumber" name="L312" href="#L312">312</a> }
-<a class="jxr_linenumber" name="L313" href="#L313">313</a> }
-<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> ( request.isIncludeTestSourceDirectory() && ( testSourceDirectories != <strong class="jxr_keyword">null</strong> ) )
-<a class="jxr_linenumber" name="L316" href="#L316">316</a> {
-<a class="jxr_linenumber" name="L317" href="#L317">317</a> <strong class="jxr_keyword">for</strong> ( File testSourceDirectory : testSourceDirectories )
-<a class="jxr_linenumber" name="L318" href="#L318">318</a> {
-<a class="jxr_linenumber" name="L319" href="#L319">319</a> <strong class="jxr_keyword">if</strong> ( testSourceDirectory.isDirectory() )
-<a class="jxr_linenumber" name="L320" href="#L320">320</a> {
-<a class="jxr_linenumber" name="L321" href="#L321">321</a> sinkListener.addSourceDirectory( testSourceDirectory );
-<a class="jxr_linenumber" name="L322" href="#L322">322</a> }
-<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>
-<a class="jxr_linenumber" name="L326" href="#L326">326</a> <strong class="jxr_keyword">if</strong> ( resources != <strong class="jxr_keyword">null</strong> )
-<a class="jxr_linenumber" name="L327" href="#L327">327</a> {
-<a class="jxr_linenumber" name="L328" href="#L328">328</a> <strong class="jxr_keyword">for</strong> ( Resource resource : resources )
-<a class="jxr_linenumber" name="L329" href="#L329">329</a> {
-<a class="jxr_linenumber" name="L330" href="#L330">330</a> <strong class="jxr_keyword">if</strong> ( resource.getDirectory() != <strong class="jxr_keyword">null</strong> )
-<a class="jxr_linenumber" name="L331" href="#L331">331</a> {
-<a class="jxr_linenumber" name="L332" href="#L332">332</a> File resourcesDirectory = <strong class="jxr_keyword">new</strong> File( resource.getDirectory() );
-<a class="jxr_linenumber" name="L333" href="#L333">333</a> <strong class="jxr_keyword">if</strong> ( resourcesDirectory.exists() && resourcesDirectory.isDirectory() )
-<a class="jxr_linenumber" name="L334" href="#L334">334</a> {
-<a class="jxr_linenumber" name="L335" href="#L335">335</a> sinkListener.addSourceDirectory( resourcesDirectory );
-<a class="jxr_linenumber" name="L336" href="#L336">336</a> getLogger().debug( <span class="jxr_string">"Added '"</span> + resourcesDirectory.getAbsolutePath()
-<a class="jxr_linenumber" name="L337" href="#L337">337</a> + <span class="jxr_string">"' as a source directory."</span> );
-<a class="jxr_linenumber" name="L338" href="#L338">338</a> }
-<a class="jxr_linenumber" name="L339" href="#L339">339</a> }
-<a class="jxr_linenumber" name="L340" href="#L340">340</a> }
-<a class="jxr_linenumber" name="L341" href="#L341">341</a> }
-<a class="jxr_linenumber" name="L342" href="#L342">342</a> }
-<a class="jxr_linenumber" name="L343" href="#L343">343</a>
-<a class="jxr_linenumber" name="L344" href="#L344">344</a> <strong class="jxr_keyword">public</strong> Configuration getConfiguration( CheckstyleExecutorRequest request )
-<a class="jxr_linenumber" name="L345" href="#L345">345</a> <strong class="jxr_keyword">throws</strong> <a href="../../../../../../org/apache/maven/plugin/checkstyle/exec/CheckstyleExecutorException.html">CheckstyleExecutorException</a>
-<a class="jxr_linenumber" name="L346" href="#L346">346</a> {
-<a class="jxr_linenumber" name="L347" href="#L347">347</a> <strong class="jxr_keyword">try</strong>
-<a class="jxr_linenumber" name="L348" href="#L348">348</a> {
-<a class="jxr_linenumber" name="L349" href="#L349">349</a> <em class="jxr_comment">// Checkstyle will always use the context classloader in order</em>
-<a class="jxr_linenumber" name="L350" href="#L350">350</a> <em class="jxr_comment">// to load resources (dtds),</em>
-<a class="jxr_linenumber" name="L351" href="#L351">351</a> <em class="jxr_comment">// so we have to fix it</em>
-<a class="jxr_linenumber" name="L352" href="#L352">352</a> ClassLoader checkstyleClassLoader = PackageNamesLoader.<strong class="jxr_keyword">class</strong>.getClassLoader();
-<a class="jxr_linenumber" name="L353" href="#L353">353</a> Thread.currentThread().setContextClassLoader( checkstyleClassLoader );
-<a class="jxr_linenumber" name="L354" href="#L354">354</a> String configFile = getConfigFile( request );
-<a class="jxr_linenumber" name="L355" href="#L355">355</a> Properties overridingProperties = getOverridingProperties( request );
-<a class="jxr_linenumber" name="L356" href="#L356">356</a> Configuration config = ConfigurationLoader
-<a class="jxr_linenumber" name="L357" href="#L357">357</a> .loadConfiguration( configFile, <strong class="jxr_keyword">new</strong> PropertiesExpander( overridingProperties ) );
-<a class="jxr_linenumber" name="L358" href="#L358">358</a> String effectiveEncoding = StringUtils.isNotEmpty( request.getEncoding() ) ? request.getEncoding() : System
-<a class="jxr_linenumber" name="L359" href="#L359">359</a> .getProperty( <span class="jxr_string">"file.encoding"</span>, <span class="jxr_string">"UTF-8"</span> );
-<a class="jxr_linenumber" name="L360" href="#L360">360</a>
-<a class="jxr_linenumber" name="L361" href="#L361">361</a> <strong class="jxr_keyword">if</strong> ( StringUtils.isEmpty( request.getEncoding() ) )
-<a class="jxr_linenumber" name="L362" href="#L362">362</a> {
-<a class="jxr_linenumber" name="L363" href="#L363">363</a> getLogger().warn( <span class="jxr_string">"File encoding has not been set, using platform encoding "</span> + effectiveEncoding
-<a class="jxr_linenumber" name="L364" href="#L364">364</a> + <span class="jxr_string">", i.e. build is platform dependent!"</span> );
+<a class="jxr_linenumber" name="L291" href="#L291">291</a> message.append( <span class="jxr_string">" "</span> );
+<a class="jxr_linenumber" name="L292" href="#L292">292</a> message.append( nbErrors );
+<a class="jxr_linenumber" name="L293" href="#L293">293</a> message.append( <span class="jxr_string">" error"</span> );
+<a class="jxr_linenumber" name="L294" href="#L294">294</a> <strong class="jxr_keyword">if</strong> ( nbErrors != 1 )
+<a class="jxr_linenumber" name="L295" href="#L295">295</a> {
+<a class="jxr_linenumber" name="L296" href="#L296">296</a> message.append( <span class="jxr_string">"s"</span> );
+<a class="jxr_linenumber" name="L297" href="#L297">297</a> }
+<a class="jxr_linenumber" name="L298" href="#L298">298</a> message.append( <span class="jxr_string">" reported by Checkstyle"</span> );
+<a class="jxr_linenumber" name="L299" href="#L299">299</a> String version = getCheckstyleVersion();
+<a class="jxr_linenumber" name="L300" href="#L300">300</a> <strong class="jxr_keyword">if</strong> ( version != <strong class="jxr_keyword">null</strong> )
+<a class="jxr_linenumber" name="L301" href="#L301">301</a> {
+<a class="jxr_linenumber" name="L302" href="#L302">302</a> message.append( <span class="jxr_string">" "</span> );
+<a class="jxr_linenumber" name="L303" href="#L303">303</a> message.append( version );
+<a class="jxr_linenumber" name="L304" href="#L304">304</a> }
+<a class="jxr_linenumber" name="L305" href="#L305">305</a> message.append( <span class="jxr_string">" with "</span> );
+<a class="jxr_linenumber" name="L306" href="#L306">306</a> message.append( request.getConfigLocation() );
+<a class="jxr_linenumber" name="L307" href="#L307">307</a> message.append( <span class="jxr_string">" ruleset."</span> );
+<a class="jxr_linenumber" name="L308" href="#L308">308</a>
+<a class="jxr_linenumber" name="L309" href="#L309">309</a> <strong class="jxr_keyword">if</strong> ( request.isFailsOnError() )
+<a class="jxr_linenumber" name="L310" href="#L310">310</a> {
+<a class="jxr_linenumber" name="L311" href="#L311">311</a> <em class="jxr_comment">// TODO: should be a failure, not an error. Report is not meant to</em>
+<a class="jxr_linenumber" name="L312" href="#L312">312</a> <em class="jxr_comment">// throw an exception here (so site would</em>
+<a class="jxr_linenumber" name="L313" href="#L313">313</a> <em class="jxr_comment">// work regardless of config), but should record this information</em>
+<a class="jxr_linenumber" name="L314" href="#L314">314</a> <strong class="jxr_keyword">throw</strong> <strong class="jxr_keyword">new</strong> <a href="../../../../../../org/apache/maven/plugin/checkstyle/exec/CheckstyleExecutorException.html">CheckstyleExecutorException</a>( message.toString() );
+<a class="jxr_linenumber" name="L315" href="#L315">315</a> }
+<a class="jxr_linenumber" name="L316" href="#L316">316</a> <strong class="jxr_keyword">else</strong>
+<a class="jxr_linenumber" name="L317" href="#L317">317</a> {
+<a class="jxr_linenumber" name="L318" href="#L318">318</a> getLogger().info( message.toString() );
+<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>
+<a class="jxr_linenumber" name="L322" href="#L322">322</a> <strong class="jxr_keyword">return</strong> checkerListener.getResults();
+<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">protected</strong> <strong class="jxr_keyword">void</strong> addSourceDirectory( CheckstyleCheckerListener sinkListener, Collection<File> sourceDirectories,
+<a class="jxr_linenumber" name="L326" href="#L326">326</a> Collection<File> testSourceDirectories, List<Resource> resources,
+<a class="jxr_linenumber" name="L327" href="#L327">327</a> <a href="../../../../../../org/apache/maven/plugin/checkstyle/exec/CheckstyleExecutorRequest.html">CheckstyleExecutorRequest</a> request )
+<a class="jxr_linenumber" name="L328" href="#L328">328</a> {
+<a class="jxr_linenumber" name="L329" href="#L329">329</a> <strong class="jxr_keyword">if</strong> ( sourceDirectories != <strong class="jxr_keyword">null</strong> )
+<a class="jxr_linenumber" name="L330" href="#L330">330</a> {
+<a class="jxr_linenumber" name="L331" href="#L331">331</a> <strong class="jxr_keyword">for</strong> ( File sourceDirectory : sourceDirectories )
+<a class="jxr_linenumber" name="L332" href="#L332">332</a> {
+<a class="jxr_linenumber" name="L333" href="#L333">333</a> <strong class="jxr_keyword">if</strong> ( sourceDirectory.exists() )
+<a class="jxr_linenumber" name="L334" href="#L334">334</a> {
+<a class="jxr_linenumber" name="L335" href="#L335">335</a> sinkListener.addSourceDirectory( sourceDirectory );
+<a class="jxr_linenumber" name="L336" href="#L336">336</a> }
+<a class="jxr_linenumber" name="L337" href="#L337">337</a> }
+<a class="jxr_linenumber" name="L338" href="#L338">338</a> }
+<a class="jxr_linenumber" name="L339" href="#L339">339</a>
+<a class="jxr_linenumber" name="L340" href="#L340">340</a> <strong class="jxr_keyword">if</strong> ( request.isIncludeTestSourceDirectory() && ( testSourceDirectories != <strong class="jxr_keyword">null</strong> ) )
+<a class="jxr_linenumber" name="L341" href="#L341">341</a> {
+<a class="jxr_linenumber" name="L342" href="#L342">342</a> <strong class="jxr_keyword">for</strong> ( File testSourceDirectory : testSourceDirectories )
+<a class="jxr_linenumber" name="L343" href="#L343">343</a> {
+<a class="jxr_linenumber" name="L344" href="#L344">344</a> <strong class="jxr_keyword">if</strong> ( testSourceDirectory.isDirectory() )
+<a class="jxr_linenumber" name="L345" href="#L345">345</a> {
+<a class="jxr_linenumber" name="L346" href="#L346">346</a> sinkListener.addSourceDirectory( testSourceDirectory );
+<a class="jxr_linenumber" name="L347" href="#L347">347</a> }
+<a class="jxr_linenumber" name="L348" href="#L348">348</a> }
+<a class="jxr_linenumber" name="L349" href="#L349">349</a> }
+<a class="jxr_linenumber" name="L350" href="#L350">350</a>
+<a class="jxr_linenumber" name="L351" href="#L351">351</a> <strong class="jxr_keyword">if</strong> ( resources != <strong class="jxr_keyword">null</strong> )
+<a class="jxr_linenumber" name="L352" href="#L352">352</a> {
+<a class="jxr_linenumber" name="L353" href="#L353">353</a> <strong class="jxr_keyword">for</strong> ( Resource resource : resources )
+<a class="jxr_linenumber" name="L354" href="#L354">354</a> {
+<a class="jxr_linenumber" name="L355" href="#L355">355</a> <strong class="jxr_keyword">if</strong> ( resource.getDirectory() != <strong class="jxr_keyword">null</strong> )
+<a class="jxr_linenumber" name="L356" href="#L356">356</a> {
+<a class="jxr_linenumber" name="L357" href="#L357">357</a> File resourcesDirectory = <strong class="jxr_keyword">new</strong> File( resource.getDirectory() );
+<a class="jxr_linenumber" name="L358" href="#L358">358</a> <strong class="jxr_keyword">if</strong> ( resourcesDirectory.exists() && resourcesDirectory.isDirectory() )
+<a class="jxr_linenumber" name="L359" href="#L359">359</a> {
+<a class="jxr_linenumber" name="L360" href="#L360">360</a> sinkListener.addSourceDirectory( resourcesDirectory );
+<a class="jxr_linenumber" name="L361" href="#L361">361</a> getLogger().debug( <span class="jxr_string">"Added '"</span> + resourcesDirectory.getAbsolutePath()
+<a class="jxr_linenumber" name="L362" href="#L362">362</a> + <span class="jxr_string">"' as a source directory."</span> );
+<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>
-<a class="jxr_linenumber" name="L367" href="#L367">367</a> <strong class="jxr_keyword">if</strong> ( <span class="jxr_string">"Checker"</span>.equals( config.getName() )
-<a class="jxr_linenumber" name="L368" href="#L368">368</a> || <span class="jxr_string">"com.puppycrawl.tools.checkstyle.Checker"</span>.equals( config.getName() ) )
-<a class="jxr_linenumber" name="L369" href="#L369">369</a> {
-<a class="jxr_linenumber" name="L370" href="#L370">370</a> <strong class="jxr_keyword">if</strong> ( config instanceof DefaultConfiguration )
-<a class="jxr_linenumber" name="L371" href="#L371">371</a> {
-<a class="jxr_linenumber" name="L372" href="#L372">372</a> <em class="jxr_comment">// MCHECKSTYLE-173 Only add the "charset" attribute if it has not been set</em>
-<a class="jxr_linenumber" name="L373" href="#L373">373</a> <strong class="jxr_keyword">try</strong>
-<a class="jxr_linenumber" name="L374" href="#L374">374</a> {
-<a class="jxr_linenumber" name="L375" href="#L375">375</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="L376" href="#L376">376</a> {
-<a class="jxr_linenumber" name="L377" href="#L377">377</a> ( (DefaultConfiguration) config ).addAttribute( <span class="jxr_string">"charset"</span>, effectiveEncoding );
-<a class="jxr_linenumber" name="L378" href="#L378">378</a> }
-<a class="jxr_linenumber" name="L379" href="#L379">379</a> }
-<a class="jxr_linenumber" name="L380" href="#L380">380</a> <strong class="jxr_keyword">catch</strong> ( CheckstyleException ex )
-<a class="jxr_linenumber" name="L381" href="#L381">381</a> {
-<a class="jxr_linenumber" name="L382" href="#L382">382</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="L383" href="#L383">383</a> ( (DefaultConfiguration) config ).addAttribute( <span class="jxr_string">"charset"</span>, effectiveEncoding );
-<a class="jxr_linenumber" name="L384" href="#L384">384</a> }
-<a class="jxr_linenumber" name="L385" href="#L385">385</a> }
-<a class="jxr_linenumber" name="L386" href="#L386">386</a> <strong class="jxr_keyword">else</strong>
-<a class="jxr_linenumber" name="L387" href="#L387">387</a> {
-<a class="jxr_linenumber" name="L388" href="#L388">388</a> getLogger().warn( <span class="jxr_string">"Failed to configure file encoding on module "</span> + config );
-<a class="jxr_linenumber" name="L389" href="#L389">389</a> }
+<a class="jxr_linenumber" name="L366" href="#L366">366</a> }
+<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> <strong class="jxr_keyword">public</strong> Configuration getConfiguration( CheckstyleExecutorRequest request )
+<a class="jxr_linenumber" name="L370" href="#L370">370</a> <strong class="jxr_keyword">throws</strong> <a href="../../../../../../org/apache/maven/plugin/checkstyle/exec/CheckstyleExecutorException.html">CheckstyleExecutorException</a>
+<a class="jxr_linenumber" name="L371" href="#L371">371</a> {
+<a class="jxr_linenumber" name="L372" href="#L372">372</a> <strong class="jxr_keyword">try</strong>
+<a class="jxr_linenumber" name="L373" href="#L373">373</a> {
+<a class="jxr_linenumber" name="L374" href="#L374">374</a> <em class="jxr_comment">// Checkstyle will always use the context classloader in order</em>
+<a class="jxr_linenumber" name="L375" href="#L375">375</a> <em class="jxr_comment">// to load resources (dtds),</em>
+<a class="jxr_linenumber" name="L376" href="#L376">376</a> <em class="jxr_comment">// so we have to fix it</em>
+<a class="jxr_linenumber" name="L377" href="#L377">377</a> ClassLoader checkstyleClassLoader = PackageNamesLoader.<strong class="jxr_keyword">class</strong>.getClassLoader();
+<a class="jxr_linenumber" name="L378" href="#L378">378</a> Thread.currentThread().setContextClassLoader( checkstyleClassLoader );
+<a class="jxr_linenumber" name="L379" href="#L379">379</a> String configFile = getConfigFile( request );
+<a class="jxr_linenumber" name="L380" href="#L380">380</a> Properties overridingProperties = getOverridingProperties( request );
+<a class="jxr_linenumber" name="L381" href="#L381">381</a> Configuration config = ConfigurationLoader
+<a class="jxr_linenumber" name="L382" href="#L382">382</a> .loadConfiguration( configFile, <strong class="jxr_keyword">new</strong> PropertiesExpander( overridingProperties ) );
+<a class="jxr_linenumber" name="L383" href="#L383">383</a> String effectiveEncoding = StringUtils.isNotEmpty( request.getEncoding() ) ? request.getEncoding() : System
+<a class="jxr_linenumber" name="L384" href="#L384">384</a> .getProperty( <span class="jxr_string">"file.encoding"</span>, <span class="jxr_string">"UTF-8"</span> );
+<a class="jxr_linenumber" name="L385" href="#L385">385</a>
+<a class="jxr_linenumber" name="L386" href="#L386">386</a> <strong class="jxr_keyword">if</strong> ( StringUtils.isEmpty( request.getEncoding() ) )
+<a class="jxr_linenumber" name="L387" href="#L387">387</a> {
+<a class="jxr_linenumber" name="L388" href="#L388">388</a> getLogger().warn( <span class="jxr_string">"File encoding has not been set, using platform encoding "</span> + effectiveEncoding
+<a class="jxr_linenumber" name="L389" href="#L389">389</a> + <span class="jxr_string">", i.e. build is platform dependent!"</span> );
<a class="jxr_linenumber" name="L390" href="#L390">390</a> }
-<a class="jxr_linenumber" name="L391" href="#L391">391</a> Configuration[] modules = config.getChildren();
-<a class="jxr_linenumber" name="L392" href="#L392">392</a> <strong class="jxr_keyword">for</strong> ( Configuration module : modules )
-<a class="jxr_linenumber" name="L393" href="#L393">393</a> {
-<a class="jxr_linenumber" name="L394" href="#L394">394</a> <strong class="jxr_keyword">if</strong> ( <span class="jxr_string">"TreeWalker"</span>.equals( module.getName() )
-<a class="jxr_linenumber" name="L395" href="#L395">395</a> || <span class="jxr_string">"com.puppycrawl.tools.checkstyle.TreeWalker"</span>.equals( module.getName() ) )
+<a class="jxr_linenumber" name="L391" href="#L391">391</a>
+<a class="jxr_linenumber" name="L392" href="#L392">392</a> <strong class="jxr_keyword">if</strong> ( <span class="jxr_string">"Checker"</span>.equals( config.getName() )
+<a class="jxr_linenumber" name="L393" href="#L393">393</a> || <span class="jxr_string">"com.puppycrawl.tools.checkstyle.Checker"</span>.equals( config.getName() ) )
+<a class="jxr_linenumber" name="L394" href="#L394">394</a> {
+<a class="jxr_linenumber" name="L395" href="#L395">395</a> <strong class="jxr_keyword">if</strong> ( config instanceof DefaultConfiguration )
<a class="jxr_linenumber" name="L396" href="#L396">396</a> {
-<a class="jxr_linenumber" name="L397" href="#L397">397</a> <strong class="jxr_keyword">if</strong> ( module instanceof DefaultConfiguration )
-<a class="jxr_linenumber" name="L398" href="#L398">398</a> {
-<a class="jxr_linenumber" name="L399" href="#L399">399</a> <em class="jxr_comment">// MCHECKSTYLE-132 DefaultConfiguration addAttribute has changed in checkstyle 5.3</em>
-<a class="jxr_linenumber" name="L400" href="#L400">400</a> <strong class="jxr_keyword">try</strong>
+<a class="jxr_linenumber" name="L397" href="#L397">397</a> <em class="jxr_comment">// MCHECKSTYLE-173 Only add the "charset" attribute if it has not been set</em>
+<a class="jxr_linenumber" name="L398" href="#L398">398</a> <strong class="jxr_keyword">try</strong>
+<a class="jxr_linenumber" name="L399" href="#L399">399</a> {
+<a class="jxr_linenumber" name="L400" href="#L400">400</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="L401" href="#L401">401</a> {
-<a class="jxr_linenumber" name="L402" href="#L402">402</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="L403" href="#L403">403</a> {
-<a class="jxr_linenumber" name="L404" href="#L404">404</a> ( (DefaultConfiguration) module ).addAttribute( <span class="jxr_string">"cacheFile"</span>, request.getCacheFile() );
-<a class="jxr_linenumber" name="L405" href="#L405">405</a> }
-<a class="jxr_linenumber" name="L406" href="#L406">406</a> }
-<a class="jxr_linenumber" name="L407" href="#L407">407</a> <strong class="jxr_keyword">catch</strong> ( CheckstyleException ex )
-<a class="jxr_linenumber" name="L408" href="#L408">408</a> {
-<a class="jxr_linenumber" name="L409" href="#L409">409</a> <em class="jxr_comment">// MCHECKSTYLE-159 - checkstyle 5.4 throws an exception instead of return null if</em>
-<a class="jxr_linenumber" name="L410" href="#L410">410</a> <em class="jxr_comment">// "cacheFile"</em>
-<a class="jxr_linenumber" name="L411" href="#L411">411</a> <em class="jxr_comment">// doesn't exist</em>
-<a class="jxr_linenumber" name="L412" href="#L412">412</a> ( (DefaultConfiguration) module ).addAttribute( <span class="jxr_string">"cacheFile"</span>, request.getCacheFile() );
-<a class="jxr_linenumber" name="L413" href="#L413">413</a> }
-<a class="jxr_linenumber" name="L414" href="#L414">414</a> }
-<a class="jxr_linenumber" name="L415" href="#L415">415</a> <strong class="jxr_keyword">else</strong>
-<a class="jxr_linenumber" name="L416" href="#L416">416</a> {
-<a class="jxr_linenumber" name="L417" href="#L417">417</a> getLogger().warn( <span class="jxr_string">"Failed to configure cache file on module "</span> + module );
-<a class="jxr_linenumber" name="L418" href="#L418">418</a> }
-<a class="jxr_linenumber" name="L419" href="#L419">419</a> }
-<a class="jxr_linenumber" name="L420" href="#L420">420</a> }
-<a class="jxr_linenumber" name="L421" href="#L421">421</a> <strong class="jxr_keyword">return</strong> config;
-<a class="jxr_linenumber" name="L422" href="#L422">422</a> }
-<a class="jxr_linenumber" name="L423" href="#L423">423</a> <strong class="jxr_keyword">catch</strong> ( CheckstyleException e )
-<a class="jxr_linenumber" name="L424" href="#L424">424</a> {
-<a class="jxr_linenumber" name="L425" href="#L425">425</a> <strong class="jxr_keyword">throw</strong> <strong class="jxr_keyword">new</strong> <a href="../../../../../../org/apache/maven/plugin/checkstyle/exec/CheckstyleExecutorException.html">CheckstyleExecutorException</a>( <span class="jxr_string">"Failed during checkstyle configuration"</span>, e );
-<a class="jxr_linenumber" name="L426" href="#L426">426</a> }
-<a class="jxr_linenumber" name="L427" href="#L427">427</a> }
-<a class="jxr_linenumber" name="L428" href="#L428">428</a>
-<a class="jxr_linenumber" name="L429" href="#L429">429</a> <strong class="jxr_keyword">private</strong> <strong class="jxr_keyword">void</strong> prepareCheckstylePaths( CheckstyleExecutorRequest request, MavenProject project,
-<a class="jxr_linenumber" name="L430" href="#L430">430</a> List<String> classPathStrings, List<String> outputDirectories,
-<a class="jxr_linenumber" name="L431" href="#L431">431</a> Collection<File> sourceDirectories, Collection<File> testSourceDirectories )
-<a class="jxr_linenumber" name="L432" href="#L432">432</a> <strong class="jxr_keyword">throws</strong> <a href="../../../../../../org/apache/maven/plugin/checkstyle/exec/CheckstyleExecutorException.html">CheckstyleExecutorException</a>
-<a class="jxr_linenumber" name="L433" href="#L433">433</a> {
-<a class="jxr_linenumber" name="L434" href="#L434">434</a> <strong class="jxr_keyword">try</strong>
-<a class="jxr_linenumber" name="L435" href="#L435">435</a> {
-<a class="jxr_linenumber" name="L436" href="#L436">436</a> outputDirectories.add( project.getBuild().getOutputDirectory() );
-<a class="jxr_linenumber" name="L437" href="#L437">437</a>
-<a class="jxr_linenumber" name="L438" href="#L438">438</a> <strong class="jxr_keyword">if</strong> ( request.isIncludeTestSourceDirectory() && ( testSourceDirectories != <strong class="jxr_keyword">null</strong> )
-<a class="jxr_linenumber" name="L439" href="#L439">439</a> && anyDirectoryExists( testSourceDirectories ) )
-<a class="jxr_linenumber" name="L440" href="#L440">440</a> {
-<a class="jxr_linenumber" name="L441" href="#L441">441</a> classPathStrings.addAll( project.getTestClasspathElements() );
-<a class="jxr_linenumber" name="L442" href="#L442">442</a> outputDirectories.add( project.getBuild().getTestOutputDirectory() );
-<a class="jxr_linenumber" name="L443" href="#L443">443</a> }
-<a class="jxr_linenumber" name="L444" href="#L444">444</a> <strong class="jxr_keyword">else</strong>
-<a class="jxr_linenumber" name="L445" href="#L445">445</a> {
-<a class="jxr_linenumber" name="L446" href="#L446">446</a> classPathStrings.addAll( project.getCompileClasspathElements() );
-<a class="jxr_linenumber" name="L447" href="#L447">447</a> }
-<a class="jxr_linenumber" name="L448" href="#L448">448</a> }
-<a class="jxr_linenumber" name="L449" href="#L449">449</a> <strong class="jxr_keyword">catch</strong> ( DependencyResolutionRequiredException e )
-<a class="jxr_linenumber" name="L450" href="#L450">450</a> {
-<a class="jxr_linenumber" name="L451" href="#L451">451</a> <strong class="jxr_keyword">throw</strong> <strong class="jxr_keyword">new</strong> <a href="../../../../../../org/apache/maven/plugin/checkstyle/exec/CheckstyleExecutorException.html">CheckstyleExecutorException</a>( e.getMessage(), e );
-<a class="jxr_linenumber" name="L452" href="#L452">452</a> }
-<a class="jxr_linenumber" name="L453" href="#L453">453</a> }
-<a class="jxr_linenumber" name="L454" href="#L454">454</a>
-<a class="jxr_linenumber" name="L455" href="#L455">455</a> <strong class="jxr_keyword">private</strong> <strong class="jxr_keyword">boolean</strong> anyDirectoryExists( Collection<File> files )
-<a class="jxr_linenumber" name="L456" href="#L456">456</a> {
-<a class="jxr_linenumber" name="L457" href="#L457">457</a> <strong class="jxr_keyword">for</strong> ( File file : files )
-<a class="jxr_linenumber" name="L458" href="#L458">458</a> {
-<a class="jxr_linenumber" name="L459" href="#L459">459</a> <strong class="jxr_keyword">if</strong> ( file.isDirectory() )
-<a class="jxr_linenumber" name="L460" href="#L460">460</a> {
-<a class="jxr_linenumber" name="L461" href="#L461">461</a> <strong class="jxr_keyword">return</strong> <strong class="jxr_keyword">true</strong>;
-<a class="jxr_linenumber" name="L462" href="#L462">462</a> }
-<a class="jxr_linenumber" name="L463" href="#L463">463</a> }
-<a class="jxr_linenumber" name="L464" href="#L464">464</a> <strong class="jxr_keyword">return</strong> false;
-<a class="jxr_linenumber" name="L465" href="#L465">465</a> }
-<a class="jxr_linenumber" name="L466" href="#L466">466</a>
-<a class="jxr_linenumber" name="L467" href="#L467">467</a> <strong class="jxr_keyword">private</strong> Properties getOverridingProperties( CheckstyleExecutorRequest request )
-<a class="jxr_linenumber" name="L468" href="#L468">468</a> <strong class="jxr_keyword">throws</strong> <a href="../../../../../../org/apache/maven/plugin/checkstyle/exec/CheckstyleExecutorException.html">CheckstyleExecutorException</a>
-<a class="jxr_linenumber" name="L469" href="#L469">469</a> {
-<a class="jxr_linenumber" name="L470" href="#L470">470</a> Properties p = <strong class="jxr_keyword">new</strong> Properties();
-<a class="jxr_linenumber" name="L471" href="#L471">471</a>
-<a class="jxr_linenumber" name="L472" href="#L472">472</a> <strong class="jxr_keyword">try</strong>
-<a class="jxr_linenumber" name="L473" href="#L473">473</a> {
-<a class="jxr_linenumber" name="L474" href="#L474">474</a> <strong class="jxr_keyword">if</strong> ( request.getPropertiesLocation() != <strong class="jxr_keyword">null</strong> )
-<a class="jxr_linenumber" name="L475" href="#L475">475</a> {
-<a class="jxr_linenumber" name="L476" href="#L476">476</a> <strong class="jxr_keyword">if</strong> ( getLogger().isDebugEnabled() )
-<a class="jxr_linenumber" name="L477" href="#L477">477</a> {
-<a class="jxr_linenumber" name="L478" href="#L478">478</a> getLogger().debug( <span class="jxr_string">"request.getPropertiesLocation() "</span> + request.getPropertiesLocation() );
-<a class="jxr_linenumber" name="L479" href="#L479">479</a> }
-<a class="jxr_linenumber" name="L480" href="#L480">480</a>
-<a class="jxr_linenumber" name="L481" href="#L481">481</a> File propertiesFile = locator.getResourceAsFile( request.getPropertiesLocation(),
-<a class="jxr_linenumber" name="L482" href="#L482">482</a> <span class="jxr_string">"checkstyle-checker.properties"</span> );
-<a class="jxr_linenumber" name="L483" href="#L483">483</a>
-<a class="jxr_linenumber" name="L484" href="#L484">484</a> FileInputStream properties = <strong class="jxr_keyword">new</strong> FileInputStream( propertiesFile );
-<a class="jxr_linenumber" name="L485" href="#L485">485</a> <strong class="jxr_keyword">try</strong>
-<a class="jxr_linenumber" name="L486" href="#L486">486</a> {
-<a class="jxr_linenumber" name="L487" href="#L487">487</a> <strong class="jxr_keyword">if</strong> ( propertiesFile != <strong class="jxr_keyword">null</strong> )
-<a class="jxr_linenumber" name="L488" href="#L488">488</a> {
-<a class="jxr_linenumber" name="L489" href="#L489">489</a> p.load( properties );
-<a class="jxr_linenumber" name="L490" href="#L490">490</a> }
-<a class="jxr_linenumber" name="L491" href="#L491">491</a> }
-<a class="jxr_linenumber" name="L492" href="#L492">492</a> <strong class="jxr_keyword">finally</strong>
-<a class="jxr_linenumber" name="L493" href="#L493">493</a> {
-<a class="jxr_linenumber" name="L494" href="#L494">494</a> IOUtils.closeQuietly( properties );
-<a class="jxr_linenumber" name="L495" href="#L495">495</a> }
-<a class="jxr_linenumber" name="L496" href="#L496">496</a> }
-<a class="jxr_linenumber" name="L497" href="#L497">497</a>
-<a class="jxr_linenumber" name="L498" href="#L498">498</a> <strong class="jxr_keyword">if</strong> ( StringUtils.isNotEmpty( request.getPropertyExpansion() ) )
-<a class="jxr_linenumber" name="L499" href="#L499">499</a> {
-<a class="jxr_linenumber" name="L500" href="#L500">500</a> String propertyExpansion = request.getPropertyExpansion();
-<a class="jxr_linenumber" name="L501" href="#L501">501</a> <em class="jxr_comment">// Convert \ to \\, so that p.load will convert it back properly</em>
-<a class="jxr_linenumber" name="L502" href="#L502">502</a> propertyExpansion = StringUtils.replace( propertyExpansion, <span class="jxr_string">"\\"</span>, <span class="jxr_string">"\\\\"</span> );
-<a class="jxr_linenumber" name="L503" href="#L503">503</a> p.load( <strong class="jxr_keyword">new</strong> ByteArrayInputStream( propertyExpansion.getBytes() ) );
-<a class="jxr_linenumber" name="L504" href="#L504">504</a> }
-<a class="jxr_linenumber" name="L505" href="#L505">505</a>
-<a class="jxr_linenumber" name="L506" href="#L506">506</a> <em class="jxr_comment">// Workaround for MCHECKSTYLE-48</em>
-<a class="jxr_linenumber" name="L507" href="#L507">507</a> <em class="jxr_comment">// Make sure that "config/maven-header.txt" is the default value</em>
-<a class="jxr_linenumber" name="L508" href="#L508">508</a> <em class="jxr_comment">// for headerLocation, if configLocation="config/maven_checks.xml"</em>
-<a class="jxr_linenumber" name="L509" href="#L509">509</a> String headerLocation = request.getHeaderLocation();
-<a class="jxr_linenumber" name="L510" href="#L510">510</a> <strong class="jxr_keyword">if</strong> ( <span class="jxr_string">"config/maven_checks.xml"</span>.equals( request.getConfigLocation() ) )
-<a class="jxr_linenumber" name="L511" href="#L511">511</a> {
-<a class="jxr_linenumber" name="L512" href="#L512">512</a>
-<a class="jxr_linenumber" name="L513" href="#L513">513</a> <strong class="jxr_keyword">if</strong> ( <span class="jxr_string">"LICENSE.txt"</span>.equals( request.getHeaderLocation() ) )
-<a class="jxr_linenumber" name="L514" href="#L514">514</a> {
-<a class="jxr_linenumber" name="L515" href="#L515">515</a> headerLocation = <span class="jxr_string">"config/maven-header.txt"</span>;
-<a class="jxr_linenumber" name="L516" href="#L516">516</a> }
-<a class="jxr_linenumber" name="L517" href="#L517">517</a> }
-<a class="jxr_linenumber" name="L518" href="#L518">518</a> <strong class="jxr_keyword">if</strong> ( getLogger().isDebugEnabled() )
-<a class="jxr_linenumber" name="L519" href="#L519">519</a> {
-<a class="jxr_linenumber" name="L520" href="#L520">520</a> getLogger().debug( <span class="jxr_string">"headerLocation "</span> + headerLocation );
-<a class="jxr_linenumber" name="L521" href="#L521">521</a> }
-<a class="jxr_linenumber" name="L522" href="#L522">522</a>
-<a class="jxr_linenumber" name="L523" href="#L523">523</a> <strong class="jxr_keyword">if</strong> ( StringUtils.isNotEmpty( headerLocation ) )
-<a class="jxr_linenumber" name="L524" href="#L524">524</a> {
-<a class="jxr_linenumber" name="L525" href="#L525">525</a> <strong class="jxr_keyword">try</strong>
-<a class="jxr_linenumber" name="L526" href="#L526">526</a> {
-<a class="jxr_linenumber" name="L527" href="#L527">527</a> File headerFile = licenseLocator.getResourceAsFile( headerLocation, <span class="jxr_string">"checkstyle-header.txt"</span> );
-<a class="jxr_linenumber" name="L528" href="#L528">528</a>
-<a class="jxr_linenumber" name="L529" href="#L529">529</a> <strong class="jxr_keyword">if</strong> ( headerFile != <strong class="jxr_keyword">null</strong> )
-<a class="jxr_linenumber" name="L530" href="#L530">530</a> {
-<a class="jxr_linenumber" name="L531" href="#L531">531</a> p.setProperty( <span class="jxr_string">"checkstyle.header.file"</span>, headerFile.getAbsolutePath() );
-<a class="jxr_linenumber" name="L532" href="#L532">532</a> }
+<a class="jxr_linenumber" name="L402" href="#L402">402</a> ( (DefaultConfiguration) config ).addAttribute( <span class="jxr_string">"charset"</span>, effectiveEncoding );
+<a class="jxr_linenumber" name="L403" href="#L403">403</a> }
+<a class="jxr_linenumber" name="L404" href="#L404">404</a> }
+<a class="jxr_linenumber" name="L405" href="#L405">405</a> <strong class="jxr_keyword">catch</strong> ( CheckstyleException ex )
+<a class="jxr_linenumber" name="L406" href="#L406">406</a> {
+<a class="jxr_linenumber" name="L407" href="#L407">407</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="L408" href="#L408">408</a> ( (DefaultConfiguration) config ).addAttribute( <span class="jxr_string">"charset"</span>, effectiveEncoding );
+<a class="jxr_linenumber" name="L409" href="#L409">409</a> }
+<a class="jxr_linenumber" name="L410" href="#L410">410</a> }
+<a class="jxr_linenumber" name="L411" href="#L411">411</a> <strong class="jxr_keyword">else</strong>
+<a class="jxr_linenumber" name="L412" href="#L412">412</a> {
+<a class="jxr_linenumber" name="L413" href="#L413">413</a> getLogger().warn( <span class="jxr_string">"Failed to configure file encoding on module "</span> + config );
+<a class="jxr_linenumber" name="L414" href="#L414">414</a> }
+<a class="jxr_linenumber" name="L415" href="#L415">415</a> }
+<a class="jxr_linenumber" name="L416" href="#L416">416</a> Configuration[] modules = config.getChildren();
+<a class="jxr_linenumber" name="L417" href="#L417">417</a> <strong class="jxr_keyword">for</strong> ( Configuration module : modules )
+<a class="jxr_linenumber" name="L418" href="#L418">418</a> {
+<a class="jxr_linenumber" name="L419" href="#L419">419</a> <strong class="jxr_keyword">if</strong> ( <span class="jxr_string">"TreeWalker"</span>.equals( module.getName() )
+<a class="jxr_linenumber" name="L420" href="#L420">420</a> || <span class="jxr_string">"com.puppycrawl.tools.checkstyle.TreeWalker"</span>.equals( module.getName() ) )
+<a class="jxr_linenumber" name="L421" href="#L421">421</a> {
+<a class="jxr_linenumber" name="L422" href="#L422">422</a> <strong class="jxr_keyword">if</strong> ( module instanceof DefaultConfiguration )
+<a class="jxr_linenumber" name="L423" href="#L423">423</a> {
+<a class="jxr_linenumber" name="L424" href="#L424">424</a> <em class="jxr_comment">// MCHECKSTYLE-132 DefaultConfiguration addAttribute has changed in checkstyle 5.3</em>
+<a class="jxr_linenumber" name="L425" href="#L425">425</a> <strong class="jxr_keyword">try</strong>
+<a class="jxr_linenumber" name="L426" href="#L426">426</a> {
+<a class="jxr_linenumber" name="L427" href="#L427">427</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="L428" href="#L428">428</a> {
+<a class="jxr_linenumber" name="L429" href="#L429">429</a> ( (DefaultConfiguration) module ).addAttribute( <span class="jxr_string">"cacheFile"</span>, request.getCacheFile() );
+<a class="jxr_linenumber" name="L430" href="#L430">430</a> }
+<a class="jxr_linenumber" name="L431" href="#L431">431</a> }
+<a class="jxr_linenumber" name="L432" href="#L432">432</a> <strong class="jxr_keyword">catch</strong> ( CheckstyleException ex )
+<a class="jxr_linenumber" name="L433" href="#L433">433</a> {
+<a class="jxr_linenumber" name="L434" href="#L434">434</a> <em class="jxr_comment">// MCHECKSTYLE-159 - checkstyle 5.4 throws an exception instead of return null if</em>
+<a class="jxr_linenumber" name="L435" href="#L435">435</a> <em class="jxr_comment">// "cacheFile"</em>
+<a class="jxr_linenumber" name="L436" href="#L436">436</a> <em class="jxr_comment">// doesn't exist</em>
+<a class="jxr_linenumber" name="L437" href="#L437">437</a> ( (DefaultConfiguration) module ).addAttribute( <span class="jxr_string">"cacheFile"</span>, request.getCacheFile() );
+<a class="jxr_linenumber" name="L438" href="#L438">438</a> }
+<a class="jxr_linenumber" name="L439" href="#L439">439</a> }
+<a class="jxr_linenumber" name="L440" href="#L440">440</a> <strong class="jxr_keyword">else</strong>
+<a class="jxr_linenumber" name="L441" href="#L441">441</a> {
+<a class="jxr_linenumber" name="L442" href="#L442">442</a> getLogger().warn( <span class="jxr_string">"Failed to configure cache file on module "</span> + module );
+<a class="jxr_linenumber" name="L443" href="#L443">443</a> }
+<a class="jxr_linenumber" name="L444" href="#L444">444</a> }
+<a class="jxr_linenumber" name="L445" href="#L445">445</a> }
+<a class="jxr_linenumber" name="L446" href="#L446">446</a> <strong class="jxr_keyword">return</strong> config;
+<a class="jxr_linenumber" name="L447" href="#L447">447</a> }
+<a class="jxr_linenumber" name="L448" href="#L448">448</a> <strong class="jxr_keyword">catch</strong> ( CheckstyleException e )
+<a class="jxr_linenumber" name="L449" href="#L449">449</a> {
+<a class="jxr_linenumber" name="L450" href="#L450">450</a> <strong class="jxr_keyword">throw</strong> <strong class="jxr_keyword">new</strong> <a href="../../../../../../org/apache/maven/plugin/checkstyle/exec/CheckstyleExecutorException.html">CheckstyleExecutorException</a>( <span class="jxr_string">"Failed during checkstyle configuration"</span>, e );
+<a class="jxr_linenumber" name="L451" href="#L451">451</a> }
+<a class="jxr_linenumber" name="L452" href="#L452">452</a> }
+<a class="jxr_linenumber" name="L453" href="#L453">453</a>
+<a class="jxr_linenumber" name="L454" href="#L454">454</a> <strong class="jxr_keyword">private</strong> <strong class="jxr_keyword">void</strong> prepareCheckstylePaths( CheckstyleExecutorRequest request, MavenProject project,
+<a class="jxr_linenumber" name="L455" href="#L455">455</a> List<String> classPathStrings, List<String> outputDirectories,
+<a class="jxr_linenumber" name="L456" href="#L456">456</a> Collection<File> sourceDirectories, Collection<File> testSourceDirectories )
+<a class="jxr_linenumber" name="L457" href="#L457">457</a> <strong class="jxr_keyword">throws</strong> <a href="../../../../../../org/apache/maven/plugin/checkstyle/exec/CheckstyleExecutorException.html">CheckstyleExecutorException</a>
+<a class="jxr_linenumber" name="L458" href="#L458">458</a> {
+<a class="jxr_linenumber" name="L459" href="#L459">459</a> <strong class="jxr_keyword">try</strong>
+<a class="jxr_linenumber" name="L460" href="#L460">460</a> {
+<a class="jxr_linenumber" name="L461" href="#L461">461</a> outputDirectories.add( project.getBuild().getOutputDirectory() );
+<a class="jxr_linenumber" name="L462" href="#L462">462</a>
+<a class="jxr_linenumber" name="L463" href="#L463">463</a> <strong class="jxr_keyword">if</strong> ( request.isIncludeTestSourceDirectory() && ( testSourceDirectories != <strong class="jxr_keyword">null</strong> )
+<a class="jxr_linenumber" name="L464" href="#L464">464</a> && anyDirectoryExists( testSourceDirectories ) )
+<a class="jxr_linenumber" name="L465" href="#L465">465</a> {
+<a class="jxr_linenumber" name="L466" href="#L466">466</a> classPathStrings.addAll( project.getTestClasspathElements() );
+<a class="jxr_linenumber" name="L467" href="#L467">467</a> outputDirectories.add( project.getBuild().getTestOutputDirectory() );
+<a class="jxr_linenumber" name="L468" href="#L468">468</a> }
+<a class="jxr_linenumber" name="L469" href="#L469">469</a> <strong class="jxr_keyword">else</strong>
+<a class="jxr_linenumber" name="L470" href="#L470">470</a> {
+<a class="jxr_linenumber" name="L471" href="#L471">471</a> classPathStrings.addAll( project.getCompileClasspathElements() );
+<a class="jxr_linenumber" name="L472" href="#L472">472</a> }
+<a class="jxr_linenumber" name="L473" href="#L473">473</a> }
+<a class="jxr_linenumber" name="L474" href="#L474">474</a> <strong class="jxr_keyword">catch</strong> ( DependencyResolutionRequiredException e )
+<a class="jxr_linenumber" name="L475" href="#L475">475</a> {
+<a class="jxr_linenumber" name="L476" href="#L476">476</a> <strong class="jxr_keyword">throw</strong> <strong class="jxr_keyword">new</strong> <a href="../../../../../../org/apache/maven/plugin/checkstyle/exec/CheckstyleExecutorException.html">CheckstyleExecutorException</a>( e.getMessage(), e );
+<a class="jxr_linenumber" name="L477" href="#L477">477</a> }
+<a class="jxr_linenumber" name="L478" href="#L478">478</a> }
+<a class="jxr_linenumber" name="L479" href="#L479">479</a>
+<a class="jxr_linenumber" name="L480" href="#L480">480</a> <strong class="jxr_keyword">private</strong> <strong class="jxr_keyword">boolean</strong> anyDirectoryExists( Collection<File> files )
+<a class="jxr_linenumber" name="L481" href="#L481">481</a> {
+<a class="jxr_linenumber" name="L482" href="#L482">482</a> <strong class="jxr_keyword">for</strong> ( File file : files )
+<a class="jxr_linenumber" name="L483" href="#L483">483</a> {
+<a class="jxr_linenumber" name="L484" href="#L484">484</a> <strong class="jxr_keyword">if</strong> ( file.isDirectory() )
+<a class="jxr_linenumber" name="L485" href="#L485">485</a> {
+<a class="jxr_linenumber" name="L486" href="#L486">486</a> <strong class="jxr_keyword">return</strong> <strong class="jxr_keyword">true</strong>;
+<a class="jxr_linenumber" name="L487" href="#L487">487</a> }
+<a class="jxr_linenumber" name="L488" href="#L488">488</a> }
+<a class="jxr_linenumber" name="L489" href="#L489">489</a> <strong class="jxr_keyword">return</strong> false;
+<a class="jxr_linenumber" name="L490" href="#L490">490</a> }
+<a class="jxr_linenumber" name="L491" href="#L491">491</a>
+<a class="jxr_linenumber" name="L492" href="#L492">492</a> <em class="jxr_javadoccomment">/**</em>
+<a class="jxr_linenumber" name="L493" href="#L493">493</a> <em class="jxr_javadoccomment"> * Get the effective Checkstyle version at runtime.</em>
+<a class="jxr_linenumber" name="L494" href="#L494">494</a> <em class="jxr_javadoccomment"> * @return the MANIFEST implementation version of Checkstyle API package (can be <code>null</code>)</em>
+<a class="jxr_linenumber" name="L495" href="#L495">495</a> <em class="jxr_javadoccomment"> *</em>
+<a class="jxr_linenumber" name="L496" href="#L496">496</a> <em class="jxr_javadoccomment"> *@todo Copied from CheckstyleReportGenerator - move to a utility class</em>
+<a class="jxr_linenumber" name="L497" href="#L497">497</a> <em class="jxr_javadoccomment"> */</em>
+<a class="jxr_linenumber" name="L498" href="#L498">498</a> <strong class="jxr_keyword">private</strong> String getCheckstyleVersion()
+<a class="jxr_linenumber" name="L499" href="#L499">499</a> {
+<a class="jxr_linenumber" name="L500" href="#L500">500</a> Package checkstyleApiPackage = Configuration.<strong class="jxr_keyword">class</strong>.getPackage();
+<a class="jxr_linenumber" name="L501" href="#L501">501</a>
+<a class="jxr_linenumber" name="L502" href="#L502">502</a> <strong class="jxr_keyword">return</strong> ( checkstyleApiPackage == <strong class="jxr_keyword">null</strong> ) ? <strong class="jxr_keyword">null</strong> : checkstyleApiPackage.getImplementationVersion();
+<a class="jxr_linenumber" name="L503" href="#L503">503</a> }
+<a class="jxr_linenumber" name="L504" href="#L504">504</a>
+<a class="jxr_linenumber" name="L505" href="#L505">505</a> <strong class="jxr_keyword">private</strong> Properties getOverridingProperties( CheckstyleExecutorRequest request )
+<a class="jxr_linenumber" name="L506" href="#L506">506</a> <strong class="jxr_keyword">throws</strong> <a href="../../../../../../org/apache/maven/plugin/checkstyle/exec/CheckstyleExecutorException.html">CheckstyleExecutorException</a>
+<a class="jxr_linenumber" name="L507" href="#L507">507</a> {
+<a class="jxr_linenumber" name="L508" href="#L508">508</a> Properties p = <strong class="jxr_keyword">new</strong> Properties();
+<a class="jxr_linenumber" name="L509" href="#L509">509</a>
+<a class="jxr_linenumber" name="L510" href="#L510">510</a> <strong class="jxr_keyword">try</strong>
+<a class="jxr_linenumber" name="L511" href="#L511">511</a> {
+<a class="jxr_linenumber" name="L512" href="#L512">512</a> <strong class="jxr_keyword">if</strong> ( request.getPropertiesLocation() != <strong class="jxr_keyword">null</strong> )
+<a class="jxr_linenumber" name="L513" href="#L513">513</a> {
+<a class="jxr_linenumber" name="L514" href="#L514">514</a> <strong class="jxr_keyword">if</strong> ( getLogger().isDebugEnabled() )
+<a class="jxr_linenumber" name="L515" href="#L515">515</a> {
+<a class="jxr_linenumber" name="L516" href="#L516">516</a> getLogger().debug( <span class="jxr_string">"request.getPropertiesLocation() "</span> + request.getPropertiesLocation() );
+<a class="jxr_linenumber" name="L517" href="#L517">517</a> }
+<a class="jxr_linenumber" name="L518" href="#L518">518</a>
+<a class="jxr_linenumber" name="L519" href="#L519">519</a> File propertiesFile = locator.getResourceAsFile( request.getPropertiesLocation(),
+<a class="jxr_linenumber" name="L520" href="#L520">520</a> <span class="jxr_string">"checkstyle-checker.properties"</span> );
+<a class="jxr_linenumber" name="L521" href="#L521">521</a>
+<a class="jxr_linenumber" name="L522" href="#L522">522</a> FileInputStream properties = <strong class="jxr_keyword">new</strong> FileInputStream( propertiesFile );
+<a class="jxr_linenumber" name="L523" href="#L523">523</a> <strong class="jxr_keyword">try</strong>
+<a class="jxr_linenumber" name="L524" href="#L524">524</a> {
+<a class="jxr_linenumber" name="L525" href="#L525">525</a> <strong class="jxr_keyword">if</strong> ( propertiesFile != <strong class="jxr_keyword">null</strong> )
+<a class="jxr_linenumber" name="L526" href="#L526">526</a> {
+<a class="jxr_linenumber" name="L527" href="#L527">527</a> p.load( properties );
+<a class="jxr_linenumber" name="L528" href="#L528">528</a> }
+<a class="jxr_linenumber" name="L529" href="#L529">529</a> }
+<a class="jxr_linenumber" name="L530" href="#L530">530</a> <strong class="jxr_keyword">finally</strong>
+<a class="jxr_linenumber" name="L531" href="#L531">531</a> {
+<a class="jxr_linenumber" name="L532" href="#L532">532</a> IOUtils.closeQuietly( properties );
<a class="jxr_linenumber" name="L533" href="#L533">533</a> }
-<a class="jxr_linenumber" name="L534" href="#L534">534</a> <strong class="jxr_keyword">catch</strong> ( FileResourceCreationException e )
-<a class="jxr_linenumber" name="L535" href="#L535">535</a> {
-<a class="jxr_linenumber" name="L536" href="#L536">536</a> getLogger().debug( <span class="jxr_string">"Unable to process header location: "</span> + headerLocation );
-<a class="jxr_linenumber" name="L537" href="#L537">537</a> getLogger().debug( <span class="jxr_string">"Checkstyle will throw exception if ${checkstyle.header.file} is used"</span> );
-<a class="jxr_linenumber" name="L538" href="#L538">538</a> }
-<a class="jxr_linenumber" name="L539" href="#L539">539</a> <strong class="jxr_keyword">catch</strong> ( ResourceNotFoundException e )
-<a class="jxr_linenumber" name="L540" href="#L540">540</a> {
-<a class="jxr_linenumber" name="L541" href="#L541">541</a> getLogger().debug( <span class="jxr_string">"Unable to process header location: "</span> + headerLocation );
-<a class="jxr_linenumber" name="L542" href="#L542">542</a> getLogger().debug( <span class="jxr_string">"Checkstyle will throw exception if ${checkstyle.header.file} is used"</span> );
-<a class="jxr_linenumber" name="L543" href="#L543">543</a> }
-<a class="jxr_linenumber" name="L544" href="#L544">544</a> }
-<a class="jxr_linenumber" name="L545" href="#L545">545</a>
-<a class="jxr_linenumber" name="L546" href="#L546">546</a> <strong class="jxr_keyword">if</strong> ( request.getCacheFile() != <strong class="jxr_keyword">null</strong> )
-<a class="jxr_linenumber" name="L547" href="#L547">547</a> {
-<a class="jxr_linenumber" name="L548" href="#L548">548</a> p.setProperty( <span class="jxr_string">"checkstyle.cache.file"</span>, request.getCacheFile() );
-<a class="jxr_linenumber" name="L549" href="#L549">549</a> }
-<a class="jxr_linenumber" name="L550" href="#L550">550</a> }
-<a class="jxr_linenumber" name="L551" href="#L551">551</a> <strong class="jxr_keyword">catch</strong> ( IOException e )
-<a class="jxr_linenumber" name="L552" href="#L552">552</a> {
-<a class="jxr_linenumber" name="L553" href="#L553">553</a> <strong class="jxr_keyword">throw</strong> <strong class="jxr_keyword">new</strong> <a href="../../../../../../org/apache/maven/plugin/checkstyle/exec/CheckstyleExecutorException.html">CheckstyleExecutorException</a>( <span class="jxr_string">"Failed to get overriding properties"</span>, e );
-<a class="jxr_linenumber" name="L554" href="#L554">554</a> }
-<a class="jxr_linenumber" name="L555" href="#L555">555</a> <strong class="jxr_keyword">catch</strong> ( FileResourceCreationException e )
-<a class="jxr_linenumber" name="L556" href="#L556">556</a> {
-<a class="jxr_linenumber" name="L557" href="#L557">557</a> <strong class="jxr_keyword">throw</strong> <strong class="jxr_keyword">new</strong> <a href="../../../../../../org/apache/maven/plugin/checkstyle/exec/CheckstyleExecutorException.html">CheckstyleExecutorException</a>( <span class="jxr_string">"Failed to get overriding properties"</span>, e );
-<a class="jxr_linenumber" name="L558" href="#L558">558</a> }
-<a class="jxr_linenumber" name="L559" href="#L559">559</a> <strong class="jxr_keyword">catch</strong> ( ResourceNotFoundException e )
-<a class="jxr_linenumber" name="L560" href="#L560">560</a> {
-<a class="jxr_linenumber" name="L561" href="#L561">561</a> <strong class="jxr_keyword">throw</strong> <strong class="jxr_keyword">new</strong> <a href="../../../../../../org/apache/maven/plugin/checkstyle/exec/CheckstyleExecutorException.html">CheckstyleExecutorException</a>( <span class="jxr_string">"Failed to get overriding properties"</span>, e );
-<a class="jxr_linenumber" name="L562" href="#L562">562</a> }
-<a class="jxr_linenumber" name="L563" href="#L563">563</a> <strong class="jxr_keyword">if</strong> ( request.getSuppressionsFileExpression() != <strong class="jxr_keyword">null</strong> )
-<a class="jxr_linenumber" name="L564" href="#L564">564</a> {
-<a class="jxr_linenumber" name="L565" href="#L565">565</a> String suppressionsFilePath = getSuppressionsFilePath( request );
+<a class="jxr_linenumber" name="L534" href="#L534">534</a> }
+<a class="jxr_linenumber" name="L535" href="#L535">535</a>
+<a class="jxr_linenumber" name="L536" href="#L536">536</a> <strong class="jxr_keyword">if</strong> ( StringUtils.isNotEmpty( request.getPropertyExpansion() ) )
+<a class="jxr_linenumber" name="L537" href="#L537">537</a> {
+<a class="jxr_linenumber" name="L538" href="#L538">538</a> String propertyExpansion = request.getPropertyExpansion();
+<a class="jxr_linenumber" name="L539" href="#L539">539</a> <em class="jxr_comment">// Convert \ to \\, so that p.load will convert it back properly</em>
+<a class="jxr_linenumber" name="L540" href="#L540">540</a> propertyExpansion = StringUtils.replace( propertyExpansion, <span class="jxr_string">"\\"</span>, <span class="jxr_string">"\\\\"</span> );
+<a class="jxr_linenumber" name="L541" href="#L541">541</a> p.load( <strong class="jxr_keyword">new</strong> ByteArrayInputStream( propertyExpansion.getBytes() ) );
+<a class="jxr_linenumber" name="L542" href="#L542">542</a> }
+<a class="jxr_linenumber" name="L543" href="#L543">543</a>
+<a class="jxr_linenumber" name="L544" href="#L544">544</a> <em class="jxr_comment">// Workaround for MCHECKSTYLE-48</em>
+<a class="jxr_linenumber" name="L545" href="#L545">545</a> <em class="jxr_comment">// Make sure that "config/maven-header.txt" is the default value</em>
+<a class="jxr_linenumber" name="L546" href="#L546">546</a> <em class="jxr_comment">// for headerLocation, if configLocation="config/maven_checks.xml"</em>
+<a class="jxr_linenumber" name="L547" href="#L547">547</a> String headerLocation = request.getHeaderLocation();
+<a class="jxr_linenumber" name="L548" href="#L548">548</a> <strong class="jxr_keyword">if</strong> ( <span class="jxr_string">"config/maven_checks.xml"</span>.equals( request.getConfigLocation() ) )
+<a class="jxr_linenumber" name="L549" href="#L549">549</a> {
+<a class="jxr_linenumber" name="L550" href="#L550">550</a>
+<a class="jxr_linenumber" name="L551" href="#L551">551</a> <strong class="jxr_keyword">if</strong> ( <span class="jxr_string">"LICENSE.txt"</span>.equals( request.getHeaderLocation() ) )
+<a class="jxr_linenumber" name="L552" href="#L552">552</a> {
+<a class="jxr_linenumber" name="L553" href="#L553">553</a> headerLocation = <span class="jxr_string">"config/maven-header.txt"</span>;
+<a class="jxr_linenumber" name="L554" href="#L554">554</a> }
+<a class="jxr_linenumber" name="L555" href="#L555">555</a> }
+<a class="jxr_linenumber" name="L556" href="#L556">556</a> <strong class="jxr_keyword">if</strong> ( getLogger().isDebugEnabled() )
+<a class="jxr_linenumber" name="L557" href="#L557">557</a> {
+<a class="jxr_linenumber" name="L558" href="#L558">558</a> getLogger().debug( <span class="jxr_string">"headerLocation "</span> + headerLocation );
+<a class="jxr_linenumber" name="L559" href="#L559">559</a> }
+<a class="jxr_linenumber" name="L560" href="#L560">560</a>
+<a class="jxr_linenumber" name="L561" href="#L561">561</a> <strong class="jxr_keyword">if</strong> ( StringUtils.isNotEmpty( headerLocation ) )
+<a class="jxr_linenumber" name="L562" href="#L562">562</a> {
+<a class="jxr_linenumber" name="L563" href="#L563">563</a> <strong class="jxr_keyword">try</strong>
+<a class="jxr_linenumber" name="L564" href="#L564">564</a> {
+<a class="jxr_linenumber" name="L565" href="#L565">565</a> File headerFile = licenseLocator.getResourceAsFile( headerLocation, <span class="jxr_string">"checkstyle-header.txt"</span> );
<a class="jxr_linenumber" name="L566" href="#L566">566</a>
-<a class="jxr_linenumber" name="L567" href="#L567">567</a> <strong class="jxr_keyword">if</strong> ( suppressionsFilePath != <strong class="jxr_keyword">null</strong> )
-<a class="jxr_linenumber" name="L568" href="#L568">568</a> {
-<a class="jxr_linenumber" name="L569" href="#L569">569</a> p.setProperty( request.getSuppressionsFileExpression(), suppressionsFilePath );
-<a class="jxr_linenumber" name="L570" href="#L570">570</a> }
-<a class="jxr_linenumber" name="L571" href="#L571">571</a> }
-<a class="jxr_linenumber" name="L572" href="#L572">572</a>
-<a class="jxr_linenumber" name="L573" href="#L573">573</a> <strong class="jxr_keyword">return</strong> p;
-<a class="jxr_linenumber" name="L574" href="#L574">574</a> }
-<a class="jxr_linenumber" name="L575" href="#L575">575</a>
-<a class="jxr_linenumber" name="L576" href="#L576">576</a> <strong class="jxr_keyword">private</strong> List<File> getFilesToProcess( CheckstyleExecutorRequest request )
-<a class="jxr_linenumber" name="L577" href="#L577">577</a> <strong class="jxr_keyword">throws</strong> IOException
-<a class="jxr_linenumber" name="L578" href="#L578">578</a> {
-<a class="jxr_linenumber" name="L579" href="#L579">579</a> StringBuilder excludesStr = <strong class="jxr_keyword">new</strong> StringBuilder();
-<a class="jxr_linenumber" name="L580" href="#L580">580</a>
-<a class="jxr_linenumber" name="L581" href="#L581">581</a> <strong class="jxr_keyword">if</strong> ( StringUtils.isNotEmpty( request.getExcludes() ) )
-<a class="jxr_linenumber" name="L582" href="#L582">582</a> {
-<a class="jxr_linenumber" name="L583" href="#L583">583</a> excludesStr.append( request.getExcludes() );
-<a class="jxr_linenumber" name="L584" href="#L584">584</a> }
-<a class="jxr_linenumber" name="L585" href="#L585">585</a>
-<a class="jxr_linenumber" name="L586" href="#L586">586</a> String[] defaultExcludes = FileUtils.getDefaultExcludes();
-<a class="jxr_linenumber" name="L587" href="#L587">587</a> <strong class="jxr_keyword">for</strong> ( String defaultExclude : defaultExcludes )
-<a class="jxr_linenumber" name="L588" href="#L588">588</a> {
-<a class="jxr_linenumber" name="L589" href="#L589">589</a> <strong class="jxr_keyword">if</strong> ( excludesStr.length() > 0 )
-<a class="jxr_linenumber" name="L590" href="#L590">590</a> {
-<a class="jxr_linenumber" name="L591" href="#L591">591</a> excludesStr.append( <span class="jxr_string">","</span> );
-<a class="jxr_linenumber" name="L592" href="#L592">592</a> }
-<a class="jxr_linenumber" name="L593" href="#L593">593</a>
-<a class="jxr_linenumber" name="L594" href="#L594">594</a> excludesStr.append( defaultExclude );
-<a class="jxr_linenumber" name="L595" href="#L595">595</a> }
-<a class="jxr_linenumber" name="L596" href="#L596">596</a>
-<a class="jxr_linenumber" name="L597" href="#L597">597</a> Set<File> files = <strong class="jxr_keyword">new</strong> LinkedHashSet<File>();
-<a class="jxr_linenumber" name="L598" href="#L598">598</a> <strong class="jxr_keyword">if</strong> ( request.isAggregate() )
-<a class="jxr_linenumber" name="L599" href="#L599">599</a> {
-<a class="jxr_linenumber" name="L600" href="#L600">600</a> <strong class="jxr_keyword">for</strong> ( MavenProject project : request.getReactorProjects() )
-<a class="jxr_linenumber" name="L601" href="#L601">601</a> {
-<a class="jxr_linenumber" name="L602" href="#L602">602</a> Set<File> sourceDirectories = <strong class="jxr_keyword">new</strong> LinkedHashSet<File>();
-<a class="jxr_linenumber" name="L603" href="#L603">603</a>
-<a class="jxr_linenumber" name="L604" href="#L604">604</a> <em class="jxr_comment">// CompileSourceRoots are absolute paths</em>
-<a class="jxr_linenumber" name="L605" href="#L605">605</a> List<String> compileSourceRoots = project.getCompileSourceRoots();
-<a class="jxr_linenumber" name="L606" href="#L606">606</a> <strong class="jxr_keyword">for</strong> ( String compileSourceRoot : compileSourceRoots )
-<a class="jxr_linenumber" name="L607" href="#L607">607</a> {
-<a class="jxr_linenumber" name="L608" href="#L608">608</a> sourceDirectories.add( <strong class="jxr_keyword">new</strong> File( compileSourceRoot ) );
-<a class="jxr_linenumber" name="L609" href="#L609">609</a> }
+<a class="jxr_linenumber" name="L567" href="#L567">567</a> <strong class="jxr_keyword">if</strong> ( headerFile != <strong class="jxr_keyword">null</strong> )
+<a class="jxr_linenumber" name="L568" href="#L568">568</a> {
+<a class="jxr_linenumber" name="L569" href="#L569">569</a> p.setProperty( <span class="jxr_string">"checkstyle.header.file"</span>, headerFile.getAbsolutePath() );
+<a class="jxr_linenumber" name="L570" href="#L570">570</a> }
+<a class="jxr_linenumber" name="L571" href="#L571">571</a> }
+<a class="jxr_linenumber" name="L572" href="#L572">572</a> <strong class="jxr_keyword">catch</strong> ( FileResourceCreationException e )
+<a class="jxr_linenumber" name="L573" href="#L573">573</a> {
+<a class="jxr_linenumber" name="L574" href="#L574">574</a> getLogger().debug( <span class="jxr_string">"Unable to process header location: "</span> + headerLocation );
+<a class="jxr_linenumber" name="L575" href="#L575">575</a> getLogger().debug( <span class="jxr_string">"Checkstyle will throw exception if ${checkstyle.header.file} is used"</span> );
+<a class="jxr_linenumber" name="L576" href="#L576">576</a> }
+<a class="jxr_linenumber" name="L577" href="#L577">577</a> <strong class="jxr_keyword">catch</strong> ( ResourceNotFoundException e )
+<a class="jxr_linenumber" name="L578" href="#L578">578</a> {
+<a class="jxr_linenumber" name="L579" href="#L579">579</a> getLogger().debug( <span class="jxr_string">"Unable to process header location: "</span> + headerLocation );
+<a class="jxr_linenumber" name="L580" href="#L580">580</a> getLogger().debug( <span class="jxr_string">"Checkstyle will throw exception if ${checkstyle.header.file} is used"</span> );
+<a class="jxr_linenumber" name="L581" href="#L581">581</a> }
+<a class="jxr_linenumber" name="L582" href="#L582">582</a> }
+<a class="jxr_linenumber" name="L583" href="#L583">583</a>
+<a class="jxr_linenumber" name="L584" href="#L584">584</a> <strong class="jxr_keyword">if</strong> ( request.getCacheFile() != <strong class="jxr_keyword">null</strong> )
+<a class="jxr_linenumber" name="L585" href="#L585">585</a> {
+<a class="jxr_linenumber" name="L586" href="#L586">586</a> p.setProperty( <span class="jxr_string">"checkstyle.cache.file"</span>, request.getCacheFile() );
+<a class="jxr_linenumber" name="L587" href="#L587">587</a> }
+<a class="jxr_linenumber" name="L588" href="#L588">588</a> }
+<a class="jxr_linenumber" name="L589" href="#L589">589</a> <strong class="jxr_keyword">catch</strong> ( IOException e )
+<a class="jxr_linenumber" name="L590" href="#L590">590</a> {
+<a class="jxr_linenumber" name="L591" href="#L591">591</a> <strong class="jxr_keyword">throw</strong> <strong class="jxr_keyword">new</strong> <a href="../../../../../../org/apache/maven/plugin/checkstyle/exec/CheckstyleExecutorException.html">CheckstyleExecutorException</a>( <span class="jxr_string">"Failed to get overriding properties"</span>, e );
+<a class="jxr_linenumber" name="L592" href="#L592">592</a> }
+<a class="jxr_linenumber" name="L593" href="#L593">593</a> <strong class="jxr_keyword">catch</strong> ( FileResourceCreationException e )
+<a class="jxr_linenumber" name="L594" href="#L594">594</a> {
+<a class="jxr_linenumber" name="L595" href="#L595">595</a> <strong class="jxr_keyword">throw</strong> <strong class="jxr_keyword">new</strong> <a href="../../../../../../org/apache/maven/plugin/checkstyle/exec/CheckstyleExecutorException.html">CheckstyleExecutorException</a>( <span class="jxr_string">"Failed to get overriding properties"</span>, e );
+<a class="jxr_linenumber" name="L596" href="#L596">596</a> }
+<a class="jxr_linenumber" name="L597" href="#L597">597</a> <strong class="jxr_keyword">catch</strong> ( ResourceNotFoundException e )
+<a class="jxr_linenumber" name="L598" href="#L598">598</a> {
+<a class="jxr_linenumber" name="L599" href="#L599">599</a> <strong class="jxr_keyword">throw</strong> <strong class="jxr_keyword">new</strong> <a href="../../../../../../org/apache/maven/plugin/checkstyle/exec/CheckstyleExecutorException.html">CheckstyleExecutorException</a>( <span class="jxr_string">"Failed to get overriding properties"</span>, e );
+<a class="jxr_linenumber" name="L600" href="#L600">600</a> }
+<a class="jxr_linenumber" name="L601" href="#L601">601</a> <strong class="jxr_keyword">if</strong> ( request.getSuppressionsFileExpression() != <strong class="jxr_keyword">null</strong> )
+<a class="jxr_linenumber" name="L602" href="#L602">602</a> {
+<a class="jxr_linenumber" name="L603" href="#L603">603</a> String suppressionsFilePath = getSuppressionsFilePath( request );
+<a class="jxr_linenumber" name="L604" href="#L604">604</a>
+<a class="jxr_linenumber" name="L605" href="#L605">605</a> <strong class="jxr_keyword">if</strong> ( suppressionsFilePath != <strong class="jxr_keyword">null</strong> )
+<a class="jxr_linenumber" name="L606" href="#L606">606</a> {
+<a class="jxr_linenumber" name="L607" href="#L607">607</a> p.setProperty( request.getSuppressionsFileExpression(), suppressionsFilePath );
+<a class="jxr_linenumber" name="L608" href="#L608">608</a> }
+<a class="jxr_linenumber" name="L609" href="#L609">609</a> }
<a class="jxr_linenumber" name="L610" href="#L610">610</a>
-<a class="jxr_linenumber" name="L611" href="#L611">611</a> Set<File> testSourceDirectories = <strong class="jxr_keyword">new</strong> LinkedHashSet<File>();
-<a class="jxr_linenumber" name="L612" href="#L612">612</a> <em class="jxr_comment">// CompileSourceRoots are absolute paths</em>
-<a class="jxr_linenumber" name="L613" href="#L613">613</a> List<String> testCompileSourceRoots = project.getTestCompileSourceRoots();
-<a class="jxr_linenumber" name="L614" href="#L614">614</a> <strong class="jxr_keyword">for</strong> ( String testCompileSourceRoot : testCompileSourceRoots )
-<a class="jxr_linenumber" name="L615" href="#L615">615</a> {
-<a class="jxr_linenumber" name="L616" href="#L616">616</a> testSourceDirectories.add( <strong class="jxr_keyword">new</strong> File( testCompileSourceRoot ) );
-<a class="jxr_linenumber" name="L617" href="#L617">617</a> }
+<a class="jxr_linenumber" name="L611" href="#L611">611</a> <strong class="jxr_keyword">return</strong> p;
+<a class="jxr_linenumber" name="L612" href="#L612">612</a> }
+<a class="jxr_linenumber" name="L613" href="#L613">613</a>
+<a class="jxr_linenumber" name="L614" href="#L614">614</a> <strong class="jxr_keyword">private</strong> List<File> getFilesToProcess( CheckstyleExecutorRequest request )
+<a class="jxr_linenumber" name="L615" href="#L615">615</a> <strong class="jxr_keyword">throws</strong> IOException
+<a class="jxr_linenumber" name="L616" href="#L616">616</a> {
+<a class="jxr_linenumber" name="L617" href="#L617">617</a> StringBuilder excludesStr = <strong class="jxr_keyword">new</strong> StringBuilder();
<a class="jxr_linenumber" name="L618" href="#L618">618</a>
-<a class="jxr_linenumber" name="L619" href="#L619">619</a> addFilesToProcess( request, sourceDirectories, project.getResources(), project.getTestResources(),
-<a class="jxr_linenumber" name="L620" href="#L620">620</a> files, testSourceDirectories );
-<a class="jxr_linenumber" name="L621" href="#L621">621</a> }
+<a class="jxr_linenumber" name="L619" href="#L619">619</a> <strong class="jxr_keyword">if</strong> ( StringUtils.isNotEmpty( request.getExcludes() ) )
+<a class="jxr_linenumber" name="L620" href="#L620">620</a> {
+<a class="jxr_linenumber" name="L621" href="#L621">621</a> excludesStr.append( request.getExcludes() );
<a class="jxr_linenumber" name="L622" href="#L622">622</a> }
-<a class="jxr_linenumber" name="L623" href="#L623">623</a> <strong class="jxr_keyword">else</strong>
-<a class="jxr_linenumber" name="L624" href="#L624">624</a> {
-<a class="jxr_linenumber" name="L625" href="#L625">625</a> Collection<File> sourceDirectories = request.getSourceDirectories();
-<a class="jxr_linenumber" name="L626" href="#L626">626</a> addFilesToProcess( request, sourceDirectories, request.getResources(),
-<a class="jxr_linenumber" name="L627" href="#L627">627</a> request.getTestResources(), files, request.getTestSourceDirectories() );
-<a class="jxr_linenumber" name="L628" href="#L628">628</a> }
-<a class="jxr_linenumber" name="L629" href="#L629">629</a>
-<a class="jxr_linenumber" name="L630" href="#L630">630</a> getLogger().debug( <span class="jxr_string">"Added "</span> + files.size() + <span class="jxr_string">" files to process."</span> );
+<a class="jxr_linenumber" name="L623" href="#L623">623</a>
+<a class="jxr_linenumber" name="L624" href="#L624">624</a> String[] defaultExcludes = FileUtils.getDefaultExcludes();
+<a class="jxr_linenumber" name="L625" href="#L625">625</a> <strong class="jxr_keyword">for</strong> ( String defaultExclude : defaultExcludes )
+<a class="jxr_linenumber" name="L626" href="#L626">626</a> {
+<a class="jxr_linenumber" name="L627" href="#L627">627</a> <strong class="jxr_keyword">if</strong> ( excludesStr.length() > 0 )
+<a class="jxr_linenumber" name="L628" href="#L628">628</a> {
+<a class="jxr_linenumber" name="L629" href="#L629">629</a> excludesStr.append( <span class="jxr_string">","</span> );
+<a class="jxr_linenumber" name="L630" href="#L630">630</a> }
<a class="jxr_linenumber" name="L631" href="#L631">631</a>
-<a class="jxr_linenumber" name="L632" href="#L632">632</a> <strong class="jxr_keyword">return</strong> <strong class="jxr_keyword">new</strong> ArrayList<File>( files );
-<a class="jxr_linenumber" name="L633" href="#L633">633</a> }
+<a class="jxr_linenumber" name="L632" href="#L632">632</a> excludesStr.append( defaultExclude );
+<a class="jxr_linenumber" name="L633" href="#L633">633</a> }
<a class="jxr_linenumber" name="L634" href="#L634">634</a>
-<a class="jxr_linenumber" name="L635" href="#L635">635</a> <strong class="jxr_keyword">private</strong> <strong class="jxr_keyword">void</strong> addFilesToProcess( CheckstyleExecutorRequest request, Collection<File> sourceDirectories,
-<a class="jxr_linenumber" name="L636" href="#L636">636</a> List<Resource> resources, List<Resource> testResources, Collection<File> files,
-<a class="jxr_linenumber" name="L637" href="#L637">637</a> Collection<File> testSourceDirectories )
-<a class="jxr_linenumber" name="L638" href="#L638">638</a> <strong class="jxr_keyword">throws</strong> IOException
-<a class="jxr_linenumber" name="L639" href="#L639">639</a> {
-<a class="jxr_linenumber" name="L640" href="#L640">640</a> <strong class="jxr_keyword">if</strong> ( sourceDirectories != <strong class="jxr_keyword">null</strong> )
-<a class="jxr_linenumber" name="L641" href="#L641">641</a> {
-<a class="jxr_linenumber" name="L642" href="#L642">642</a> <strong class="jxr_keyword">for</strong> ( File sourceDirectory : sourceDirectories )
-<a class="jxr_linenumber" name="L643" href="#L643">643</a> {
-<a class="jxr_linenumber" name="L644" href="#L644">644</a> <strong class="jxr_keyword">if</strong> ( sourceDirectory.isDirectory() )
+<a class="jxr_linenumber" name="L635" href="#L635">635</a> Set<File> files = <strong class="jxr_keyword">new</strong> LinkedHashSet<File>();
+<a class="jxr_linenumber" name="L636" href="#L636">636</a> <strong class="jxr_keyword">if</strong> ( request.isAggregate() )
+<a class="jxr_linenumber" name="L637" href="#L637">637</a> {
+<a class="jxr_linenumber" name="L638" href="#L638">638</a> <strong class="jxr_keyword">for</strong> ( MavenProject project : request.getReactorProjects() )
+<a class="jxr_linenumber" name="L639" href="#L639">639</a> {
+<a class="jxr_linenumber" name="L640" href="#L640">640</a> Set<File> sourceDirectories = <strong class="jxr_keyword">new</strong> LinkedHashSet<File>();
+<a class="jxr_linenumber" name="L641" href="#L641">641</a>
+<a class="jxr_linenumber" name="L642" href="#L642">642</a> <em class="jxr_comment">// CompileSourceRoots are absolute paths</em>
+<a class="jxr_linenumber" name="L643" href="#L643">643</a> List<String> compileSourceRoots = project.getCompileSourceRoots();
+<a class="jxr_linenumber" name="L644" href="#L644">644</a> <strong class="jxr_keyword">for</strong> ( String compileSourceRoot : compileSourceRoots )
<a class="jxr_linenumber" name="L645" href="#L645">645</a> {
-<a class="jxr_linenumber" name="L646" href="#L646">646</a> <strong class="jxr_keyword">final</strong> List<File> sourceFiles =
-<a class="jxr_linenumber" name="L647" href="#L647">647</a> FileUtils.getFiles( sourceDirectory, request.getIncludes(), request.getExcludes() );
-<a class="jxr_linenumber" name="L648" href="#L648">648</a> files.addAll( sourceFiles );
-<a class="jxr_linenumber" name="L649" href="#L649">649</a> getLogger().debug( <span class="jxr_string">"Added "</span> + sourceFiles.size() + <span class="jxr_string">" source files found in '"</span>
-<a class="jxr_linenumber" name="L650" href="#L650">650</a> + sourceDirectory.getAbsolutePath() + <span class="jxr_string">"'."</span> );
-<a class="jxr_linenumber" name="L651" href="#L651">651</a> }
-<a class="jxr_linenumber" name="L652" href="#L652">652</a> }
-<a class="jxr_linenumber" name="L653" href="#L653">653</a> }
-<a class="jxr_linenumber" name="L654" href="#L654">654</a>
[... 464 lines stripped ...]