From: markt <markt@13f79535-47bb-0310-9956-ffa450edef68>
Date: Fri, 14 Jan 2011 16:48:44 +0000 (+0000)
Subject: Add comments explaining false +ve
X-Git-Url: https://git.internetallee.de/?a=commitdiff_plain;h=a44f3b8603ebdc9fdac418c237e62b4269bfc28f;p=tomcat7.0

Add comments explaining false +ve

git-svn-id: https://svn.apache.org/repos/asf/tomcat/trunk@1059069 13f79535-47bb-0310-9956-ffa450edef68
---

diff --git a/res/findbugs/filter-false-positives.xml b/res/findbugs/filter-false-positives.xml
index fdc042f79..853248119 100644
--- a/res/findbugs/filter-false-positives.xml
+++ b/res/findbugs/filter-false-positives.xml
@@ -17,16 +17,19 @@
 <FindBugsFilter>
   <!--  Considered to be false positives -->
   <Match>
+    <!-- Have to trigger GC for leak detection to work. Clearly documented -->
     <Class name="org.apache.catalina.core.StandardHost" />
     <Method name="findReloadedContextMemoryLeaks" />
     <Bug code="Dm" />
   </Match>
   <Match>
+    <!-- Sleep is short, needs to keep lock -->
     <Class name="org.apache.catalina.startup.HostConfig" />
     <Method name="checkResources" />
     <Bug code="SWL" />
   </Match>
   <Match>
+    <!-- Class name needs to start with a lower case letter in this case -->
     <Class name="org.apache.naming.java.javaURLContextFactory" />
     <Bug code="Nm" />
   </Match>