diff options
author | Noel Grandin <noel@peralex.com> | 2014-11-11 15:32:10 +0200 |
---|---|---|
committer | Noel Grandin <noelgrandin@gmail.com> | 2014-11-12 08:01:55 +0000 |
commit | 36ff1527c9cb20542d3097d123d221c40a356795 (patch) | |
tree | 52590ce642803a862bc331b3eae2e2b100b8e85f /qadevOOo | |
parent | 1eb31467a5af90fe41dc646dd716bdb7d3e5db45 (diff) |
java: reduce excessive code indentation levels
by using early return in some methods
Change-Id: I3611c8c89b3a94ef7e1772d178acf065fd7fcdc7
Reviewed-on: https://gerrit.libreoffice.org/12374
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
Tested-by: Noel Grandin <noelgrandin@gmail.com>
Diffstat (limited to 'qadevOOo')
-rw-r--r-- | qadevOOo/tests/java/ifc/chart/_XChartDataArray.java | 29 |
1 files changed, 15 insertions, 14 deletions
diff --git a/qadevOOo/tests/java/ifc/chart/_XChartDataArray.java b/qadevOOo/tests/java/ifc/chart/_XChartDataArray.java index 14e811efd67a..99a41b5df2f3 100644 --- a/qadevOOo/tests/java/ifc/chart/_XChartDataArray.java +++ b/qadevOOo/tests/java/ifc/chart/_XChartDataArray.java @@ -56,23 +56,24 @@ public class _XChartDataArray extends MultiMethodTest { mbExcludeSetRowAndSetColumn = true; msExcludeMessage = (String)o; } - if (!mbExcludeSetRowAndSetColumn) { - XPropertySet xProp = UnoRuntime.queryInterface(XPropertySet.class, oObj); - if(xProp != null) { - try { - boolean columnAsLabel = ((Boolean)xProp.getPropertyValue("ChartColumnAsLabel")).booleanValue(); - boolean rowAsLabel = ((Boolean)xProp.getPropertyValue("ChartRowAsLabel")).booleanValue(); - if (!columnAsLabel) { - xProp.setPropertyValue("ChartColumnAsLabel", Boolean.TRUE); - } - if (!rowAsLabel) { - xProp.setPropertyValue("ChartRowAsLabel", Boolean.TRUE); - } + if (mbExcludeSetRowAndSetColumn) { + return; + } + XPropertySet xProp = UnoRuntime.queryInterface(XPropertySet.class, oObj); + if(xProp != null) { + try { + boolean columnAsLabel = ((Boolean)xProp.getPropertyValue("ChartColumnAsLabel")).booleanValue(); + boolean rowAsLabel = ((Boolean)xProp.getPropertyValue("ChartRowAsLabel")).booleanValue(); + if (!columnAsLabel) { + xProp.setPropertyValue("ChartColumnAsLabel", Boolean.TRUE); } - catch(Exception e) { - // ignore + if (!rowAsLabel) { + xProp.setPropertyValue("ChartRowAsLabel", Boolean.TRUE); } } + catch(Exception e) { + // ignore + } } } |