java: remove unnecessary return statements

found by PMD

Change-Id: I3d150421948d17eaed34fac2b1a212da34288d1a

android/experimental/LibreOffice4Android/src/org/libreoffice/android/DocumentLoader.java

@@ -956,7 +956,6 @@ public class DocumentLoader
                                                          Animation.RELATIVE_TO_SELF, 0, Animation.RELATIVE_TO_SELF, 0);
             outToLeft.setDuration(500);
             viewFlipper.setOutAnimation(outToLeft);
-            return;
         }
 
         private void setAnimationInFromLeft(){
@@ -969,7 +968,6 @@ public class DocumentLoader
                                                           Animation.RELATIVE_TO_SELF, 0, Animation.RELATIVE_TO_SELF, 0);
             outToRight.setDuration(500);
             viewFlipper.setOutAnimation(outToRight);
-            return;
         }
 
         private PageViewer fetch( int page ){

android/experimental/LibreOffice4Android/src/org/libreoffice/ui/LibreOfficeUIActivity.java

@@ -387,7 +387,6 @@ public class LibreOfficeUIActivity extends SherlockActivity implements ActionBar
 				break;
 		}
 		this.onResume();
-		return;
     }
     
     public void readPreferences(){

comphelper/qa/complex/comphelper/SequenceOutputStreamUnitTest.java

@@ -132,7 +132,7 @@ public class SequenceOutputStreamUnitTest /* extends ComplexTestCase*/ {
             m_aTestHelper.Error ( "Exception: " + e );
             return /* false */;
         }
-        return /* true */;
+        // return /* true */;
     }
 
     private static XMultiServiceFactory getMSF()

odk/examples/java/EmbedDocument/EmbeddedObject/OwnEmbeddedObject.java

@@ -362,7 +362,6 @@ public final class OwnEmbeddedObject extends WeakBase
     public void reload(com.sun.star.beans.PropertyValue[] aMediaArgs, com.sun.star.beans.PropertyValue[] aObjectArgs) throws com.sun.star.lang.IllegalArgumentException, com.sun.star.embed.WrongStateException, com.sun.star.io.IOException, com.sun.star.uno.Exception
     {
         // not implemented currently
-        return;
     }
 
     // com.sun.star.embed.XEmbedPersist:

qadevOOo/runner/util/utils.java

@@ -211,7 +211,6 @@ public class utils {
             //create the directory
             new_dir.mkdirs();
         }
-        return;
     }
 
 

qadevOOo/tests/java/ifc/beans/_XFastPropertySet.java

@@ -150,7 +150,6 @@ public class _XFastPropertySet extends MultiMethodTest {
             e.printStackTrace(log);
             tRes.tested("getFastPropertyValue()",false);
         }
-        return;
     }
 
 

qadevOOo/tests/java/ifc/beans/_XMultiPropertySet.java

@@ -334,8 +334,6 @@ public class _XMultiPropertySet extends MultiMethodTest {
         testPropsNames = new String[nr];
         for (int i=0; i<nr; i++) testPropsNames[i] = ST.nextToken();
         testPropsAmount = nr;
-        return;
-
     }
 
     /**

qadevOOo/tests/java/ifc/beans/_XPropertyAccess.java

@@ -89,7 +89,6 @@ public class _XPropertyAccess extends MultiMethodTest {
         }
 
         tRes.tested("getPropertyValues()", ok );
-        return;
     }
 
     /**
@@ -219,8 +218,6 @@ public class _XPropertyAccess extends MultiMethodTest {
         }
 
         tRes.tested("setPropertyValues()", ok);
-        return;
-
     }
 
 }    /// finish class XPropertyAccess

qadevOOo/tests/java/ifc/beans/_XPropertyContainer.java

@@ -211,7 +211,6 @@ public class _XPropertyContainer extends MultiMethodTest {
         ok &= test;
 
         tRes.tested("addProperty()", ok );
-        return;
     }
 
     /**
@@ -294,7 +293,6 @@ public class _XPropertyContainer extends MultiMethodTest {
         }
 
         tRes.tested("removeProperty()", ok );
-        return;
     }
 
 

qadevOOo/tests/java/ifc/beans/_XPropertySet.java

@@ -133,9 +133,6 @@ public class _XPropertySet extends MultiMethodTest {
             tRes.tested("getPropertySetInfo()", true );
             getPropsToTest(propertySetInfo);
         }
-
-        return;
-
     } // end of getPropertySetInfo()
 
     /**
@@ -194,9 +191,6 @@ public class _XPropertySet extends MultiMethodTest {
             }
             tRes.tested("addPropertyChangeListener()", !error);
         }
-
-        return;
-
     } // end of addPropertyChangeListener()
 
     /**
@@ -254,9 +248,6 @@ public class _XPropertySet extends MultiMethodTest {
             }
             tRes.tested("addVetoableChangeListener()",!error);
         }
-
-        return;
-
     } // end of addVetoableChangeListener()
 
 
@@ -322,9 +313,6 @@ public class _XPropertySet extends MultiMethodTest {
             }
             tRes.tested("setPropertyValue()",!error);
         } //endif
-
-        return;
-
     } // end of setPropertyValue()
 
     /**
@@ -390,8 +378,6 @@ public class _XPropertySet extends MultiMethodTest {
             }
             tRes.tested("getPropertyValue()", !error) ;
         }
-
-        return;
     }
 
     /**
@@ -464,9 +450,6 @@ public class _XPropertySet extends MultiMethodTest {
             }
             tRes.tested("removePropertyChangeListener()",!error);
         }
-
-        return;
-
     } // end of removePropertyChangeListener()
 
 
@@ -539,9 +522,6 @@ public class _XPropertySet extends MultiMethodTest {
             }
             tRes.tested("removeVetoableChangeListener()",!error);
         }
-
-        return;
-
     } // end of removeVetoableChangeListener()
 
     /**

qadevOOo/tests/java/ifc/beans/_XPropertySetInfo.java

@@ -49,7 +49,6 @@ public class _XPropertySetInfo extends MultiMethodTest {
         Property[] properties = oObj.getProperties();
         IsThere = properties[0];
         tRes.tested("getProperties()", ( properties != null ));
-        return;
     }
 
     /**
@@ -88,7 +87,6 @@ public class _XPropertySetInfo extends MultiMethodTest {
             result = true;
         }
         tRes.tested("getPropertyByName()", result);
-        return;
     }
 
     /**

qadevOOo/tests/java/ifc/container/_XEnumeration.java

@@ -89,7 +89,6 @@ public class _XEnumeration extends MultiMethodTest {
             result &= locResult;
         }
         tRes.tested("hasMoreElements()", result);
-        return;
     } // end hasMoreElements
 
     /**
@@ -119,9 +118,6 @@ public class _XEnumeration extends MultiMethodTest {
         }
 
         tRes.tested("nextElement()", result);
-
-        return;
-
     } // end NextElement
 
 } //end XEnumeration

qadevOOo/tests/java/ifc/container/_XEnumerationAccess.java

@@ -40,7 +40,6 @@ public class _XEnumerationAccess extends MultiMethodTest{
         log.println( "createing Enumeration" );
         XEnumeration oEnum = oObj.createEnumeration();
         tRes.tested( "createEnumeration()", oEnum != null );
-        return;
     } // end createEnumeration()
 }  // finish class _XEnumerationAccess
 

qadevOOo/tests/java/ifc/container/_XNameAccess.java

@@ -44,7 +44,6 @@ public class _XNameAccess extends MultiMethodTest {
 
         result = (Names != null);
         tRes.tested("getElementNames()", result);
-        return;
     } // end getElementNames()
 
     /**
@@ -86,8 +85,6 @@ public class _XNameAccess extends MultiMethodTest {
     result &= loc_result;
 
     tRes.tested("hasByName()", result);
-
-        return;
     } // end hasByName()
 
 
@@ -145,9 +142,6 @@ public class _XNameAccess extends MultiMethodTest {
 
         result &= loc_result;
         tRes.tested("getByName()", result);
-
-        return;
-
     } // end getByName()
 } /// finished class _XNameAccess
 

qadevOOo/tests/java/ifc/container/_XNameContainer.java

@@ -241,8 +241,6 @@ public class _XNameContainer extends MultiMethodTest {
         }
 
         tRes.tested("removeByName()", result);
-
-        return;
     } // end removeByName()
 } //XNameContainer
 

qadevOOo/tests/java/ifc/form/_XReset.java

@@ -162,9 +162,6 @@ public class _XReset extends MultiMethodTest {
          shortWait();
          tRes.tested("removeResetListener()", !approveReset[0] && !resetted[0]);
          //removing the second listener here may avoid crashing the office
-
-         return;
-
     } // finished _removeResetListener()
 
     /**

qadevOOo/tests/java/ifc/form/validation/_XValidatableFormComponent.java

@@ -207,8 +207,6 @@ public class _XValidatableFormComponent extends MultiMethodTest
             testPropsNames[i] = ST.nextToken();
 
         testPropsAmount = nr;
-
-        return;
     }
 
     protected class MyListener implements XFormComponentValidityListener

qadevOOo/tests/java/ifc/frame/_XComponentLoader.java

@@ -141,7 +141,6 @@ public class _XComponentLoader extends MultiMethodTest {
         }
 
         tRes.tested("loadComponentFromURL()", result);
-        return;
     }
 
 }

qadevOOo/tests/java/ifc/frame/_XDispatchProvider.java

@@ -149,7 +149,6 @@ public class _XDispatchProvider extends MultiMethodTest {
         }
 
         tRes.tested("queryDispatches()", true);
-        return;
     }
 
 }

qadevOOo/tests/java/ifc/lang/_XComponent.java

@@ -94,8 +94,6 @@ public class _XComponent extends MultiMethodTest {
 
         oObj.addEventListener( listener1 );
         oObj.addEventListener( listener2 );
-
-        return;
     } // finished _addEventListener()
 
     /**

qadevOOo/tests/java/ifc/util/_XReplaceDescriptor.java

@@ -58,7 +58,6 @@ public class _XReplaceDescriptor extends MultiMethodTest {
         oObj.setReplaceString("XReplaceDescriptor");
         String cmpReplaceStr = oObj.getReplaceString();
         tRes.tested("setReplaceString()", !(cmpReplaceStr.equals(oldReplaceStr)));
-        return;
     }
 
 }  // finish class _XReplaceDescriptor

scripting/examples/java/Highlight/HighlightText.java

@@ -118,8 +118,6 @@ public class HighlightText implements com.sun.star.awt.XActionListener {
         exitButton.addActionListener(this);
 
         findDialog.execute();
-
-        return;
     }
 
     public void actionPerformed(ActionEvent e) {