[incubator-hop] branch master updated: Add a nested comment explaining why this method is empty, throw an UnsupportedOperationException or complete the implementation.

[hansva]

This is an automated email from the ASF dual-hosted git repository.

hansva pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-hop.git


The following commit(s) were added to refs/heads/master by this push:
     new 30f7610  Add a nested comment explaining why this method is empty, 
throw an UnsupportedOperationException or complete the implementation.
     new 35ca900  Merge pull request #654 from 
nanthakumar1305/sonarqube_20210301
30f7610 is described below

commit 30f761011efeee0fe12f91179e18ca658b7ea932
Author: nanthakumar1305 <nanthakumar1...@gmail.com>
AuthorDate: Mon Mar 1 06:25:33 2021 +0000

    Add a nested comment explaining why this method is empty, throw an 
UnsupportedOperationException or complete the implementation.
---
 .../main/java/org/apache/hop/core/ProgressNullMonitorListener.java   | 5 +++++
 1 file changed, 5 insertions(+)

diff --git 
a/core/src/main/java/org/apache/hop/core/ProgressNullMonitorListener.java 
b/core/src/main/java/org/apache/hop/core/ProgressNullMonitorListener.java
index c30b418..a4c32ce 100644
--- a/core/src/main/java/org/apache/hop/core/ProgressNullMonitorListener.java
+++ b/core/src/main/java/org/apache/hop/core/ProgressNullMonitorListener.java
@@ -22,10 +22,12 @@ public class ProgressNullMonitorListener implements 
IProgressMonitor {
 
   @Override
   public void beginTask( String message, int nrWorks ) {
+      // Do nothing because of beginTask( "", 0 ).
   }
 
   @Override
   public void subTask( String message ) {
+      // Do nothing because of subTask( "" ).
   }
 
   @Override
@@ -35,13 +37,16 @@ public class ProgressNullMonitorListener implements 
IProgressMonitor {
 
   @Override
   public void worked( int nrWorks ) {
+      // Do nothing because of worked( 0 ).
   }
 
   @Override
   public void done() {
+      // Do nothing because of done().
   }
 
   @Override
   public void setTaskName( String taskName ) {
+      // Do nothing because of setTaskName( "" ).
   }
 }