cxf-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Freeman Fang <>
Subject Re: More odd PMD-related build failures
Date Fri, 27 Oct 2006 02:03:05 GMT
Hi Eoghan,

I also saw these problem on my machine. After mvn -U to update maven pmd 
plugin, these pmd error will not cause build failure but only throw 

Thanks very much


Glynn, Eoghan wrote:

>Some questionable PMD violations have popped up in my builds after
>updating to the commits that have gone in over the last few days. Not
>sure why this doesn't seem to be impacting on the cruisecontrol builds
>(maybe its running with -Pnochecks).
>The first[1] seems to be a kosher warning, but the detail (line number
>etc.) is lost, seemingly because the method is annotated with
>@SuppressWarnings. This is broken ... PMD shouldn't be effected by a
>@SupressWarnings annotation, but even it is, it should be fully
>supressed not just stripped of any useful detail but still causing the
>build to fail. In this particular class, there are only a few nested
>ifs, so its clear where the bad code is, but still annoying.
>The second[2] is spurious and seems to be caused by
>AbstractEndpointFactory having both public and private methods called
>The third[3] is another example of PMD's maddening stupidity when a
>method name is overloaded (IriDecoderHelper.getElement()).
>I'm also seeing PMD just completely dying on me when building
>rt/ws/addr, arghh!
>All in all I like PMD less and less, but as long as we're stuck with it,
>PLEASE be sure to run a full PMD check before committing.
>[1] rt/frontend/jaxws/target/pmd.xml:
><?xml version="1.0" encoding="UTF-8"?>
><pmd version="3.6" timestamp="2006-10-26T15:29:03.979" elapsedTime="0s">
>suppressiontype="annotation" msg="These nested if statements could be
>[2] rt/frontend/simple/target/pmd.xml:
><?xml version="1.0" encoding="UTF-8"?>
><pmd version="3.6" timestamp="2006-10-26T15:51:15.576" elapsedTime="0s">
><violation line="81" rule="UnusedPrivateMethod" ruleset="Unused Code
>Rules" package="org.apache.cxf.service.factory"
>class="AbstractEndpointFactory" method="setProperties"
>PrivateMethod" priority="3">
>Avoid unused private methods such as 'setProperties(EndpointInfo)'
>[3] rt/bindings/http/target/pmd.xml:
><?xml version="1.0" encoding="UTF-8"?>
><pmd version="3.6" timestamp="2006-10-26T16:15:12.435" elapsedTime="0s">
><violation line="247" rule="UnusedPrivateMethod" ruleset="Unused Code
>Rules" package="org.apache.cxf.binding.http" class="IriDecoderHelper"
>PrivateMethod" priority="3">
>Avoid unused private methods such as 'getElement(Element,int)'

Freeman Fang
Software Engineer

IONA Asia Pacific Software Development Center
No.2 Floor A Unit Information Center
Zhongguancun Software Park Haidian District,
Beijing, P.R.China

Tel.: +86-10-82825151 -  ex. 551
Fax: +86-10-8282-5210
Making Software Work Together TM

View raw message