McFoggy commented on a change in pull request #438:
URL: https://github.com/apache/maven/pull/438#discussion_r564502239
##########
File path:
maven-plugin-api/src/main/java/org/apache/maven/plugin/logging/Log.java
##########
@@ -63,6 +65,20 @@
*/
void debug( Throwable error );
+ /**
+ * Send a message to the user in the <b>debug</b> error level by computing
the message
+ * only when needed. The supplier will be called only if @see
#isDebugEnabled() is <b>true</b>.
+ *
+ * @param messageSupplier a non null Supplier of the message to use
+ */
+ default void debug( Supplier<String> messageSupplier )
Review comment:
I used the default methods not to break any other potential
implementation.
If it is preferred not to do so then I can go with a breaking change and
introduction of an Abstract class between the implementations.
Let's wait a bit for others POV.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to 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
