Re: Optional Reporter

Yes I believe I do need to add a reporter.  
I am currently working on the Maven 2 Findbugs plugin.  We currently have a reporter that we add to give other functionality.  I guess you could in some cases use a template or XSL sheet but our report do query Findbugs for information which is not in the reports currently.   This would allow extensibility for reporters like we do for plugins.  The change would be small.  I could add the code and send patch file to implement this feature.   I have hesitated because it seems like the code is in a heavy change state.

I am looking at the newer codebase (1.1.1) and it looks as though the best way invoke Findbugs from within another java application would be as follows:


            String[] options;
 
            TextUICommandLine commandLine = new TextUICommandLine();
            FindBugs findBugs = createEngine(commandLine, options);

            "set up any other findBugs parameter"

            try {
                findBugs.execute( );
            } catch (Exception e) {
                ... exception handling
            }

This would seem to be the best way with the least amount of potential imapct from code refactoring.  I am getting hit with it now in the Maven plugin because of refactoring.

Am I missing any other way to invoke FindBugs?



-- 
Regards,


Garvin LeClaire
garvin.leclaire@xxxxxxxxx

On 10/3/06, Brian Cole <tuc@xxxxxxxxxx> wrote:

howdy,

Do you actually need the functionality of a Reporter, or can you do
what you
want with XSLT? We have been using XSLT to customize output like this:

     findbugs -textui -html:/path/to/your/stylesheet.xsl ...

I would like to hear more about what you're trying to do.

-brian

On Oct 3, 2006, at 7:17 AM, Garvin LeClaire wrote:
> It would be nice to have the ability to add a reporter.
> This could be done by a commandline option where the class is
> stated for a class on the class path.
>
> i.e.
>
> findbugs -textui -reporter com.acme.findbug.PrettyReporter
>
> --
> Regards,
>
> Garvin LeClaire
> garvin.leclaire@xxxxxxxxx