From c33913188311d78421665b9a031a8be1889723d3 Mon Sep 17 00:00:00 2001 From: Herve Boutemy Date: Mon, 28 Apr 2014 21:42:54 +0000 Subject: [PATCH] removed too verbose debug log items git-svn-id: https://svn.apache.org/repos/asf/maven/plugin-tools/trunk@1590814 13f79535-47bb-0310-9956-ffa450edef68 --- .../scanner/DefaultMojoAnnotationsScanner.java | 16 +++++++++++----- .../scanner/visitors/MojoAnnotationVisitor.java | 6 +----- .../scanner/visitors/MojoClassVisitor.java | 2 +- .../scanner/visitors/MojoFieldVisitor.java | 3 +-- 4 files changed, 14 insertions(+), 13 deletions(-) diff --git a/maven-plugin-tools-annotations/src/main/java/org/apache/maven/tools/plugin/annotations/scanner/DefaultMojoAnnotationsScanner.java b/maven-plugin-tools-annotations/src/main/java/org/apache/maven/tools/plugin/annotations/scanner/DefaultMojoAnnotationsScanner.java index aa225d0..8e3ea42 100644 --- a/maven-plugin-tools-annotations/src/main/java/org/apache/maven/tools/plugin/annotations/scanner/DefaultMojoAnnotationsScanner.java +++ b/maven-plugin-tools-annotations/src/main/java/org/apache/maven/tools/plugin/annotations/scanner/DefaultMojoAnnotationsScanner.java @@ -145,9 +145,12 @@ public class DefaultMojoAnnotationsScanner } if ( isStoreClass( mojoClassVisitor.getMojoAnnotatedClass() ) != null ) { - getLogger().debug( - "found MojoAnnotatedClass:" + mojoClassVisitor.getMojoAnnotatedClass().getClassName() + ":" - + mojoClassVisitor.getMojoAnnotatedClass() ); + if ( getLogger().isDebugEnabled() ) + { + getLogger().debug( "found MojoAnnotatedClass:" + + mojoClassVisitor.getMojoAnnotatedClass().getClassName() + ":" + + mojoClassVisitor.getMojoAnnotatedClass() ); + } mojoClassVisitor.getMojoAnnotatedClass().setArtifact( artifact ); mojoAnnotatedClasses.put( mojoClassVisitor.getMojoAnnotatedClass().getClassName(), mojoClassVisitor.getMojoAnnotatedClass() ); @@ -215,8 +218,11 @@ public class DefaultMojoAnnotationsScanner if ( isStoreClass( mojoAnnotatedClass ) != null ) { - getLogger().debug( "found MojoAnnotatedClass:" + mojoAnnotatedClass.getClassName() + ":" - + mojoAnnotatedClass ); + if ( getLogger().isDebugEnabled() ) + { + getLogger().debug( "found MojoAnnotatedClass:" + mojoAnnotatedClass.getClassName() + ":" + + mojoAnnotatedClass ); + } mojoAnnotatedClass.setArtifact( artifact ); mojoAnnotatedClasses.put( mojoAnnotatedClass.getClassName(), mojoAnnotatedClass ); } diff --git a/maven-plugin-tools-annotations/src/main/java/org/apache/maven/tools/plugin/annotations/scanner/visitors/MojoAnnotationVisitor.java b/maven-plugin-tools-annotations/src/main/java/org/apache/maven/tools/plugin/annotations/scanner/visitors/MojoAnnotationVisitor.java index a44c541..2993553 100644 --- a/maven-plugin-tools-annotations/src/main/java/org/apache/maven/tools/plugin/annotations/scanner/visitors/MojoAnnotationVisitor.java +++ b/maven-plugin-tools-annotations/src/main/java/org/apache/maven/tools/plugin/annotations/scanner/visitors/MojoAnnotationVisitor.java @@ -41,7 +41,7 @@ public class MojoAnnotationVisitor MojoAnnotationVisitor( Logger logger, String annotationClassName ) { - super(Opcodes.ASM4); + super( Opcodes.ASM4 ); this.logger = logger; this.annotationClassName = annotationClassName; } @@ -54,24 +54,20 @@ public class MojoAnnotationVisitor public void visit( String name, Object value ) { annotationValues.put( name, value ); - logger.debug( "MojoAnnotationVisitor#visit:" + name + ":" + value ); } public void visitEnum( String name, String desc, String value ) { annotationValues.put( name, value ); - logger.debug( "MojoAnnotationVisitor#visitEnum:" + name + ":" + desc + ":" + value ); } public AnnotationVisitor visitAnnotation( String name, String desc ) { - logger.debug( "MojoAnnotationVisitor#visitAnnotation:" + name + ":" + desc ); return new MojoAnnotationVisitor( logger, this.annotationClassName ); } public AnnotationVisitor visitArray( String s ) { - logger.debug( "MojoAnnotationVisitor#visitArray" ); return new MojoAnnotationVisitor( logger, this.annotationClassName ); } diff --git a/maven-plugin-tools-annotations/src/main/java/org/apache/maven/tools/plugin/annotations/scanner/visitors/MojoClassVisitor.java b/maven-plugin-tools-annotations/src/main/java/org/apache/maven/tools/plugin/annotations/scanner/visitors/MojoClassVisitor.java index 449093d..427035e 100644 --- a/maven-plugin-tools-annotations/src/main/java/org/apache/maven/tools/plugin/annotations/scanner/visitors/MojoClassVisitor.java +++ b/maven-plugin-tools-annotations/src/main/java/org/apache/maven/tools/plugin/annotations/scanner/visitors/MojoClassVisitor.java @@ -53,7 +53,7 @@ public class MojoClassVisitor public MojoClassVisitor( Logger logger ) { - super(Opcodes.ASM4); + super( Opcodes.ASM4 ); this.logger = logger; } diff --git a/maven-plugin-tools-annotations/src/main/java/org/apache/maven/tools/plugin/annotations/scanner/visitors/MojoFieldVisitor.java b/maven-plugin-tools-annotations/src/main/java/org/apache/maven/tools/plugin/annotations/scanner/visitors/MojoFieldVisitor.java index 73e713b..4a61a01 100644 --- a/maven-plugin-tools-annotations/src/main/java/org/apache/maven/tools/plugin/annotations/scanner/visitors/MojoFieldVisitor.java +++ b/maven-plugin-tools-annotations/src/main/java/org/apache/maven/tools/plugin/annotations/scanner/visitors/MojoFieldVisitor.java @@ -44,7 +44,7 @@ public class MojoFieldVisitor MojoFieldVisitor( Logger logger, String fieldName, String className ) { - super(Opcodes.ASM4); + super( Opcodes.ASM4 ); this.logger = logger; this.fieldName = fieldName; this.className = className; @@ -62,7 +62,6 @@ public class MojoFieldVisitor public AnnotationVisitor visitAnnotation( String desc, boolean visible ) { - logger.debug( "MojoFieldVisitor#visitAnnotation:" + desc ); String annotationClassName = Type.getType( desc ).getClassName(); if ( !MojoAnnotationsScanner.FIELD_LEVEL_ANNOTATIONS.contains( annotationClassName ) ) {