Hi Chris,
found just one duplication:
<Match>
<Class name="org.apache.fop.afp.AFPPaintingState"/>
<Field name="colorConverter"/>
<Bug pattern="SE_BAD_FIELD_STORE"/>
</Match>
using following template:
<xsl:template match="Match">
<xsl:if test="preceding-sibling::Match[
Class/@name = current()/Class/@name
and Field/@name = current()/Field/@name
and Bug/@pattern = current()/Bug/@pattern]">
<xsl:copy-of select="."/>
</xsl:if>
</xsl:template>
HTH,
Le 18/01/2012 18:07, Chris Bowditch a écrit :
> On 18/01/2012 15:51, Glenn Adams wrote:
>> I recall that someone (Jeremias or Simon?) had mechanically generated
>> some exclusions to add to this file. It is possible that duplicates
>> got inserted in that process. I know of no reason to retain duplicates.
>
> Thanks Glenn. I've removed the duplicates that I noticed initially, but
> I can see there are some more. Due to the size of the file, it will
> probably require some automated process to remove all the duplicates.
>
> Thanks,
>
> Chris
>
>>
>> On Wed, Jan 18, 2012 at 6:39 AM, Chris Bowditch
>> <[email protected] <mailto:[email protected]>> wrote:
>>
>> Hi Fellow Committers,
>>
>> I was reviewing the current set of warnings generated by FindBugs
>> and found myself needing to add an extra exclusion. I noticed that
>> the exclusions file contains a lot of duplicates, e.g.
>>
>> <Match>
>> <Class name="org.apache.fop.render.afp.AFPImageHandlerRawCCITTFax"/>
>> <Method name="setAdditionalParameters"/>
>> <Bug pattern="BC_UNCONFIRMED_CAST"/>
>> </Match>
>> <Match>
>> <Class name="org.apache.fop.render.afp.AFPImageHandlerRawCCITTFax"/>
>> <Method name="setAdditionalParameters"/>
>> <Bug pattern="BC_UNCONFIRMED_CAST"/>
>> </Match>
>>
>> Does anyone know why the duplicates exist? I confirmed that
>> removing the duplicates does not re-introduce any warnings with
>> FindBugs 1.3.9.
>>
>> Thanks,
>>
>> Chris
>>
>>
>
--
Pascal