[
https://issues.apache.org/jira/browse/DOXIA-534?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16265843#comment-16265843
]
ASF GitHub Bot commented on DOXIA-534:
--------------------------------------
slachiewicz commented on a change in pull request #5: [DOXIA-534] Migrate
logging to Sl4j
URL: https://github.com/apache/maven-doxia/pull/5#discussion_r153053842
##########
File path:
doxia-modules/doxia-module-fo/src/main/java/org/apache/maven/doxia/module/fo/FoAggregateSink.java
##########
@@ -1322,12 +1322,12 @@ private SinkEventAttributeSet getGraphicsAttributes(
String logo )
}
catch ( IOException e )
{
- getLog().debug( e );
+ getLog().debug( e.getMessage() );
Review comment:
there is no constructor with Exception, i will change to e.getMessage(), e
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
> Rethink the need of Doxia Logging API
> -------------------------------------
>
> Key: DOXIA-534
> URL: https://issues.apache.org/jira/browse/DOXIA-534
> Project: Maven Doxia
> Issue Type: Task
> Components: Logging API
> Affects Versions: 1.6
> Reporter: Michael Osipov
>
> We should rethink whether we still need the Doxia Logging API and can replace
> it with standard tools like Plexus-injected logging or SLF4J.
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
