diff options
| author | rbuj <robert.buj@gmail.com> | 2014-08-19 19:41:13 +0200 |
|---|---|---|
| committer | Noel Grandin <noelgrandin@gmail.com> | 2014-08-20 01:48:33 -0500 |
| commit | b9813addf2816e4058069892963e50a97643023d (patch) | |
| tree | efe28bc9e8399181fb24e38cb7057e006db74735 /xmerge | |
| parent | 1558b50e4fd5c26275efd824e731dc46fca781a4 (diff) | |
xmerge: Convert a primitive type into a string
Avoid unnecessary temporaries when converting primitive data types into a String.
Change-Id: If361383be52d8258be7c8c9bddf3ba05d5ca92c1
Reviewed-on: https://gerrit.libreoffice.org/11026
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
Tested-by: Noel Grandin <noelgrandin@gmail.com>
Diffstat (limited to 'xmerge')
3 files changed, 5 insertions, 7 deletions
diff --git a/xmerge/source/xmerge/java/org/openoffice/xmerge/converter/xml/ParaStyle.java b/xmerge/source/xmerge/java/org/openoffice/xmerge/converter/xml/ParaStyle.java index 0234a1b4a6ea..28c964c22452 100644 --- a/xmerge/source/xmerge/java/org/openoffice/xmerge/converter/xml/ParaStyle.java +++ b/xmerge/source/xmerge/java/org/openoffice/xmerge/converter/xml/ParaStyle.java @@ -534,7 +534,7 @@ public class ParaStyle extends Style implements Cloneable { for (int i = 0; i <= TEXT_INDENT; i++) { if (isSet[i]) { double temp = value[i] / 100.0; - String stringVal = (new Double(temp)).toString() + "mm"; + String stringVal = Double.toString(temp) + "mm"; node.setAttribute(attrName[i], stringVal); } } @@ -542,10 +542,10 @@ public class ParaStyle extends Style implements Cloneable { if (isSet[LINE_HEIGHT]) { String stringVal; if ((value[LINE_HEIGHT] & LH_PCT) != 0) - stringVal = (Integer.valueOf(value[LINE_HEIGHT] & LH_VALUEMASK)).toString() + "%"; + stringVal = Integer.toString(value[LINE_HEIGHT] & LH_VALUEMASK) + "%"; else { double temp = (value[LINE_HEIGHT] & LH_VALUEMASK) / 100.0; - stringVal = (new Double(temp)).toString() + "mm"; + stringVal = Double.toString(temp) + "mm"; } node.setAttribute(attrName[LINE_HEIGHT], stringVal); } diff --git a/xmerge/source/xmerge/java/org/openoffice/xmerge/converter/xml/TextStyle.java b/xmerge/source/xmerge/java/org/openoffice/xmerge/converter/xml/TextStyle.java index 83efc31d4660..c752f1e1fb4f 100644 --- a/xmerge/source/xmerge/java/org/openoffice/xmerge/converter/xml/TextStyle.java +++ b/xmerge/source/xmerge/java/org/openoffice/xmerge/converter/xml/TextStyle.java @@ -559,8 +559,7 @@ public class TextStyle extends Style implements Cloneable { node.setAttribute("style:text-position", "sub 58%"); if (sizeInPoints != 0) { - Integer fs = Integer.valueOf(sizeInPoints); - node.setAttribute("fo:font-size", fs.toString() + "pt"); + node.setAttribute("fo:font-size", Integer.toString(sizeInPoints) + "pt"); } if (fontName != null) diff --git a/xmerge/source/xmerge/java/org/openoffice/xmerge/converter/xml/sxc/CellStyle.java b/xmerge/source/xmerge/java/org/openoffice/xmerge/converter/xml/sxc/CellStyle.java index e437e18764d6..434912c6159d 100644 --- a/xmerge/source/xmerge/java/org/openoffice/xmerge/converter/xml/sxc/CellStyle.java +++ b/xmerge/source/xmerge/java/org/openoffice/xmerge/converter/xml/sxc/CellStyle.java @@ -412,8 +412,7 @@ public class CellStyle extends Style implements Cloneable { node.setAttribute("style:text-position", "sub 58%"); if (fmt.getFontSize() != 0) { - Integer fs = Integer.valueOf(fmt.getFontSize()); - node.setAttribute("fo:font-size", fs.toString() + "pt"); + node.setAttribute("fo:font-size", Integer.toString(fmt.getFontSize()) + "pt"); } if (fmt.getFontName() != null) |