From 5f35e3c944dde67cb860f9c1bfe0fc34cb0b0d88 Mon Sep 17 00:00:00 2001 From: Thibault Meyer Date: Mon, 7 Nov 2016 07:46:58 +0100 Subject: [PATCH] Remove redundant local variables --- .../ebeaninternal/server/grammer/EqlAdapterHelper.java | 4 +--- .../tests/model/ivo/converter/ExhangeCompoundType.java | 4 +--- .../model/ivo/converter/JodaIntervalCompoundType.java | 3 +-- .../avaje/tests/query/other/TestFindIterateHeapDump.java | 7 ++----- 4 files changed, 5 insertions(+), 13 deletions(-) diff --git a/src/main/java/com/avaje/ebeaninternal/server/grammer/EqlAdapterHelper.java b/src/main/java/com/avaje/ebeaninternal/server/grammer/EqlAdapterHelper.java index 93d84b882..b08e642eb 100644 --- a/src/main/java/com/avaje/ebeaninternal/server/grammer/EqlAdapterHelper.java +++ b/src/main/java/com/avaje/ebeaninternal/server/grammer/EqlAdapterHelper.java @@ -132,8 +132,6 @@ class EqlAdapterHelper { } private String unquote(String value) { - String raw = value.substring(1, value.length() - 1); - //raw.replaceAll(); - return raw; + return value.substring(1, value.length() - 1); } } diff --git a/src/test/java/com/avaje/tests/model/ivo/converter/ExhangeCompoundType.java b/src/test/java/com/avaje/tests/model/ivo/converter/ExhangeCompoundType.java index e7c1db45b..7ccd33f12 100644 --- a/src/test/java/com/avaje/tests/model/ivo/converter/ExhangeCompoundType.java +++ b/src/test/java/com/avaje/tests/model/ivo/converter/ExhangeCompoundType.java @@ -14,9 +14,7 @@ public class ExhangeCompoundType implements CompoundType { @SuppressWarnings({ "unchecked", "rawtypes" }) public CompoundTypeProperty[] getProperties() { - - CompoundTypeProperty[] props = {new RateProp(), new CMoneyProp()}; - return props; + return new CompoundTypeProperty[]{new RateProp(), new CMoneyProp()}; } static class RateProp implements CompoundTypeProperty { diff --git a/src/test/java/com/avaje/tests/model/ivo/converter/JodaIntervalCompoundType.java b/src/test/java/com/avaje/tests/model/ivo/converter/JodaIntervalCompoundType.java index 21d06060e..82780da7d 100644 --- a/src/test/java/com/avaje/tests/model/ivo/converter/JodaIntervalCompoundType.java +++ b/src/test/java/com/avaje/tests/model/ivo/converter/JodaIntervalCompoundType.java @@ -13,8 +13,7 @@ public class JodaIntervalCompoundType implements CompoundType{ @SuppressWarnings({ "unchecked", "rawtypes" }) public CompoundTypeProperty[] getProperties() { - CompoundTypeProperty[] props = {new Start(), new End()}; - return props; + return new CompoundTypeProperty[]{new Start(), new End()}; } static class Start implements CompoundTypeProperty { diff --git a/src/test/java/com/avaje/tests/query/other/TestFindIterateHeapDump.java b/src/test/java/com/avaje/tests/query/other/TestFindIterateHeapDump.java index 08df73b74..f5ae9ad73 100644 --- a/src/test/java/com/avaje/tests/query/other/TestFindIterateHeapDump.java +++ b/src/test/java/com/avaje/tests/query/other/TestFindIterateHeapDump.java @@ -3,7 +3,6 @@ package com.avaje.tests.query.other; import com.avaje.ebean.BaseTestCase; import com.avaje.ebean.Ebean; import com.avaje.ebean.EbeanServer; -import com.avaje.ebean.QueryEachConsumer; import com.avaje.tests.model.basic.EBasic; import com.sun.management.HotSpotDiagnosticMXBean; import org.junit.Ignore; @@ -115,10 +114,8 @@ public class TestFindIterateHeapDump extends BaseTestCase { */ private static HotSpotDiagnosticMXBean getHotspotMBean() { try { - MBeanServer server = ManagementFactory.getPlatformMBeanServer(); - HotSpotDiagnosticMXBean bean = ManagementFactory.newPlatformMXBeanProxy(server, HOTSPOT_BEAN_NAME, - HotSpotDiagnosticMXBean.class); - return bean; + final MBeanServer server = ManagementFactory.getPlatformMBeanServer(); + return ManagementFactory.newPlatformMXBeanProxy(server, HOTSPOT_BEAN_NAME, HotSpotDiagnosticMXBean.class); } catch (RuntimeException re) { throw re; } catch (Exception exp) {