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 ad...@apache.org on 2019/04/11 18:18:15 UTC
svn commit: r1857356 [46/47] - in
/maven/website/components/plugins-archives/maven-pmd-plugin-LATEST: ./
apidocs/ apidocs/org/apache/maven/plugins/pmd/
apidocs/org/apache/maven/plugins/pmd/class-use/
apidocs/org/apache/maven/plugins/pmd/model/ apidocs/...
Modified: maven/website/components/plugins-archives/maven-pmd-plugin-LATEST/xref/org/apache/maven/plugins/pmd/PmdReportGenerator.html
==============================================================================
--- maven/website/components/plugins-archives/maven-pmd-plugin-LATEST/xref/org/apache/maven/plugins/pmd/PmdReportGenerator.html (original)
+++ maven/website/components/plugins-archives/maven-pmd-plugin-LATEST/xref/org/apache/maven/plugins/pmd/PmdReportGenerator.html Thu Apr 11 18:18:13 2019
@@ -33,417 +33,498 @@
<a class="jxr_linenumber" name="L25" href="#L25">25</a> <strong class="jxr_keyword">import</strong> java.util.Collection;
<a class="jxr_linenumber" name="L26" href="#L26">26</a> <strong class="jxr_keyword">import</strong> java.util.Collections;
<a class="jxr_linenumber" name="L27" href="#L27">27</a> <strong class="jxr_keyword">import</strong> java.util.Comparator;
-<a class="jxr_linenumber" name="L28" href="#L28">28</a> <strong class="jxr_keyword">import</strong> java.util.HashSet;
-<a class="jxr_linenumber" name="L29" href="#L29">29</a> <strong class="jxr_keyword">import</strong> java.util.List;
-<a class="jxr_linenumber" name="L30" href="#L30">30</a> <strong class="jxr_keyword">import</strong> java.util.Map;
-<a class="jxr_linenumber" name="L31" href="#L31">31</a> <strong class="jxr_keyword">import</strong> java.util.ResourceBundle;
-<a class="jxr_linenumber" name="L32" href="#L32">32</a> <strong class="jxr_keyword">import</strong> java.util.Set;
-<a class="jxr_linenumber" name="L33" href="#L33">33</a>
-<a class="jxr_linenumber" name="L34" href="#L34">34</a> <strong class="jxr_keyword">import</strong> org.apache.maven.doxia.sink.Sink;
-<a class="jxr_linenumber" name="L35" href="#L35">35</a> <strong class="jxr_keyword">import</strong> org.apache.maven.plugin.logging.Log;
-<a class="jxr_linenumber" name="L36" href="#L36">36</a> <strong class="jxr_keyword">import</strong> org.codehaus.plexus.util.StringUtils;
-<a class="jxr_linenumber" name="L37" href="#L37">37</a>
-<a class="jxr_linenumber" name="L38" href="#L38">38</a> <strong class="jxr_keyword">import</strong> net.sourceforge.pmd.Report.ProcessingError;
-<a class="jxr_linenumber" name="L39" href="#L39">39</a> <strong class="jxr_keyword">import</strong> net.sourceforge.pmd.RuleViolation;
-<a class="jxr_linenumber" name="L40" href="#L40">40</a>
-<a class="jxr_linenumber" name="L41" href="#L41">41</a> <em class="jxr_javadoccomment">/**</em>
-<a class="jxr_linenumber" name="L42" href="#L42">42</a> <em class="jxr_javadoccomment"> * Render the PMD violations into Doxia events.</em>
-<a class="jxr_linenumber" name="L43" href="#L43">43</a> <em class="jxr_javadoccomment"> *</em>
-<a class="jxr_linenumber" name="L44" href="#L44">44</a> <em class="jxr_javadoccomment"> * @author Brett Porter</em>
-<a class="jxr_linenumber" name="L45" href="#L45">45</a> <em class="jxr_javadoccomment"> * @version $Id$</em>
-<a class="jxr_linenumber" name="L46" href="#L46">46</a> <em class="jxr_javadoccomment"> */</em>
-<a class="jxr_linenumber" name="L47" href="#L47">47</a> <strong class="jxr_keyword">public</strong> <strong class="jxr_keyword">class</strong> <a href="../../../../../org/apache/maven/plugins/pmd/PmdReportGenerator.html">PmdReportGenerator</a>
-<a class="jxr_linenumber" name="L48" href="#L48">48</a> {
-<a class="jxr_linenumber" name="L49" href="#L49">49</a> <strong class="jxr_keyword">private</strong> Log log;
-<a class="jxr_linenumber" name="L50" href="#L50">50</a>
-<a class="jxr_linenumber" name="L51" href="#L51">51</a> <strong class="jxr_keyword">private</strong> Sink sink;
+<a class="jxr_linenumber" name="L28" href="#L28">28</a> <strong class="jxr_keyword">import</strong> java.util.HashMap;
+<a class="jxr_linenumber" name="L29" href="#L29">29</a> <strong class="jxr_keyword">import</strong> java.util.HashSet;
+<a class="jxr_linenumber" name="L30" href="#L30">30</a> <strong class="jxr_keyword">import</strong> java.util.List;
+<a class="jxr_linenumber" name="L31" href="#L31">31</a> <strong class="jxr_keyword">import</strong> java.util.Map;
+<a class="jxr_linenumber" name="L32" href="#L32">32</a> <strong class="jxr_keyword">import</strong> java.util.ResourceBundle;
+<a class="jxr_linenumber" name="L33" href="#L33">33</a> <strong class="jxr_keyword">import</strong> java.util.Set;
+<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.maven.doxia.sink.Sink;
+<a class="jxr_linenumber" name="L36" href="#L36">36</a> <strong class="jxr_keyword">import</strong> org.apache.maven.plugin.logging.Log;
+<a class="jxr_linenumber" name="L37" href="#L37">37</a> <strong class="jxr_keyword">import</strong> org.codehaus.plexus.util.StringUtils;
+<a class="jxr_linenumber" name="L38" href="#L38">38</a>
+<a class="jxr_linenumber" name="L39" href="#L39">39</a> <strong class="jxr_keyword">import</strong> net.sourceforge.pmd.Report.ProcessingError;
+<a class="jxr_linenumber" name="L40" href="#L40">40</a> <strong class="jxr_keyword">import</strong> net.sourceforge.pmd.RulePriority;
+<a class="jxr_linenumber" name="L41" href="#L41">41</a> <strong class="jxr_keyword">import</strong> net.sourceforge.pmd.RuleViolation;
+<a class="jxr_linenumber" name="L42" href="#L42">42</a>
+<a class="jxr_linenumber" name="L43" href="#L43">43</a> <em class="jxr_javadoccomment">/**</em>
+<a class="jxr_linenumber" name="L44" href="#L44">44</a> <em class="jxr_javadoccomment"> * Render the PMD violations into Doxia events.</em>
+<a class="jxr_linenumber" name="L45" href="#L45">45</a> <em class="jxr_javadoccomment"> *</em>
+<a class="jxr_linenumber" name="L46" href="#L46">46</a> <em class="jxr_javadoccomment"> * @author Brett Porter</em>
+<a class="jxr_linenumber" name="L47" href="#L47">47</a> <em class="jxr_javadoccomment"> * @version $Id$</em>
+<a class="jxr_linenumber" name="L48" href="#L48">48</a> <em class="jxr_javadoccomment"> */</em>
+<a class="jxr_linenumber" name="L49" href="#L49">49</a> <strong class="jxr_keyword">public</strong> <strong class="jxr_keyword">class</strong> <a href="../../../../../org/apache/maven/plugins/pmd/PmdReportGenerator.html">PmdReportGenerator</a>
+<a class="jxr_linenumber" name="L50" href="#L50">50</a> {
+<a class="jxr_linenumber" name="L51" href="#L51">51</a> <strong class="jxr_keyword">private</strong> Log log;
<a class="jxr_linenumber" name="L52" href="#L52">52</a>
-<a class="jxr_linenumber" name="L53" href="#L53">53</a> <strong class="jxr_keyword">private</strong> String currentFilename;
+<a class="jxr_linenumber" name="L53" href="#L53">53</a> <strong class="jxr_keyword">private</strong> Sink sink;
<a class="jxr_linenumber" name="L54" href="#L54">54</a>
-<a class="jxr_linenumber" name="L55" href="#L55">55</a> <strong class="jxr_keyword">private</strong> ResourceBundle bundle;
+<a class="jxr_linenumber" name="L55" href="#L55">55</a> <strong class="jxr_keyword">private</strong> String currentFilename;
<a class="jxr_linenumber" name="L56" href="#L56">56</a>
-<a class="jxr_linenumber" name="L57" href="#L57">57</a> <strong class="jxr_keyword">private</strong> Set<RuleViolation> violations = <strong class="jxr_keyword">new</strong> HashSet<>();
+<a class="jxr_linenumber" name="L57" href="#L57">57</a> <strong class="jxr_keyword">private</strong> ResourceBundle bundle;
<a class="jxr_linenumber" name="L58" href="#L58">58</a>
-<a class="jxr_linenumber" name="L59" href="#L59">59</a> <strong class="jxr_keyword">private</strong> List<ProcessingError> processingErrors = <strong class="jxr_keyword">new</strong> ArrayList<>();
+<a class="jxr_linenumber" name="L59" href="#L59">59</a> <strong class="jxr_keyword">private</strong> Set<RuleViolation> violations = <strong class="jxr_keyword">new</strong> HashSet<>();
<a class="jxr_linenumber" name="L60" href="#L60">60</a>
-<a class="jxr_linenumber" name="L61" href="#L61">61</a> <strong class="jxr_keyword">private</strong> <strong class="jxr_keyword">boolean</strong> aggregate;
+<a class="jxr_linenumber" name="L61" href="#L61">61</a> <strong class="jxr_keyword">private</strong> List<ProcessingError> processingErrors = <strong class="jxr_keyword">new</strong> ArrayList<>();
<a class="jxr_linenumber" name="L62" href="#L62">62</a>
-<a class="jxr_linenumber" name="L63" href="#L63">63</a> <strong class="jxr_keyword">private</strong> <strong class="jxr_keyword">boolean</strong> renderRuleViolationPriority;
+<a class="jxr_linenumber" name="L63" href="#L63">63</a> <strong class="jxr_keyword">private</strong> <strong class="jxr_keyword">boolean</strong> aggregate;
<a class="jxr_linenumber" name="L64" href="#L64">64</a>
-<a class="jxr_linenumber" name="L65" href="#L65">65</a> <em class="jxr_comment">// The number of erroneous files</em>
-<a class="jxr_linenumber" name="L66" href="#L66">66</a> <strong class="jxr_keyword">private</strong> <strong class="jxr_keyword">int</strong> fileCount = 0;
-<a class="jxr_linenumber" name="L67" href="#L67">67</a>
-<a class="jxr_linenumber" name="L68" href="#L68">68</a> <strong class="jxr_keyword">private</strong> Map<File, PmdFileInfo> files;
-<a class="jxr_linenumber" name="L69" href="#L69">69</a>
-<a class="jxr_linenumber" name="L70" href="#L70">70</a> <em class="jxr_comment">// private List<Metric> metrics = new ArrayList<Metric>();</em>
-<a class="jxr_linenumber" name="L71" href="#L71">71</a>
-<a class="jxr_linenumber" name="L72" href="#L72">72</a> <strong class="jxr_keyword">public</strong> <a href="../../../../../org/apache/maven/plugins/pmd/PmdReportGenerator.html">PmdReportGenerator</a>( Log log, Sink sink, ResourceBundle bundle, <strong class="jxr_keyword">boolean</strong> aggregate )
-<a class="jxr_linenumber" name="L73" href="#L73">73</a> {
-<a class="jxr_linenumber" name="L74" href="#L74">74</a> <strong class="jxr_keyword">this</strong>.log = log;
-<a class="jxr_linenumber" name="L75" href="#L75">75</a> <strong class="jxr_keyword">this</strong>.sink = sink;
-<a class="jxr_linenumber" name="L76" href="#L76">76</a> <strong class="jxr_keyword">this</strong>.bundle = bundle;
-<a class="jxr_linenumber" name="L77" href="#L77">77</a> <strong class="jxr_keyword">this</strong>.aggregate = aggregate;
-<a class="jxr_linenumber" name="L78" href="#L78">78</a> }
-<a class="jxr_linenumber" name="L79" href="#L79">79</a>
-<a class="jxr_linenumber" name="L80" href="#L80">80</a> <strong class="jxr_keyword">private</strong> String getTitle()
-<a class="jxr_linenumber" name="L81" href="#L81">81</a> {
-<a class="jxr_linenumber" name="L82" href="#L82">82</a> <strong class="jxr_keyword">return</strong> bundle.getString( <span class="jxr_string">"report.pmd.title"</span> );
-<a class="jxr_linenumber" name="L83" href="#L83">83</a> }
-<a class="jxr_linenumber" name="L84" href="#L84">84</a>
-<a class="jxr_linenumber" name="L85" href="#L85">85</a> <strong class="jxr_keyword">public</strong> <strong class="jxr_keyword">void</strong> setViolations( Collection<RuleViolation> violations )
-<a class="jxr_linenumber" name="L86" href="#L86">86</a> {
-<a class="jxr_linenumber" name="L87" href="#L87">87</a> <strong class="jxr_keyword">this</strong>.violations = <strong class="jxr_keyword">new</strong> HashSet<>( violations );
-<a class="jxr_linenumber" name="L88" href="#L88">88</a> }
-<a class="jxr_linenumber" name="L89" href="#L89">89</a>
-<a class="jxr_linenumber" name="L90" href="#L90">90</a> <strong class="jxr_keyword">public</strong> List<RuleViolation> getViolations()
-<a class="jxr_linenumber" name="L91" href="#L91">91</a> {
-<a class="jxr_linenumber" name="L92" href="#L92">92</a> <strong class="jxr_keyword">return</strong> <strong class="jxr_keyword">new</strong> ArrayList<>( violations );
-<a class="jxr_linenumber" name="L93" href="#L93">93</a> }
-<a class="jxr_linenumber" name="L94" href="#L94">94</a>
-<a class="jxr_linenumber" name="L95" href="#L95">95</a> <strong class="jxr_keyword">public</strong> <strong class="jxr_keyword">void</strong> setProcessingErrors( Collection<ProcessingError> errors )
-<a class="jxr_linenumber" name="L96" href="#L96">96</a> {
-<a class="jxr_linenumber" name="L97" href="#L97">97</a> <strong class="jxr_keyword">this</strong>.processingErrors = <strong class="jxr_keyword">new</strong> ArrayList<>( errors );
-<a class="jxr_linenumber" name="L98" href="#L98">98</a> }
-<a class="jxr_linenumber" name="L99" href="#L99">99</a>
-<a class="jxr_linenumber" name="L100" href="#L100">100</a> <strong class="jxr_keyword">public</strong> List<ProcessingError> getProcessingErrors()
-<a class="jxr_linenumber" name="L101" href="#L101">101</a> {
-<a class="jxr_linenumber" name="L102" href="#L102">102</a> <strong class="jxr_keyword">return</strong> processingErrors;
-<a class="jxr_linenumber" name="L103" href="#L103">103</a> }
-<a class="jxr_linenumber" name="L104" href="#L104">104</a>
-<a class="jxr_linenumber" name="L105" href="#L105">105</a> <em class="jxr_comment">// public List<Metric> getMetrics()</em>
-<a class="jxr_linenumber" name="L106" href="#L106">106</a> <em class="jxr_comment">// {</em>
-<a class="jxr_linenumber" name="L107" href="#L107">107</a> <em class="jxr_comment">// return metrics;</em>
-<a class="jxr_linenumber" name="L108" href="#L108">108</a> <em class="jxr_comment">// }</em>
-<a class="jxr_linenumber" name="L109" href="#L109">109</a> <em class="jxr_comment">//</em>
-<a class="jxr_linenumber" name="L110" href="#L110">110</a> <em class="jxr_comment">// public void setMetrics( List<Metric> metrics )</em>
-<a class="jxr_linenumber" name="L111" href="#L111">111</a> <em class="jxr_comment">// {</em>
-<a class="jxr_linenumber" name="L112" href="#L112">112</a> <em class="jxr_comment">// this.metrics = metrics;</em>
-<a class="jxr_linenumber" name="L113" href="#L113">113</a> <em class="jxr_comment">// }</em>
-<a class="jxr_linenumber" name="L114" href="#L114">114</a>
-<a class="jxr_linenumber" name="L115" href="#L115">115</a> <strong class="jxr_keyword">private</strong> String shortenFilename( String filename, PmdFileInfo fileInfo )
-<a class="jxr_linenumber" name="L116" href="#L116">116</a> {
-<a class="jxr_linenumber" name="L117" href="#L117">117</a> String result = filename;
-<a class="jxr_linenumber" name="L118" href="#L118">118</a> <strong class="jxr_keyword">if</strong> ( fileInfo != <strong class="jxr_keyword">null</strong> && fileInfo.getSourceDirectory() != <strong class="jxr_keyword">null</strong> )
-<a class="jxr_linenumber" name="L119" href="#L119">119</a> {
-<a class="jxr_linenumber" name="L120" href="#L120">120</a> result = StringUtils.substring( result, fileInfo.getSourceDirectory().getAbsolutePath().length() + 1 );
-<a class="jxr_linenumber" name="L121" href="#L121">121</a> }
-<a class="jxr_linenumber" name="L122" href="#L122">122</a> <strong class="jxr_keyword">return</strong> StringUtils.replace( result, <span class="jxr_string">"\\"</span>, <span class="jxr_string">"/"</span> );
-<a class="jxr_linenumber" name="L123" href="#L123">123</a> }
-<a class="jxr_linenumber" name="L124" href="#L124">124</a>
-<a class="jxr_linenumber" name="L125" href="#L125">125</a> <strong class="jxr_keyword">private</strong> String makeFileSectionName( String filename, PmdFileInfo fileInfo )
-<a class="jxr_linenumber" name="L126" href="#L126">126</a> {
-<a class="jxr_linenumber" name="L127" href="#L127">127</a> <strong class="jxr_keyword">if</strong> ( aggregate && fileInfo != <strong class="jxr_keyword">null</strong> && fileInfo.getProject() != <strong class="jxr_keyword">null</strong> )
-<a class="jxr_linenumber" name="L128" href="#L128">128</a> {
-<a class="jxr_linenumber" name="L129" href="#L129">129</a> <strong class="jxr_keyword">return</strong> fileInfo.getProject().getName() + <span class="jxr_string">" - "</span> + filename;
-<a class="jxr_linenumber" name="L130" href="#L130">130</a> }
-<a class="jxr_linenumber" name="L131" href="#L131">131</a> <strong class="jxr_keyword">return</strong> filename;
-<a class="jxr_linenumber" name="L132" href="#L132">132</a> }
-<a class="jxr_linenumber" name="L133" href="#L133">133</a>
-<a class="jxr_linenumber" name="L134" href="#L134">134</a> <strong class="jxr_keyword">private</strong> <a href="../../../../../org/apache/maven/plugins/pmd/PmdFileInfo.html">PmdFileInfo</a> determineFileInfo( String filename )
-<a class="jxr_linenumber" name="L135" href="#L135">135</a> <strong class="jxr_keyword">throws</strong> IOException
-<a class="jxr_linenumber" name="L136" href="#L136">136</a> {
-<a class="jxr_linenumber" name="L137" href="#L137">137</a> File canonicalFilename = <strong class="jxr_keyword">new</strong> File( filename ).getCanonicalFile();
-<a class="jxr_linenumber" name="L138" href="#L138">138</a> <a href="../../../../../org/apache/maven/plugins/pmd/PmdFileInfo.html">PmdFileInfo</a> fileInfo = files.get( canonicalFilename );
-<a class="jxr_linenumber" name="L139" href="#L139">139</a> <strong class="jxr_keyword">if</strong> ( fileInfo == <strong class="jxr_keyword">null</strong> )
-<a class="jxr_linenumber" name="L140" href="#L140">140</a> {
-<a class="jxr_linenumber" name="L141" href="#L141">141</a> log.warn( <span class="jxr_string">"Couldn't determine PmdFileInfo for file "</span> + filename + <span class="jxr_string">" (canonical: "</span> + canonicalFilename
-<a class="jxr_linenumber" name="L142" href="#L142">142</a> + <span class="jxr_string">"). XRef links won't be available."</span> );
-<a class="jxr_linenumber" name="L143" href="#L143">143</a> }
-<a class="jxr_linenumber" name="L144" href="#L144">144</a>
-<a class="jxr_linenumber" name="L145" href="#L145">145</a> <strong class="jxr_keyword">return</strong> fileInfo;
-<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> <strong class="jxr_keyword">private</strong> <strong class="jxr_keyword">void</strong> startFileSection( String currentFilename, PmdFileInfo fileInfo )
-<a class="jxr_linenumber" name="L149" href="#L149">149</a> {
-<a class="jxr_linenumber" name="L150" href="#L150">150</a> sink.section2();
-<a class="jxr_linenumber" name="L151" href="#L151">151</a> sink.sectionTitle2();
-<a class="jxr_linenumber" name="L152" href="#L152">152</a>
-<a class="jxr_linenumber" name="L153" href="#L153">153</a> <em class="jxr_comment">// prepare the filename</em>
-<a class="jxr_linenumber" name="L154" href="#L154">154</a> <strong class="jxr_keyword">this</strong>.currentFilename = shortenFilename( currentFilename, fileInfo );
-<a class="jxr_linenumber" name="L155" href="#L155">155</a>
-<a class="jxr_linenumber" name="L156" href="#L156">156</a> sink.text( makeFileSectionName( <strong class="jxr_keyword">this</strong>.currentFilename, fileInfo ) );
-<a class="jxr_linenumber" name="L157" href="#L157">157</a> sink.sectionTitle2_();
-<a class="jxr_linenumber" name="L158" href="#L158">158</a>
-<a class="jxr_linenumber" name="L159" href="#L159">159</a> sink.table();
-<a class="jxr_linenumber" name="L160" href="#L160">160</a> sink.tableRow();
-<a class="jxr_linenumber" name="L161" href="#L161">161</a> sink.tableHeaderCell();
-<a class="jxr_linenumber" name="L162" href="#L162">162</a> sink.text( bundle.getString( <span class="jxr_string">"report.pmd.column.violation"</span> ) );
-<a class="jxr_linenumber" name="L163" href="#L163">163</a> sink.tableHeaderCell_();
-<a class="jxr_linenumber" name="L164" href="#L164">164</a> <strong class="jxr_keyword">if</strong> ( <strong class="jxr_keyword">this</strong>.renderRuleViolationPriority )
-<a class="jxr_linenumber" name="L165" href="#L165">165</a> {
-<a class="jxr_linenumber" name="L166" href="#L166">166</a> sink.tableHeaderCell();
-<a class="jxr_linenumber" name="L167" href="#L167">167</a> sink.text( bundle.getString( <span class="jxr_string">"report.pmd.column.priority"</span> ) );
-<a class="jxr_linenumber" name="L168" href="#L168">168</a> sink.tableHeaderCell_();
-<a class="jxr_linenumber" name="L169" href="#L169">169</a> }
-<a class="jxr_linenumber" name="L170" href="#L170">170</a> sink.tableHeaderCell();
-<a class="jxr_linenumber" name="L171" href="#L171">171</a> sink.text( bundle.getString( <span class="jxr_string">"report.pmd.column.line"</span> ) );
-<a class="jxr_linenumber" name="L172" href="#L172">172</a> sink.tableHeaderCell_();
-<a class="jxr_linenumber" name="L173" href="#L173">173</a> sink.tableRow_();
-<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> <strong class="jxr_keyword">private</strong> <strong class="jxr_keyword">void</strong> endFileSection()
-<a class="jxr_linenumber" name="L177" href="#L177">177</a> {
-<a class="jxr_linenumber" name="L178" href="#L178">178</a> sink.table_();
-<a class="jxr_linenumber" name="L179" href="#L179">179</a> sink.section2_();
-<a class="jxr_linenumber" name="L180" href="#L180">180</a> }
-<a class="jxr_linenumber" name="L181" href="#L181">181</a>
-<a class="jxr_linenumber" name="L182" href="#L182">182</a> <strong class="jxr_keyword">private</strong> <strong class="jxr_keyword">void</strong> processSingleRuleViolation( RuleViolation ruleViolation, PmdFileInfo fileInfo )
-<a class="jxr_linenumber" name="L183" href="#L183">183</a> {
-<a class="jxr_linenumber" name="L184" href="#L184">184</a> sink.tableRow();
-<a class="jxr_linenumber" name="L185" href="#L185">185</a> sink.tableCell();
-<a class="jxr_linenumber" name="L186" href="#L186">186</a> sink.text( ruleViolation.getDescription() );
-<a class="jxr_linenumber" name="L187" href="#L187">187</a> sink.tableCell_();
-<a class="jxr_linenumber" name="L188" href="#L188">188</a>
-<a class="jxr_linenumber" name="L189" href="#L189">189</a> <strong class="jxr_keyword">if</strong> ( <strong class="jxr_keyword">this</strong>.renderRuleViolationPriority )
-<a class="jxr_linenumber" name="L190" href="#L190">190</a> {
-<a class="jxr_linenumber" name="L191" href="#L191">191</a> sink.tableCell();
-<a class="jxr_linenumber" name="L192" href="#L192">192</a> sink.text( String.valueOf( ruleViolation.getRule().getPriority().getPriority() ) );
-<a class="jxr_linenumber" name="L193" href="#L193">193</a> sink.tableCell_();
-<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> sink.tableCell();
+<a class="jxr_linenumber" name="L65" href="#L65">65</a> <strong class="jxr_keyword">private</strong> <strong class="jxr_keyword">boolean</strong> renderRuleViolationPriority;
+<a class="jxr_linenumber" name="L66" href="#L66">66</a>
+<a class="jxr_linenumber" name="L67" href="#L67">67</a> <strong class="jxr_keyword">private</strong> <strong class="jxr_keyword">boolean</strong> renderViolationsByPriority;
+<a class="jxr_linenumber" name="L68" href="#L68">68</a>
+<a class="jxr_linenumber" name="L69" href="#L69">69</a> <strong class="jxr_keyword">private</strong> Map<File, PmdFileInfo> files;
+<a class="jxr_linenumber" name="L70" href="#L70">70</a>
+<a class="jxr_linenumber" name="L71" href="#L71">71</a> <em class="jxr_comment">// private List<Metric> metrics = new ArrayList<Metric>();</em>
+<a class="jxr_linenumber" name="L72" href="#L72">72</a>
+<a class="jxr_linenumber" name="L73" href="#L73">73</a> <strong class="jxr_keyword">public</strong> <a href="../../../../../org/apache/maven/plugins/pmd/PmdReportGenerator.html">PmdReportGenerator</a>( Log log, Sink sink, ResourceBundle bundle, <strong class="jxr_keyword">boolean</strong> aggregate )
+<a class="jxr_linenumber" name="L74" href="#L74">74</a> {
+<a class="jxr_linenumber" name="L75" href="#L75">75</a> <strong class="jxr_keyword">this</strong>.log = log;
+<a class="jxr_linenumber" name="L76" href="#L76">76</a> <strong class="jxr_keyword">this</strong>.sink = sink;
+<a class="jxr_linenumber" name="L77" href="#L77">77</a> <strong class="jxr_keyword">this</strong>.bundle = bundle;
+<a class="jxr_linenumber" name="L78" href="#L78">78</a> <strong class="jxr_keyword">this</strong>.aggregate = aggregate;
+<a class="jxr_linenumber" name="L79" href="#L79">79</a> }
+<a class="jxr_linenumber" name="L80" href="#L80">80</a>
+<a class="jxr_linenumber" name="L81" href="#L81">81</a> <strong class="jxr_keyword">private</strong> String getTitle()
+<a class="jxr_linenumber" name="L82" href="#L82">82</a> {
+<a class="jxr_linenumber" name="L83" href="#L83">83</a> <strong class="jxr_keyword">return</strong> bundle.getString( <span class="jxr_string">"report.pmd.title"</span> );
+<a class="jxr_linenumber" name="L84" href="#L84">84</a> }
+<a class="jxr_linenumber" name="L85" href="#L85">85</a>
+<a class="jxr_linenumber" name="L86" href="#L86">86</a> <strong class="jxr_keyword">public</strong> <strong class="jxr_keyword">void</strong> setViolations( Collection<RuleViolation> violations )
+<a class="jxr_linenumber" name="L87" href="#L87">87</a> {
+<a class="jxr_linenumber" name="L88" href="#L88">88</a> <strong class="jxr_keyword">this</strong>.violations = <strong class="jxr_keyword">new</strong> HashSet<>( violations );
+<a class="jxr_linenumber" name="L89" href="#L89">89</a> }
+<a class="jxr_linenumber" name="L90" href="#L90">90</a>
+<a class="jxr_linenumber" name="L91" href="#L91">91</a> <strong class="jxr_keyword">public</strong> List<RuleViolation> getViolations()
+<a class="jxr_linenumber" name="L92" href="#L92">92</a> {
+<a class="jxr_linenumber" name="L93" href="#L93">93</a> <strong class="jxr_keyword">return</strong> <strong class="jxr_keyword">new</strong> ArrayList<>( violations );
+<a class="jxr_linenumber" name="L94" href="#L94">94</a> }
+<a class="jxr_linenumber" name="L95" href="#L95">95</a>
+<a class="jxr_linenumber" name="L96" href="#L96">96</a> <strong class="jxr_keyword">public</strong> <strong class="jxr_keyword">void</strong> setProcessingErrors( Collection<ProcessingError> errors )
+<a class="jxr_linenumber" name="L97" href="#L97">97</a> {
+<a class="jxr_linenumber" name="L98" href="#L98">98</a> <strong class="jxr_keyword">this</strong>.processingErrors = <strong class="jxr_keyword">new</strong> ArrayList<>( errors );
+<a class="jxr_linenumber" name="L99" href="#L99">99</a> }
+<a class="jxr_linenumber" name="L100" href="#L100">100</a>
+<a class="jxr_linenumber" name="L101" href="#L101">101</a> <strong class="jxr_keyword">public</strong> List<ProcessingError> getProcessingErrors()
+<a class="jxr_linenumber" name="L102" href="#L102">102</a> {
+<a class="jxr_linenumber" name="L103" href="#L103">103</a> <strong class="jxr_keyword">return</strong> processingErrors;
+<a class="jxr_linenumber" name="L104" href="#L104">104</a> }
+<a class="jxr_linenumber" name="L105" href="#L105">105</a>
+<a class="jxr_linenumber" name="L106" href="#L106">106</a> <em class="jxr_comment">// public List<Metric> getMetrics()</em>
+<a class="jxr_linenumber" name="L107" href="#L107">107</a> <em class="jxr_comment">// {</em>
+<a class="jxr_linenumber" name="L108" href="#L108">108</a> <em class="jxr_comment">// return metrics;</em>
+<a class="jxr_linenumber" name="L109" href="#L109">109</a> <em class="jxr_comment">// }</em>
+<a class="jxr_linenumber" name="L110" href="#L110">110</a> <em class="jxr_comment">//</em>
+<a class="jxr_linenumber" name="L111" href="#L111">111</a> <em class="jxr_comment">// public void setMetrics( List<Metric> metrics )</em>
+<a class="jxr_linenumber" name="L112" href="#L112">112</a> <em class="jxr_comment">// {</em>
+<a class="jxr_linenumber" name="L113" href="#L113">113</a> <em class="jxr_comment">// this.metrics = metrics;</em>
+<a class="jxr_linenumber" name="L114" href="#L114">114</a> <em class="jxr_comment">// }</em>
+<a class="jxr_linenumber" name="L115" href="#L115">115</a>
+<a class="jxr_linenumber" name="L116" href="#L116">116</a> <strong class="jxr_keyword">private</strong> String shortenFilename( String filename, PmdFileInfo fileInfo )
+<a class="jxr_linenumber" name="L117" href="#L117">117</a> {
+<a class="jxr_linenumber" name="L118" href="#L118">118</a> String result = filename;
+<a class="jxr_linenumber" name="L119" href="#L119">119</a> <strong class="jxr_keyword">if</strong> ( fileInfo != <strong class="jxr_keyword">null</strong> && fileInfo.getSourceDirectory() != <strong class="jxr_keyword">null</strong> )
+<a class="jxr_linenumber" name="L120" href="#L120">120</a> {
+<a class="jxr_linenumber" name="L121" href="#L121">121</a> result = StringUtils.substring( result, fileInfo.getSourceDirectory().getAbsolutePath().length() + 1 );
+<a class="jxr_linenumber" name="L122" href="#L122">122</a> }
+<a class="jxr_linenumber" name="L123" href="#L123">123</a> <strong class="jxr_keyword">return</strong> StringUtils.replace( result, <span class="jxr_string">"\\"</span>, <span class="jxr_string">"/"</span> );
+<a class="jxr_linenumber" name="L124" href="#L124">124</a> }
+<a class="jxr_linenumber" name="L125" href="#L125">125</a>
+<a class="jxr_linenumber" name="L126" href="#L126">126</a> <strong class="jxr_keyword">private</strong> String makeFileSectionName( String filename, PmdFileInfo fileInfo )
+<a class="jxr_linenumber" name="L127" href="#L127">127</a> {
+<a class="jxr_linenumber" name="L128" href="#L128">128</a> <strong class="jxr_keyword">if</strong> ( aggregate && fileInfo != <strong class="jxr_keyword">null</strong> && fileInfo.getProject() != <strong class="jxr_keyword">null</strong> )
+<a class="jxr_linenumber" name="L129" href="#L129">129</a> {
+<a class="jxr_linenumber" name="L130" href="#L130">130</a> <strong class="jxr_keyword">return</strong> fileInfo.getProject().getName() + <span class="jxr_string">" - "</span> + filename;
+<a class="jxr_linenumber" name="L131" href="#L131">131</a> }
+<a class="jxr_linenumber" name="L132" href="#L132">132</a> <strong class="jxr_keyword">return</strong> filename;
+<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> <strong class="jxr_keyword">private</strong> <a href="../../../../../org/apache/maven/plugins/pmd/PmdFileInfo.html">PmdFileInfo</a> determineFileInfo( String filename )
+<a class="jxr_linenumber" name="L136" href="#L136">136</a> <strong class="jxr_keyword">throws</strong> IOException
+<a class="jxr_linenumber" name="L137" href="#L137">137</a> {
+<a class="jxr_linenumber" name="L138" href="#L138">138</a> File canonicalFilename = <strong class="jxr_keyword">new</strong> File( filename ).getCanonicalFile();
+<a class="jxr_linenumber" name="L139" href="#L139">139</a> <a href="../../../../../org/apache/maven/plugins/pmd/PmdFileInfo.html">PmdFileInfo</a> fileInfo = files.get( canonicalFilename );
+<a class="jxr_linenumber" name="L140" href="#L140">140</a> <strong class="jxr_keyword">if</strong> ( fileInfo == <strong class="jxr_keyword">null</strong> )
+<a class="jxr_linenumber" name="L141" href="#L141">141</a> {
+<a class="jxr_linenumber" name="L142" href="#L142">142</a> log.warn( <span class="jxr_string">"Couldn't determine PmdFileInfo for file "</span> + filename + <span class="jxr_string">" (canonical: "</span> + canonicalFilename
+<a class="jxr_linenumber" name="L143" href="#L143">143</a> + <span class="jxr_string">"). XRef links won't be available."</span> );
+<a class="jxr_linenumber" name="L144" href="#L144">144</a> }
+<a class="jxr_linenumber" name="L145" href="#L145">145</a>
+<a class="jxr_linenumber" name="L146" href="#L146">146</a> <strong class="jxr_keyword">return</strong> fileInfo;
+<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">private</strong> <strong class="jxr_keyword">void</strong> startFileSection( <strong class="jxr_keyword">int</strong> level, String currentFilename, PmdFileInfo fileInfo )
+<a class="jxr_linenumber" name="L150" href="#L150">150</a> {
+<a class="jxr_linenumber" name="L151" href="#L151">151</a> sink.section( level, <strong class="jxr_keyword">null</strong> );
+<a class="jxr_linenumber" name="L152" href="#L152">152</a> sink.sectionTitle( level, <strong class="jxr_keyword">null</strong> );
+<a class="jxr_linenumber" name="L153" href="#L153">153</a>
+<a class="jxr_linenumber" name="L154" href="#L154">154</a> <em class="jxr_comment">// prepare the filename</em>
+<a class="jxr_linenumber" name="L155" href="#L155">155</a> <strong class="jxr_keyword">this</strong>.currentFilename = shortenFilename( currentFilename, fileInfo );
+<a class="jxr_linenumber" name="L156" href="#L156">156</a>
+<a class="jxr_linenumber" name="L157" href="#L157">157</a> sink.text( makeFileSectionName( <strong class="jxr_keyword">this</strong>.currentFilename, fileInfo ) );
+<a class="jxr_linenumber" name="L158" href="#L158">158</a> sink.sectionTitle_( level );
+<a class="jxr_linenumber" name="L159" href="#L159">159</a>
+<a class="jxr_linenumber" name="L160" href="#L160">160</a> sink.table();
+<a class="jxr_linenumber" name="L161" href="#L161">161</a> sink.tableRow();
+<a class="jxr_linenumber" name="L162" href="#L162">162</a> sink.tableHeaderCell();
+<a class="jxr_linenumber" name="L163" href="#L163">163</a> sink.text( bundle.getString( <span class="jxr_string">"report.pmd.column.rule"</span> ) );
+<a class="jxr_linenumber" name="L164" href="#L164">164</a> sink.tableHeaderCell_();
+<a class="jxr_linenumber" name="L165" href="#L165">165</a> sink.tableHeaderCell();
+<a class="jxr_linenumber" name="L166" href="#L166">166</a> sink.text( bundle.getString( <span class="jxr_string">"report.pmd.column.violation"</span> ) );
+<a class="jxr_linenumber" name="L167" href="#L167">167</a> sink.tableHeaderCell_();
+<a class="jxr_linenumber" name="L168" href="#L168">168</a> <strong class="jxr_keyword">if</strong> ( <strong class="jxr_keyword">this</strong>.renderRuleViolationPriority )
+<a class="jxr_linenumber" name="L169" href="#L169">169</a> {
+<a class="jxr_linenumber" name="L170" href="#L170">170</a> sink.tableHeaderCell();
+<a class="jxr_linenumber" name="L171" href="#L171">171</a> sink.text( bundle.getString( <span class="jxr_string">"report.pmd.column.priority"</span> ) );
+<a class="jxr_linenumber" name="L172" href="#L172">172</a> sink.tableHeaderCell_();
+<a class="jxr_linenumber" name="L173" href="#L173">173</a> }
+<a class="jxr_linenumber" name="L174" href="#L174">174</a> sink.tableHeaderCell();
+<a class="jxr_linenumber" name="L175" href="#L175">175</a> sink.text( bundle.getString( <span class="jxr_string">"report.pmd.column.line"</span> ) );
+<a class="jxr_linenumber" name="L176" href="#L176">176</a> sink.tableHeaderCell_();
+<a class="jxr_linenumber" name="L177" href="#L177">177</a> sink.tableRow_();
+<a class="jxr_linenumber" name="L178" href="#L178">178</a> }
+<a class="jxr_linenumber" name="L179" href="#L179">179</a>
+<a class="jxr_linenumber" name="L180" href="#L180">180</a> <strong class="jxr_keyword">private</strong> <strong class="jxr_keyword">void</strong> endFileSection( <strong class="jxr_keyword">int</strong> level )
+<a class="jxr_linenumber" name="L181" href="#L181">181</a> {
+<a class="jxr_linenumber" name="L182" href="#L182">182</a> sink.table_();
+<a class="jxr_linenumber" name="L183" href="#L183">183</a> sink.section_( level );
+<a class="jxr_linenumber" name="L184" href="#L184">184</a> }
+<a class="jxr_linenumber" name="L185" href="#L185">185</a>
+<a class="jxr_linenumber" name="L186" href="#L186">186</a> <strong class="jxr_keyword">private</strong> <strong class="jxr_keyword">void</strong> processSingleRuleViolation( RuleViolation ruleViolation, PmdFileInfo fileInfo )
+<a class="jxr_linenumber" name="L187" href="#L187">187</a> {
+<a class="jxr_linenumber" name="L188" href="#L188">188</a> sink.tableRow();
+<a class="jxr_linenumber" name="L189" href="#L189">189</a> sink.tableCell();
+<a class="jxr_linenumber" name="L190" href="#L190">190</a> sink.link( ruleViolation.getRule().getExternalInfoUrl() );
+<a class="jxr_linenumber" name="L191" href="#L191">191</a> sink.text( ruleViolation.getRule().getName() );
+<a class="jxr_linenumber" name="L192" href="#L192">192</a> sink.link_();
+<a class="jxr_linenumber" name="L193" href="#L193">193</a> sink.tableCell_();
+<a class="jxr_linenumber" name="L194" href="#L194">194</a> sink.tableCell();
+<a class="jxr_linenumber" name="L195" href="#L195">195</a> sink.text( ruleViolation.getDescription() );
+<a class="jxr_linenumber" name="L196" href="#L196">196</a> sink.tableCell_();
<a class="jxr_linenumber" name="L197" href="#L197">197</a>
-<a class="jxr_linenumber" name="L198" href="#L198">198</a> <strong class="jxr_keyword">int</strong> beginLine = ruleViolation.getBeginLine();
-<a class="jxr_linenumber" name="L199" href="#L199">199</a> outputLineLink( beginLine, fileInfo );
-<a class="jxr_linenumber" name="L200" href="#L200">200</a> <strong class="jxr_keyword">int</strong> endLine = ruleViolation.getEndLine();
-<a class="jxr_linenumber" name="L201" href="#L201">201</a> <strong class="jxr_keyword">if</strong> ( endLine != beginLine )
-<a class="jxr_linenumber" name="L202" href="#L202">202</a> {
-<a class="jxr_linenumber" name="L203" href="#L203">203</a> sink.text( <span class="jxr_string">"&#x2013;"</span> );
-<a class="jxr_linenumber" name="L204" href="#L204">204</a> outputLineLink( endLine, fileInfo );
-<a class="jxr_linenumber" name="L205" href="#L205">205</a> }
+<a class="jxr_linenumber" name="L198" href="#L198">198</a> <strong class="jxr_keyword">if</strong> ( <strong class="jxr_keyword">this</strong>.renderRuleViolationPriority )
+<a class="jxr_linenumber" name="L199" href="#L199">199</a> {
+<a class="jxr_linenumber" name="L200" href="#L200">200</a> sink.tableCell();
+<a class="jxr_linenumber" name="L201" href="#L201">201</a> sink.text( String.valueOf( ruleViolation.getRule().getPriority().getPriority() ) );
+<a class="jxr_linenumber" name="L202" href="#L202">202</a> sink.tableCell_();
+<a class="jxr_linenumber" name="L203" href="#L203">203</a> }
+<a class="jxr_linenumber" name="L204" href="#L204">204</a>
+<a class="jxr_linenumber" name="L205" href="#L205">205</a> sink.tableCell();
<a class="jxr_linenumber" name="L206" href="#L206">206</a>
-<a class="jxr_linenumber" name="L207" href="#L207">207</a> sink.tableCell_();
-<a class="jxr_linenumber" name="L208" href="#L208">208</a> sink.tableRow_();
-<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> <em class="jxr_comment">// PMD might run the analysis multi-threaded, so the violations might be reported</em>
-<a class="jxr_linenumber" name="L212" href="#L212">212</a> <em class="jxr_comment">// out of order. We sort them here by filename and line number before writing them to</em>
-<a class="jxr_linenumber" name="L213" href="#L213">213</a> <em class="jxr_comment">// the report.</em>
-<a class="jxr_linenumber" name="L214" href="#L214">214</a> <strong class="jxr_keyword">private</strong> <strong class="jxr_keyword">void</strong> processViolations()
-<a class="jxr_linenumber" name="L215" href="#L215">215</a> <strong class="jxr_keyword">throws</strong> IOException
-<a class="jxr_linenumber" name="L216" href="#L216">216</a> {
-<a class="jxr_linenumber" name="L217" href="#L217">217</a> sink.section1();
-<a class="jxr_linenumber" name="L218" href="#L218">218</a> sink.sectionTitle1();
-<a class="jxr_linenumber" name="L219" href="#L219">219</a> sink.text( bundle.getString( <span class="jxr_string">"report.pmd.files"</span> ) );
-<a class="jxr_linenumber" name="L220" href="#L220">220</a> sink.sectionTitle1_();
-<a class="jxr_linenumber" name="L221" href="#L221">221</a>
-<a class="jxr_linenumber" name="L222" href="#L222">222</a> <em class="jxr_comment">// TODO files summary</em>
-<a class="jxr_linenumber" name="L223" href="#L223">223</a>
-<a class="jxr_linenumber" name="L224" href="#L224">224</a> fileCount = files.size();
-<a class="jxr_linenumber" name="L225" href="#L225">225</a> List<RuleViolation> violations2 = <strong class="jxr_keyword">new</strong> ArrayList<>( violations );
-<a class="jxr_linenumber" name="L226" href="#L226">226</a> Collections.sort( violations2, <strong class="jxr_keyword">new</strong> Comparator<RuleViolation>()
-<a class="jxr_linenumber" name="L227" href="#L227">227</a> {
-<a class="jxr_linenumber" name="L228" href="#L228">228</a> <em class="jxr_javadoccomment">/** {@inheritDoc} */</em>
-<a class="jxr_linenumber" name="L229" href="#L229">229</a> <strong class="jxr_keyword">public</strong> <strong class="jxr_keyword">int</strong> compare( RuleViolation o1, RuleViolation o2 )
-<a class="jxr_linenumber" name="L230" href="#L230">230</a> {
-<a class="jxr_linenumber" name="L231" href="#L231">231</a> <strong class="jxr_keyword">int</strong> filenames = o1.getFilename().compareTo( o2.getFilename() );
-<a class="jxr_linenumber" name="L232" href="#L232">232</a> <strong class="jxr_keyword">if</strong> ( filenames == 0 )
-<a class="jxr_linenumber" name="L233" href="#L233">233</a> {
-<a class="jxr_linenumber" name="L234" href="#L234">234</a> <strong class="jxr_keyword">return</strong> o1.getBeginLine() - o2.getBeginLine();
-<a class="jxr_linenumber" name="L235" href="#L235">235</a> }
-<a class="jxr_linenumber" name="L236" href="#L236">236</a> <strong class="jxr_keyword">else</strong>
-<a class="jxr_linenumber" name="L237" href="#L237">237</a> {
-<a class="jxr_linenumber" name="L238" href="#L238">238</a> <strong class="jxr_keyword">return</strong> filenames;
-<a class="jxr_linenumber" name="L239" href="#L239">239</a> }
-<a class="jxr_linenumber" name="L240" href="#L240">240</a> }
-<a class="jxr_linenumber" name="L241" href="#L241">241</a> } );
-<a class="jxr_linenumber" name="L242" href="#L242">242</a>
-<a class="jxr_linenumber" name="L243" href="#L243">243</a> <strong class="jxr_keyword">boolean</strong> fileSectionStarted = false;
-<a class="jxr_linenumber" name="L244" href="#L244">244</a> String previousFilename = <strong class="jxr_keyword">null</strong>;
-<a class="jxr_linenumber" name="L245" href="#L245">245</a> <strong class="jxr_keyword">for</strong> ( RuleViolation ruleViolation : violations2 )
-<a class="jxr_linenumber" name="L246" href="#L246">246</a> {
-<a class="jxr_linenumber" name="L247" href="#L247">247</a> String currentFn = ruleViolation.getFilename();
-<a class="jxr_linenumber" name="L248" href="#L248">248</a> <a href="../../../../../org/apache/maven/plugins/pmd/PmdFileInfo.html">PmdFileInfo</a> fileInfo = determineFileInfo( currentFn );
-<a class="jxr_linenumber" name="L249" href="#L249">249</a>
-<a class="jxr_linenumber" name="L250" href="#L250">250</a> <strong class="jxr_keyword">if</strong> ( !currentFn.equalsIgnoreCase( previousFilename ) && fileSectionStarted )
-<a class="jxr_linenumber" name="L251" href="#L251">251</a> {
-<a class="jxr_linenumber" name="L252" href="#L252">252</a> endFileSection();
-<a class="jxr_linenumber" name="L253" href="#L253">253</a> fileSectionStarted = false;
-<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> ( !fileSectionStarted )
-<a class="jxr_linenumber" name="L256" href="#L256">256</a> {
-<a class="jxr_linenumber" name="L257" href="#L257">257</a> startFileSection( currentFn, fileInfo );
-<a class="jxr_linenumber" name="L258" href="#L258">258</a> fileSectionStarted = <strong class="jxr_keyword">true</strong>;
-<a class="jxr_linenumber" name="L259" href="#L259">259</a> }
-<a class="jxr_linenumber" name="L260" href="#L260">260</a>
-<a class="jxr_linenumber" name="L261" href="#L261">261</a> processSingleRuleViolation( ruleViolation, fileInfo );
-<a class="jxr_linenumber" name="L262" href="#L262">262</a>
-<a class="jxr_linenumber" name="L263" href="#L263">263</a> previousFilename = currentFn;
-<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> ( fileSectionStarted )
-<a class="jxr_linenumber" name="L267" href="#L267">267</a> {
-<a class="jxr_linenumber" name="L268" href="#L268">268</a> endFileSection();
-<a class="jxr_linenumber" name="L269" href="#L269">269</a> }
-<a class="jxr_linenumber" name="L270" href="#L270">270</a>
-<a class="jxr_linenumber" name="L271" href="#L271">271</a> <strong class="jxr_keyword">if</strong> ( fileCount == 0 )
-<a class="jxr_linenumber" name="L272" href="#L272">272</a> {
-<a class="jxr_linenumber" name="L273" href="#L273">273</a> sink.paragraph();
-<a class="jxr_linenumber" name="L274" href="#L274">274</a> sink.text( bundle.getString( <span class="jxr_string">"report.pmd.noProblems"</span> ) );
-<a class="jxr_linenumber" name="L275" href="#L275">275</a> sink.paragraph_();
-<a class="jxr_linenumber" name="L276" href="#L276">276</a> }
-<a class="jxr_linenumber" name="L277" href="#L277">277</a>
-<a class="jxr_linenumber" name="L278" href="#L278">278</a> sink.section1_();
-<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> <strong class="jxr_keyword">private</strong> <strong class="jxr_keyword">void</strong> outputLineLink( <strong class="jxr_keyword">int</strong> line, PmdFileInfo fileInfo )
-<a class="jxr_linenumber" name="L282" href="#L282">282</a> {
-<a class="jxr_linenumber" name="L283" href="#L283">283</a> String xrefLocation = <strong class="jxr_keyword">null</strong>;
-<a class="jxr_linenumber" name="L284" href="#L284">284</a> <strong class="jxr_keyword">if</strong> ( fileInfo != <strong class="jxr_keyword">null</strong> )
-<a class="jxr_linenumber" name="L285" href="#L285">285</a> {
-<a class="jxr_linenumber" name="L286" href="#L286">286</a> xrefLocation = fileInfo.getXrefLocation();
-<a class="jxr_linenumber" name="L287" href="#L287">287</a> }
-<a class="jxr_linenumber" name="L288" href="#L288">288</a>
-<a class="jxr_linenumber" name="L289" href="#L289">289</a> <strong class="jxr_keyword">if</strong> ( xrefLocation != <strong class="jxr_keyword">null</strong> )
-<a class="jxr_linenumber" name="L290" href="#L290">290</a> {
-<a class="jxr_linenumber" name="L291" href="#L291">291</a> sink.link( xrefLocation + <span class="jxr_string">"/"</span> + currentFilename.replaceAll( <span class="jxr_string">"\\.java$"</span>, <span class="jxr_string">".html"</span> ) + <span class="jxr_string">"#L"</span> + line );
-<a class="jxr_linenumber" name="L292" href="#L292">292</a> }
-<a class="jxr_linenumber" name="L293" href="#L293">293</a> sink.text( String.valueOf( line ) );
-<a class="jxr_linenumber" name="L294" href="#L294">294</a> <strong class="jxr_keyword">if</strong> ( xrefLocation != <strong class="jxr_keyword">null</strong> )
-<a class="jxr_linenumber" name="L295" href="#L295">295</a> {
-<a class="jxr_linenumber" name="L296" href="#L296">296</a> sink.link_();
-<a class="jxr_linenumber" name="L297" href="#L297">297</a> }
-<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">private</strong> <strong class="jxr_keyword">void</strong> processProcessingErrors() <strong class="jxr_keyword">throws</strong> IOException
-<a class="jxr_linenumber" name="L301" href="#L301">301</a> {
-<a class="jxr_linenumber" name="L302" href="#L302">302</a> <em class="jxr_comment">// sort the problem by filename first, since PMD is executed multi-threaded</em>
-<a class="jxr_linenumber" name="L303" href="#L303">303</a> <em class="jxr_comment">// and might reports the results unsorted</em>
-<a class="jxr_linenumber" name="L304" href="#L304">304</a> Collections.sort( processingErrors, <strong class="jxr_keyword">new</strong> Comparator<ProcessingError>()
-<a class="jxr_linenumber" name="L305" href="#L305">305</a> {
-<a class="jxr_linenumber" name="L306" href="#L306">306</a> @Override
-<a class="jxr_linenumber" name="L307" href="#L307">307</a> <strong class="jxr_keyword">public</strong> <strong class="jxr_keyword">int</strong> compare( ProcessingError e1, ProcessingError e2 )
-<a class="jxr_linenumber" name="L308" href="#L308">308</a> {
-<a class="jxr_linenumber" name="L309" href="#L309">309</a> <strong class="jxr_keyword">return</strong> e1.getFile().compareTo( e2.getFile() );
-<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>
-<a class="jxr_linenumber" name="L313" href="#L313">313</a> sink.section1();
-<a class="jxr_linenumber" name="L314" href="#L314">314</a> sink.sectionTitle1();
-<a class="jxr_linenumber" name="L315" href="#L315">315</a> sink.text( bundle.getString( <span class="jxr_string">"report.pmd.processingErrors.title"</span> ) );
-<a class="jxr_linenumber" name="L316" href="#L316">316</a> sink.sectionTitle1_();
-<a class="jxr_linenumber" name="L317" href="#L317">317</a>
-<a class="jxr_linenumber" name="L318" href="#L318">318</a> sink.table();
-<a class="jxr_linenumber" name="L319" href="#L319">319</a> sink.tableRow();
-<a class="jxr_linenumber" name="L320" href="#L320">320</a> sink.tableHeaderCell();
-<a class="jxr_linenumber" name="L321" href="#L321">321</a> sink.text( bundle.getString( <span class="jxr_string">"report.pmd.processingErrors.column.filename"</span> ) );
-<a class="jxr_linenumber" name="L322" href="#L322">322</a> sink.tableHeaderCell_();
-<a class="jxr_linenumber" name="L323" href="#L323">323</a> sink.tableHeaderCell();
-<a class="jxr_linenumber" name="L324" href="#L324">324</a> sink.text( bundle.getString( <span class="jxr_string">"report.pmd.processingErrors.column.problem"</span> ) );
-<a class="jxr_linenumber" name="L325" href="#L325">325</a> sink.tableHeaderCell_();
-<a class="jxr_linenumber" name="L326" href="#L326">326</a> sink.tableRow_();
-<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> ( ProcessingError error : processingErrors )
-<a class="jxr_linenumber" name="L329" href="#L329">329</a> {
-<a class="jxr_linenumber" name="L330" href="#L330">330</a> processSingleProcessingError( error );
-<a class="jxr_linenumber" name="L331" href="#L331">331</a> }
-<a class="jxr_linenumber" name="L332" href="#L332">332</a>
-<a class="jxr_linenumber" name="L333" href="#L333">333</a> sink.table_();
+<a class="jxr_linenumber" name="L207" href="#L207">207</a> <strong class="jxr_keyword">int</strong> beginLine = ruleViolation.getBeginLine();
+<a class="jxr_linenumber" name="L208" href="#L208">208</a> outputLineLink( beginLine, fileInfo );
+<a class="jxr_linenumber" name="L209" href="#L209">209</a> <strong class="jxr_keyword">int</strong> endLine = ruleViolation.getEndLine();
+<a class="jxr_linenumber" name="L210" href="#L210">210</a> <strong class="jxr_keyword">if</strong> ( endLine != beginLine )
+<a class="jxr_linenumber" name="L211" href="#L211">211</a> {
+<a class="jxr_linenumber" name="L212" href="#L212">212</a> sink.text( <span class="jxr_string">"&#x2013;"</span> ); <em class="jxr_comment">// \u2013 is a medium long dash character</em>
+<a class="jxr_linenumber" name="L213" href="#L213">213</a> outputLineLink( endLine, fileInfo );
+<a class="jxr_linenumber" name="L214" href="#L214">214</a> }
+<a class="jxr_linenumber" name="L215" href="#L215">215</a>
+<a class="jxr_linenumber" name="L216" href="#L216">216</a> sink.tableCell_();
+<a class="jxr_linenumber" name="L217" href="#L217">217</a> sink.tableRow_();
+<a class="jxr_linenumber" name="L218" href="#L218">218</a> }
+<a class="jxr_linenumber" name="L219" href="#L219">219</a>
+<a class="jxr_linenumber" name="L220" href="#L220">220</a> <em class="jxr_comment">// PMD might run the analysis multi-threaded, so the violations might be reported</em>
+<a class="jxr_linenumber" name="L221" href="#L221">221</a> <em class="jxr_comment">// out of order. We sort them here by filename and line number before writing them to</em>
+<a class="jxr_linenumber" name="L222" href="#L222">222</a> <em class="jxr_comment">// the report.</em>
+<a class="jxr_linenumber" name="L223" href="#L223">223</a> <strong class="jxr_keyword">private</strong> <strong class="jxr_keyword">void</strong> renderViolations()
+<a class="jxr_linenumber" name="L224" href="#L224">224</a> <strong class="jxr_keyword">throws</strong> IOException
+<a class="jxr_linenumber" name="L225" href="#L225">225</a> {
+<a class="jxr_linenumber" name="L226" href="#L226">226</a> sink.section1();
+<a class="jxr_linenumber" name="L227" href="#L227">227</a> sink.sectionTitle1();
+<a class="jxr_linenumber" name="L228" href="#L228">228</a> sink.text( bundle.getString( <span class="jxr_string">"report.pmd.files"</span> ) );
+<a class="jxr_linenumber" name="L229" href="#L229">229</a> sink.sectionTitle1_();
+<a class="jxr_linenumber" name="L230" href="#L230">230</a>
+<a class="jxr_linenumber" name="L231" href="#L231">231</a> <em class="jxr_comment">// TODO files summary</em>
+<a class="jxr_linenumber" name="L232" href="#L232">232</a>
+<a class="jxr_linenumber" name="L233" href="#L233">233</a> List<RuleViolation> violations2 = <strong class="jxr_keyword">new</strong> ArrayList<>( violations );
+<a class="jxr_linenumber" name="L234" href="#L234">234</a> renderViolationsTable( 2, violations2 );
+<a class="jxr_linenumber" name="L235" href="#L235">235</a>
+<a class="jxr_linenumber" name="L236" href="#L236">236</a> sink.section1_();
+<a class="jxr_linenumber" name="L237" href="#L237">237</a> }
+<a class="jxr_linenumber" name="L238" href="#L238">238</a>
+<a class="jxr_linenumber" name="L239" href="#L239">239</a> <strong class="jxr_keyword">private</strong> <strong class="jxr_keyword">void</strong> renderViolationsByPriority() <strong class="jxr_keyword">throws</strong> IOException
+<a class="jxr_linenumber" name="L240" href="#L240">240</a> {
+<a class="jxr_linenumber" name="L241" href="#L241">241</a> <strong class="jxr_keyword">if</strong> ( !renderViolationsByPriority )
+<a class="jxr_linenumber" name="L242" href="#L242">242</a> {
+<a class="jxr_linenumber" name="L243" href="#L243">243</a> <strong class="jxr_keyword">return</strong>;
+<a class="jxr_linenumber" name="L244" href="#L244">244</a> }
+<a class="jxr_linenumber" name="L245" href="#L245">245</a>
+<a class="jxr_linenumber" name="L246" href="#L246">246</a> <strong class="jxr_keyword">boolean</strong> oldPriorityColumn = <strong class="jxr_keyword">this</strong>.renderRuleViolationPriority;
+<a class="jxr_linenumber" name="L247" href="#L247">247</a> <strong class="jxr_keyword">this</strong>.renderRuleViolationPriority = false;
+<a class="jxr_linenumber" name="L248" href="#L248">248</a>
+<a class="jxr_linenumber" name="L249" href="#L249">249</a> sink.section1();
+<a class="jxr_linenumber" name="L250" href="#L250">250</a> sink.sectionTitle1();
+<a class="jxr_linenumber" name="L251" href="#L251">251</a> sink.text( bundle.getString( <span class="jxr_string">"report.pmd.violationsByPriority"</span> ) );
+<a class="jxr_linenumber" name="L252" href="#L252">252</a> sink.sectionTitle1_();
+<a class="jxr_linenumber" name="L253" href="#L253">253</a>
+<a class="jxr_linenumber" name="L254" href="#L254">254</a> Map<RulePriority, List<RuleViolation>> violationsByPriority = <strong class="jxr_keyword">new</strong> HashMap<>();
+<a class="jxr_linenumber" name="L255" href="#L255">255</a> <strong class="jxr_keyword">for</strong> ( RuleViolation violation : violations )
+<a class="jxr_linenumber" name="L256" href="#L256">256</a> {
+<a class="jxr_linenumber" name="L257" href="#L257">257</a> RulePriority priority = violation.getRule().getPriority();
+<a class="jxr_linenumber" name="L258" href="#L258">258</a> List<RuleViolation> violationSegment = violationsByPriority.get( priority );
+<a class="jxr_linenumber" name="L259" href="#L259">259</a> <strong class="jxr_keyword">if</strong> ( violationSegment == <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> violationSegment = <strong class="jxr_keyword">new</strong> ArrayList<>();
+<a class="jxr_linenumber" name="L262" href="#L262">262</a> violationsByPriority.put( priority, violationSegment );
+<a class="jxr_linenumber" name="L263" href="#L263">263</a> }
+<a class="jxr_linenumber" name="L264" href="#L264">264</a> violationsByPriority.get( violation.getRule().getPriority() ).add( violation );
+<a class="jxr_linenumber" name="L265" href="#L265">265</a> }
+<a class="jxr_linenumber" name="L266" href="#L266">266</a>
+<a class="jxr_linenumber" name="L267" href="#L267">267</a> <strong class="jxr_keyword">for</strong> ( RulePriority priority : RulePriority.values() )
+<a class="jxr_linenumber" name="L268" href="#L268">268</a> {
+<a class="jxr_linenumber" name="L269" href="#L269">269</a> List<RuleViolation> violationsWithPriority = violationsByPriority.get( priority );
+<a class="jxr_linenumber" name="L270" href="#L270">270</a> <strong class="jxr_keyword">if</strong> ( violationsWithPriority == <strong class="jxr_keyword">null</strong> || violationsWithPriority.isEmpty() )
+<a class="jxr_linenumber" name="L271" href="#L271">271</a> {
+<a class="jxr_linenumber" name="L272" href="#L272">272</a> <strong class="jxr_keyword">continue</strong>;
+<a class="jxr_linenumber" name="L273" href="#L273">273</a> }
+<a class="jxr_linenumber" name="L274" href="#L274">274</a>
+<a class="jxr_linenumber" name="L275" href="#L275">275</a> sink.section2();
+<a class="jxr_linenumber" name="L276" href="#L276">276</a> sink.sectionTitle2();
+<a class="jxr_linenumber" name="L277" href="#L277">277</a> sink.text( bundle.getString( <span class="jxr_string">"report.pmd.priority"</span> ) + <span class="jxr_string">" "</span> + priority.getPriority() );
+<a class="jxr_linenumber" name="L278" href="#L278">278</a> sink.sectionTitle2_();
+<a class="jxr_linenumber" name="L279" href="#L279">279</a>
+<a class="jxr_linenumber" name="L280" href="#L280">280</a> renderViolationsTable( 3, violationsWithPriority );
+<a class="jxr_linenumber" name="L281" href="#L281">281</a>
+<a class="jxr_linenumber" name="L282" href="#L282">282</a> sink.section2_();
+<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> <strong class="jxr_keyword">if</strong> ( violations.isEmpty() )
+<a class="jxr_linenumber" name="L286" href="#L286">286</a> {
+<a class="jxr_linenumber" name="L287" href="#L287">287</a> sink.paragraph();
+<a class="jxr_linenumber" name="L288" href="#L288">288</a> sink.text( bundle.getString( <span class="jxr_string">"report.pmd.noProblems"</span> ) );
+<a class="jxr_linenumber" name="L289" href="#L289">289</a> sink.paragraph_();
+<a class="jxr_linenumber" name="L290" href="#L290">290</a> }
+<a class="jxr_linenumber" name="L291" href="#L291">291</a>
+<a class="jxr_linenumber" name="L292" href="#L292">292</a> sink.section1_();
+<a class="jxr_linenumber" name="L293" href="#L293">293</a>
+<a class="jxr_linenumber" name="L294" href="#L294">294</a> <strong class="jxr_keyword">this</strong>.renderRuleViolationPriority = oldPriorityColumn;
+<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">private</strong> <strong class="jxr_keyword">void</strong> renderViolationsTable( <strong class="jxr_keyword">int</strong> level, List<RuleViolation> violationSegment )
+<a class="jxr_linenumber" name="L298" href="#L298">298</a> <strong class="jxr_keyword">throws</strong> IOException
+<a class="jxr_linenumber" name="L299" href="#L299">299</a> {
+<a class="jxr_linenumber" name="L300" href="#L300">300</a> Collections.sort( violationSegment, <strong class="jxr_keyword">new</strong> Comparator<RuleViolation>()
+<a class="jxr_linenumber" name="L301" href="#L301">301</a> {
+<a class="jxr_linenumber" name="L302" href="#L302">302</a> <em class="jxr_javadoccomment">/** {@inheritDoc} */</em>
+<a class="jxr_linenumber" name="L303" href="#L303">303</a> <strong class="jxr_keyword">public</strong> <strong class="jxr_keyword">int</strong> compare( RuleViolation o1, RuleViolation o2 )
+<a class="jxr_linenumber" name="L304" href="#L304">304</a> {
+<a class="jxr_linenumber" name="L305" href="#L305">305</a> <strong class="jxr_keyword">int</strong> filenames = o1.getFilename().compareTo( o2.getFilename() );
+<a class="jxr_linenumber" name="L306" href="#L306">306</a> <strong class="jxr_keyword">if</strong> ( filenames == 0 )
+<a class="jxr_linenumber" name="L307" href="#L307">307</a> {
+<a class="jxr_linenumber" name="L308" href="#L308">308</a> <strong class="jxr_keyword">return</strong> o1.getBeginLine() - o2.getBeginLine();
+<a class="jxr_linenumber" name="L309" href="#L309">309</a> }
+<a class="jxr_linenumber" name="L310" href="#L310">310</a> <strong class="jxr_keyword">else</strong>
+<a class="jxr_linenumber" name="L311" href="#L311">311</a> {
+<a class="jxr_linenumber" name="L312" href="#L312">312</a> <strong class="jxr_keyword">return</strong> filenames;
+<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> } );
+<a class="jxr_linenumber" name="L316" href="#L316">316</a>
+<a class="jxr_linenumber" name="L317" href="#L317">317</a> <strong class="jxr_keyword">boolean</strong> fileSectionStarted = false;
+<a class="jxr_linenumber" name="L318" href="#L318">318</a> String previousFilename = <strong class="jxr_keyword">null</strong>;
+<a class="jxr_linenumber" name="L319" href="#L319">319</a> <strong class="jxr_keyword">for</strong> ( RuleViolation ruleViolation : violationSegment )
+<a class="jxr_linenumber" name="L320" href="#L320">320</a> {
+<a class="jxr_linenumber" name="L321" href="#L321">321</a> String currentFn = ruleViolation.getFilename();
+<a class="jxr_linenumber" name="L322" href="#L322">322</a> <a href="../../../../../org/apache/maven/plugins/pmd/PmdFileInfo.html">PmdFileInfo</a> fileInfo = determineFileInfo( currentFn );
+<a class="jxr_linenumber" name="L323" href="#L323">323</a>
+<a class="jxr_linenumber" name="L324" href="#L324">324</a> <strong class="jxr_keyword">if</strong> ( !currentFn.equalsIgnoreCase( previousFilename ) && fileSectionStarted )
+<a class="jxr_linenumber" name="L325" href="#L325">325</a> {
+<a class="jxr_linenumber" name="L326" href="#L326">326</a> endFileSection( level );
+<a class="jxr_linenumber" name="L327" href="#L327">327</a> fileSectionStarted = false;
+<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> ( !fileSectionStarted )
+<a class="jxr_linenumber" name="L330" href="#L330">330</a> {
+<a class="jxr_linenumber" name="L331" href="#L331">331</a> startFileSection( level, currentFn, fileInfo );
+<a class="jxr_linenumber" name="L332" href="#L332">332</a> fileSectionStarted = <strong class="jxr_keyword">true</strong>;
+<a class="jxr_linenumber" name="L333" href="#L333">333</a> }
<a class="jxr_linenumber" name="L334" href="#L334">334</a>
-<a class="jxr_linenumber" name="L335" href="#L335">335</a> sink.section1_();
-<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> <strong class="jxr_keyword">private</strong> <strong class="jxr_keyword">void</strong> processSingleProcessingError( ProcessingError error ) <strong class="jxr_keyword">throws</strong> IOException
-<a class="jxr_linenumber" name="L339" href="#L339">339</a> {
-<a class="jxr_linenumber" name="L340" href="#L340">340</a> String filename = error.getFile();
-<a class="jxr_linenumber" name="L341" href="#L341">341</a> <a href="../../../../../org/apache/maven/plugins/pmd/PmdFileInfo.html">PmdFileInfo</a> fileInfo = determineFileInfo( filename );
-<a class="jxr_linenumber" name="L342" href="#L342">342</a> filename = makeFileSectionName( shortenFilename( filename, fileInfo ), fileInfo );
-<a class="jxr_linenumber" name="L343" href="#L343">343</a>
-<a class="jxr_linenumber" name="L344" href="#L344">344</a> sink.tableRow();
-<a class="jxr_linenumber" name="L345" href="#L345">345</a> sink.tableCell();
-<a class="jxr_linenumber" name="L346" href="#L346">346</a> sink.text( filename );
-<a class="jxr_linenumber" name="L347" href="#L347">347</a> sink.tableCell_();
-<a class="jxr_linenumber" name="L348" href="#L348">348</a> sink.tableCell();
-<a class="jxr_linenumber" name="L349" href="#L349">349</a> sink.text( error.getMsg() );
-<a class="jxr_linenumber" name="L350" href="#L350">350</a> sink.verbatim( <strong class="jxr_keyword">null</strong> );
-<a class="jxr_linenumber" name="L351" href="#L351">351</a> sink.rawText( error.getDetail() );
-<a class="jxr_linenumber" name="L352" href="#L352">352</a> sink.verbatim_();
-<a class="jxr_linenumber" name="L353" href="#L353">353</a> sink.tableCell_();
-<a class="jxr_linenumber" name="L354" href="#L354">354</a> sink.tableRow_();
-<a class="jxr_linenumber" name="L355" href="#L355">355</a> }
-<a class="jxr_linenumber" name="L356" href="#L356">356</a>
-<a class="jxr_linenumber" name="L357" href="#L357">357</a> <strong class="jxr_keyword">public</strong> <strong class="jxr_keyword">void</strong> beginDocument()
-<a class="jxr_linenumber" name="L358" href="#L358">358</a> {
-<a class="jxr_linenumber" name="L359" href="#L359">359</a> sink.head();
-<a class="jxr_linenumber" name="L360" href="#L360">360</a> sink.title();
-<a class="jxr_linenumber" name="L361" href="#L361">361</a> sink.text( getTitle() );
-<a class="jxr_linenumber" name="L362" href="#L362">362</a> sink.title_();
-<a class="jxr_linenumber" name="L363" href="#L363">363</a> sink.head_();
+<a class="jxr_linenumber" name="L335" href="#L335">335</a> processSingleRuleViolation( ruleViolation, fileInfo );
+<a class="jxr_linenumber" name="L336" href="#L336">336</a>
+<a class="jxr_linenumber" name="L337" href="#L337">337</a> previousFilename = currentFn;
+<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> ( fileSectionStarted )
+<a class="jxr_linenumber" name="L341" href="#L341">341</a> {
+<a class="jxr_linenumber" name="L342" href="#L342">342</a> endFileSection( level );
+<a class="jxr_linenumber" name="L343" href="#L343">343</a> }
+<a class="jxr_linenumber" name="L344" href="#L344">344</a> }
+<a class="jxr_linenumber" name="L345" href="#L345">345</a>
+<a class="jxr_linenumber" name="L346" href="#L346">346</a> <strong class="jxr_keyword">private</strong> <strong class="jxr_keyword">void</strong> outputLineLink( <strong class="jxr_keyword">int</strong> line, PmdFileInfo fileInfo )
+<a class="jxr_linenumber" name="L347" href="#L347">347</a> {
+<a class="jxr_linenumber" name="L348" href="#L348">348</a> String xrefLocation = <strong class="jxr_keyword">null</strong>;
+<a class="jxr_linenumber" name="L349" href="#L349">349</a> <strong class="jxr_keyword">if</strong> ( fileInfo != <strong class="jxr_keyword">null</strong> )
+<a class="jxr_linenumber" name="L350" href="#L350">350</a> {
+<a class="jxr_linenumber" name="L351" href="#L351">351</a> xrefLocation = fileInfo.getXrefLocation();
+<a class="jxr_linenumber" name="L352" href="#L352">352</a> }
+<a class="jxr_linenumber" name="L353" href="#L353">353</a>
+<a class="jxr_linenumber" name="L354" href="#L354">354</a> <strong class="jxr_keyword">if</strong> ( xrefLocation != <strong class="jxr_keyword">null</strong> )
+<a class="jxr_linenumber" name="L355" href="#L355">355</a> {
+<a class="jxr_linenumber" name="L356" href="#L356">356</a> sink.link( xrefLocation + <span class="jxr_string">"/"</span> + currentFilename.replaceAll( <span class="jxr_string">"\\.java$"</span>, <span class="jxr_string">".html"</span> ) + <span class="jxr_string">"#L"</span> + line );
+<a class="jxr_linenumber" name="L357" href="#L357">357</a> }
+<a class="jxr_linenumber" name="L358" href="#L358">358</a> sink.text( String.valueOf( line ) );
+<a class="jxr_linenumber" name="L359" href="#L359">359</a> <strong class="jxr_keyword">if</strong> ( xrefLocation != <strong class="jxr_keyword">null</strong> )
+<a class="jxr_linenumber" name="L360" href="#L360">360</a> {
+<a class="jxr_linenumber" name="L361" href="#L361">361</a> sink.link_();
+<a class="jxr_linenumber" name="L362" href="#L362">362</a> }
+<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> sink.body();
-<a class="jxr_linenumber" name="L366" href="#L366">366</a>
-<a class="jxr_linenumber" name="L367" href="#L367">367</a> sink.section1();
-<a class="jxr_linenumber" name="L368" href="#L368">368</a> sink.sectionTitle1();
-<a class="jxr_linenumber" name="L369" href="#L369">369</a> sink.text( getTitle() );
-<a class="jxr_linenumber" name="L370" href="#L370">370</a> sink.sectionTitle1_();
-<a class="jxr_linenumber" name="L371" href="#L371">371</a>
-<a class="jxr_linenumber" name="L372" href="#L372">372</a> sink.paragraph();
-<a class="jxr_linenumber" name="L373" href="#L373">373</a> sink.text( bundle.getString( <span class="jxr_string">"report.pmd.pmdlink"</span> ) + <span class="jxr_string">" "</span> );
-<a class="jxr_linenumber" name="L374" href="#L374">374</a> sink.link( <span class="jxr_string">"https://pmd.github.io"</span> );
-<a class="jxr_linenumber" name="L375" href="#L375">375</a> sink.text( <span class="jxr_string">"PMD"</span> );
-<a class="jxr_linenumber" name="L376" href="#L376">376</a> sink.link_();
-<a class="jxr_linenumber" name="L377" href="#L377">377</a> sink.text( <span class="jxr_string">" "</span> + AbstractPmdReport.getPmdVersion() + <span class="jxr_string">"."</span> );
-<a class="jxr_linenumber" name="L378" href="#L378">378</a> sink.paragraph_();
-<a class="jxr_linenumber" name="L379" href="#L379">379</a>
-<a class="jxr_linenumber" name="L380" href="#L380">380</a> sink.section1_();
-<a class="jxr_linenumber" name="L381" href="#L381">381</a>
-<a class="jxr_linenumber" name="L382" href="#L382">382</a> <em class="jxr_comment">// TODO overall summary</em>
-<a class="jxr_linenumber" name="L383" href="#L383">383</a> }
-<a class="jxr_linenumber" name="L384" href="#L384">384</a>
-<a class="jxr_linenumber" name="L385" href="#L385">385</a> <em class="jxr_comment">/*</em>
-<a class="jxr_linenumber" name="L386" href="#L386">386</a> <em class="jxr_comment"> * private void processMetrics() { if ( metrics.size() == 0 ) { return; } sink.section1(); sink.sectionTitle1();</em>
-<a class="jxr_linenumber" name="L387" href="#L387">387</a> <em class="jxr_comment"> * sink.text( "Metrics" ); sink.sectionTitle1_(); sink.table(); sink.tableRow(); sink.tableHeaderCell(); sink.text(</em>
-<a class="jxr_linenumber" name="L388" href="#L388">388</a> <em class="jxr_comment"> * "Name" ); sink.tableHeaderCell_(); sink.tableHeaderCell(); sink.text( "Count" ); sink.tableHeaderCell_();</em>
-<a class="jxr_linenumber" name="L389" href="#L389">389</a> <em class="jxr_comment"> * sink.tableHeaderCell(); sink.text( "High" ); sink.tableHeaderCell_(); sink.tableHeaderCell(); sink.text( "Low" );</em>
-<a class="jxr_linenumber" name="L390" href="#L390">390</a> <em class="jxr_comment"> * sink.tableHeaderCell_(); sink.tableHeaderCell(); sink.text( "Average" ); sink.tableHeaderCell_();</em>
-<a class="jxr_linenumber" name="L391" href="#L391">391</a> <em class="jxr_comment"> * sink.tableRow_(); for ( Metric met : metrics ) { sink.tableRow(); sink.tableCell(); sink.text(</em>
-<a class="jxr_linenumber" name="L392" href="#L392">392</a> <em class="jxr_comment"> * met.getMetricName() ); sink.tableCell_(); sink.tableCell(); sink.text( String.valueOf( met.getCount() ) );</em>
-<a class="jxr_linenumber" name="L393" href="#L393">393</a> <em class="jxr_comment"> * sink.tableCell_(); sink.tableCell(); sink.text( String.valueOf( met.getHighValue() ) ); sink.tableCell_();</em>
-<a class="jxr_linenumber" name="L394" href="#L394">394</a> <em class="jxr_comment"> * sink.tableCell(); sink.text( String.valueOf( met.getLowValue() ) ); sink.tableCell_(); sink.tableCell();</em>
-<a class="jxr_linenumber" name="L395" href="#L395">395</a> <em class="jxr_comment"> * sink.text( String.valueOf( met.getAverage() ) ); sink.tableCell_(); sink.tableRow_(); } sink.table_();</em>
-<a class="jxr_linenumber" name="L396" href="#L396">396</a> <em class="jxr_comment"> * sink.section1_(); }</em>
-<a class="jxr_linenumber" name="L397" href="#L397">397</a> <em class="jxr_comment"> */</em>
-<a class="jxr_linenumber" name="L398" href="#L398">398</a>
-<a class="jxr_linenumber" name="L399" href="#L399">399</a> <strong class="jxr_keyword">public</strong> <strong class="jxr_keyword">void</strong> render()
-<a class="jxr_linenumber" name="L400" href="#L400">400</a> <strong class="jxr_keyword">throws</strong> IOException
-<a class="jxr_linenumber" name="L401" href="#L401">401</a> {
-<a class="jxr_linenumber" name="L402" href="#L402">402</a> processViolations();
-<a class="jxr_linenumber" name="L403" href="#L403">403</a>
-<a class="jxr_linenumber" name="L404" href="#L404">404</a> <strong class="jxr_keyword">if</strong> ( !processingErrors.isEmpty() )
-<a class="jxr_linenumber" name="L405" href="#L405">405</a> {
-<a class="jxr_linenumber" name="L406" href="#L406">406</a> processProcessingErrors();
-<a class="jxr_linenumber" name="L407" href="#L407">407</a> }
-<a class="jxr_linenumber" name="L408" href="#L408">408</a> }
-<a class="jxr_linenumber" name="L409" href="#L409">409</a>
-<a class="jxr_linenumber" name="L410" href="#L410">410</a> <strong class="jxr_keyword">public</strong> <strong class="jxr_keyword">void</strong> endDocument()
-<a class="jxr_linenumber" name="L411" href="#L411">411</a> <strong class="jxr_keyword">throws</strong> IOException
-<a class="jxr_linenumber" name="L412" href="#L412">412</a> {
-<a class="jxr_linenumber" name="L413" href="#L413">413</a> <em class="jxr_comment">// The Metrics report useless with the current PMD metrics impl.</em>
-<a class="jxr_linenumber" name="L414" href="#L414">414</a> <em class="jxr_comment">// For instance, run the coupling ruleset and you will get a boatload</em>
-<a class="jxr_linenumber" name="L415" href="#L415">415</a> <em class="jxr_comment">// of excessive imports metrics, none of which is really any use.</em>
-<a class="jxr_linenumber" name="L416" href="#L416">416</a> <em class="jxr_comment">// TODO Determine if we are going to just ignore metrics.</em>
-<a class="jxr_linenumber" name="L417" href="#L417">417</a>
-<a class="jxr_linenumber" name="L418" href="#L418">418</a> <em class="jxr_comment">// processMetrics();</em>
-<a class="jxr_linenumber" name="L419" href="#L419">419</a>
-<a class="jxr_linenumber" name="L420" href="#L420">420</a> sink.body_();
+<a class="jxr_linenumber" name="L365" href="#L365">365</a> <strong class="jxr_keyword">private</strong> <strong class="jxr_keyword">void</strong> processProcessingErrors() <strong class="jxr_keyword">throws</strong> IOException
+<a class="jxr_linenumber" name="L366" href="#L366">366</a> {
+<a class="jxr_linenumber" name="L367" href="#L367">367</a> <em class="jxr_comment">// sort the problem by filename first, since PMD is executed multi-threaded</em>
+<a class="jxr_linenumber" name="L368" href="#L368">368</a> <em class="jxr_comment">// and might reports the results unsorted</em>
+<a class="jxr_linenumber" name="L369" href="#L369">369</a> Collections.sort( processingErrors, <strong class="jxr_keyword">new</strong> Comparator<ProcessingError>()
+<a class="jxr_linenumber" name="L370" href="#L370">370</a> {
+<a class="jxr_linenumber" name="L371" href="#L371">371</a> @Override
+<a class="jxr_linenumber" name="L372" href="#L372">372</a> <strong class="jxr_keyword">public</strong> <strong class="jxr_keyword">int</strong> compare( ProcessingError e1, ProcessingError e2 )
+<a class="jxr_linenumber" name="L373" href="#L373">373</a> {
+<a class="jxr_linenumber" name="L374" href="#L374">374</a> <strong class="jxr_keyword">return</strong> e1.getFile().compareTo( e2.getFile() );
+<a class="jxr_linenumber" name="L375" href="#L375">375</a> }
+<a class="jxr_linenumber" name="L376" href="#L376">376</a> } );
+<a class="jxr_linenumber" name="L377" href="#L377">377</a>
+<a class="jxr_linenumber" name="L378" href="#L378">378</a> sink.section1();
+<a class="jxr_linenumber" name="L379" href="#L379">379</a> sink.sectionTitle1();
+<a class="jxr_linenumber" name="L380" href="#L380">380</a> sink.text( bundle.getString( <span class="jxr_string">"report.pmd.processingErrors.title"</span> ) );
+<a class="jxr_linenumber" name="L381" href="#L381">381</a> sink.sectionTitle1_();
+<a class="jxr_linenumber" name="L382" href="#L382">382</a>
+<a class="jxr_linenumber" name="L383" href="#L383">383</a> sink.table();
+<a class="jxr_linenumber" name="L384" href="#L384">384</a> sink.tableRow();
+<a class="jxr_linenumber" name="L385" href="#L385">385</a> sink.tableHeaderCell();
+<a class="jxr_linenumber" name="L386" href="#L386">386</a> sink.text( bundle.getString( <span class="jxr_string">"report.pmd.processingErrors.column.filename"</span> ) );
+<a class="jxr_linenumber" name="L387" href="#L387">387</a> sink.tableHeaderCell_();
+<a class="jxr_linenumber" name="L388" href="#L388">388</a> sink.tableHeaderCell();
+<a class="jxr_linenumber" name="L389" href="#L389">389</a> sink.text( bundle.getString( <span class="jxr_string">"report.pmd.processingErrors.column.problem"</span> ) );
+<a class="jxr_linenumber" name="L390" href="#L390">390</a> sink.tableHeaderCell_();
+<a class="jxr_linenumber" name="L391" href="#L391">391</a> sink.tableRow_();
+<a class="jxr_linenumber" name="L392" href="#L392">392</a>
+<a class="jxr_linenumber" name="L393" href="#L393">393</a> <strong class="jxr_keyword">for</strong> ( ProcessingError error : processingErrors )
+<a class="jxr_linenumber" name="L394" href="#L394">394</a> {
+<a class="jxr_linenumber" name="L395" href="#L395">395</a> processSingleProcessingError( error );
+<a class="jxr_linenumber" name="L396" href="#L396">396</a> }
+<a class="jxr_linenumber" name="L397" href="#L397">397</a>
+<a class="jxr_linenumber" name="L398" href="#L398">398</a> sink.table_();
+<a class="jxr_linenumber" name="L399" href="#L399">399</a>
+<a class="jxr_linenumber" name="L400" href="#L400">400</a> sink.section1_();
+<a class="jxr_linenumber" name="L401" href="#L401">401</a> }
+<a class="jxr_linenumber" name="L402" href="#L402">402</a>
+<a class="jxr_linenumber" name="L403" href="#L403">403</a> <strong class="jxr_keyword">private</strong> <strong class="jxr_keyword">void</strong> processSingleProcessingError( ProcessingError error ) <strong class="jxr_keyword">throws</strong> IOException
+<a class="jxr_linenumber" name="L404" href="#L404">404</a> {
+<a class="jxr_linenumber" name="L405" href="#L405">405</a> String filename = error.getFile();
+<a class="jxr_linenumber" name="L406" href="#L406">406</a> <a href="../../../../../org/apache/maven/plugins/pmd/PmdFileInfo.html">PmdFileInfo</a> fileInfo = determineFileInfo( filename );
+<a class="jxr_linenumber" name="L407" href="#L407">407</a> filename = makeFileSectionName( shortenFilename( filename, fileInfo ), fileInfo );
+<a class="jxr_linenumber" name="L408" href="#L408">408</a>
+<a class="jxr_linenumber" name="L409" href="#L409">409</a> sink.tableRow();
+<a class="jxr_linenumber" name="L410" href="#L410">410</a> sink.tableCell();
+<a class="jxr_linenumber" name="L411" href="#L411">411</a> sink.text( filename );
+<a class="jxr_linenumber" name="L412" href="#L412">412</a> sink.tableCell_();
+<a class="jxr_linenumber" name="L413" href="#L413">413</a> sink.tableCell();
+<a class="jxr_linenumber" name="L414" href="#L414">414</a> sink.text( error.getMsg() );
+<a class="jxr_linenumber" name="L415" href="#L415">415</a> sink.verbatim( <strong class="jxr_keyword">null</strong> );
+<a class="jxr_linenumber" name="L416" href="#L416">416</a> sink.rawText( error.getDetail() );
+<a class="jxr_linenumber" name="L417" href="#L417">417</a> sink.verbatim_();
+<a class="jxr_linenumber" name="L418" href="#L418">418</a> sink.tableCell_();
+<a class="jxr_linenumber" name="L419" href="#L419">419</a> sink.tableRow_();
+<a class="jxr_linenumber" name="L420" href="#L420">420</a> }
<a class="jxr_linenumber" name="L421" href="#L421">421</a>
-<a class="jxr_linenumber" name="L422" href="#L422">422</a> sink.flush();
-<a class="jxr_linenumber" name="L423" href="#L423">423</a>
-<a class="jxr_linenumber" name="L424" href="#L424">424</a> sink.close();
-<a class="jxr_linenumber" name="L425" href="#L425">425</a> }
-<a class="jxr_linenumber" name="L426" href="#L426">426</a>
-<a class="jxr_linenumber" name="L427" href="#L427">427</a> <strong class="jxr_keyword">public</strong> <strong class="jxr_keyword">void</strong> setFiles( Map<File, PmdFileInfo> files )
-<a class="jxr_linenumber" name="L428" href="#L428">428</a> {
-<a class="jxr_linenumber" name="L429" href="#L429">429</a> <strong class="jxr_keyword">this</strong>.files = files;
-<a class="jxr_linenumber" name="L430" href="#L430">430</a> }
+<a class="jxr_linenumber" name="L422" href="#L422">422</a> <strong class="jxr_keyword">public</strong> <strong class="jxr_keyword">void</strong> beginDocument()
+<a class="jxr_linenumber" name="L423" href="#L423">423</a> {
+<a class="jxr_linenumber" name="L424" href="#L424">424</a> sink.head();
+<a class="jxr_linenumber" name="L425" href="#L425">425</a> sink.title();
+<a class="jxr_linenumber" name="L426" href="#L426">426</a> sink.text( getTitle() );
+<a class="jxr_linenumber" name="L427" href="#L427">427</a> sink.title_();
+<a class="jxr_linenumber" name="L428" href="#L428">428</a> sink.head_();
+<a class="jxr_linenumber" name="L429" href="#L429">429</a>
+<a class="jxr_linenumber" name="L430" href="#L430">430</a> sink.body();
<a class="jxr_linenumber" name="L431" href="#L431">431</a>
-<a class="jxr_linenumber" name="L432" href="#L432">432</a> <strong class="jxr_keyword">public</strong> <strong class="jxr_keyword">void</strong> setRenderRuleViolationPriority( <strong class="jxr_keyword">boolean</strong> renderRuleViolationPriority )
-<a class="jxr_linenumber" name="L433" href="#L433">433</a> {
-<a class="jxr_linenumber" name="L434" href="#L434">434</a> <strong class="jxr_keyword">this</strong>.renderRuleViolationPriority = renderRuleViolationPriority;
-<a class="jxr_linenumber" name="L435" href="#L435">435</a> }
-<a class="jxr_linenumber" name="L436" href="#L436">436</a> }
+<a class="jxr_linenumber" name="L432" href="#L432">432</a> sink.section1();
+<a class="jxr_linenumber" name="L433" href="#L433">433</a> sink.sectionTitle1();
+<a class="jxr_linenumber" name="L434" href="#L434">434</a> sink.text( getTitle() );
+<a class="jxr_linenumber" name="L435" href="#L435">435</a> sink.sectionTitle1_();
+<a class="jxr_linenumber" name="L436" href="#L436">436</a>
+<a class="jxr_linenumber" name="L437" href="#L437">437</a> sink.paragraph();
+<a class="jxr_linenumber" name="L438" href="#L438">438</a> sink.text( bundle.getString( <span class="jxr_string">"report.pmd.pmdlink"</span> ) + <span class="jxr_string">" "</span> );
+<a class="jxr_linenumber" name="L439" href="#L439">439</a> sink.link( <span class="jxr_string">"https://pmd.github.io"</span> );
+<a class="jxr_linenumber" name="L440" href="#L440">440</a> sink.text( <span class="jxr_string">"PMD"</span> );
+<a class="jxr_linenumber" name="L441" href="#L441">441</a> sink.link_();
[... 84 lines stripped ...]