Kaydet (Commit) 62ef9367 authored tarafından rbuj's avatar rbuj Kaydeden (comit) Noel Grandin

xmerge: remove String constructor invocation

Change-Id: Ibd7a94c0f53be3b3a3e9ee764dbdeb6199857441
Reviewed-on: https://gerrit.libreoffice.org/10576Reviewed-by: 's avatarNoel Grandin <noelgrandin@gmail.com>
Tested-by: 's avatarNoel Grandin <noelgrandin@gmail.com>
üst 7d06a060
...@@ -191,7 +191,7 @@ public class EmbeddedXMLObject extends EmbeddedObject { ...@@ -191,7 +191,7 @@ public class EmbeddedXMLObject extends EmbeddedObject {
builder = factory.newDocumentBuilder(); builder = factory.newDocumentBuilder();
} }
byte[] data = zipFile.getNamedBytes(new String(objName + "/" + name)); byte[] data = zipFile.getNamedBytes((objName + "/" + name));
if (data != null) { if (data != null) {
return OfficeDocument.parse(builder, data); return OfficeDocument.parse(builder, data);
} }
...@@ -222,15 +222,15 @@ public class EmbeddedXMLObject extends EmbeddedObject { ...@@ -222,15 +222,15 @@ public class EmbeddedXMLObject extends EmbeddedObject {
void write(OfficeZip zip) throws IOException { void write(OfficeZip zip) throws IOException {
if (hasChanged == true) { if (hasChanged == true) {
if (contentDOM != null) { if (contentDOM != null) {
zip.setNamedBytes(new String(objName + "/content.xml"), zip.setNamedBytes((objName + "/content.xml"),
OfficeDocument.docToBytes(contentDOM)); OfficeDocument.docToBytes(contentDOM));
} }
if (settingsDOM != null) { if (settingsDOM != null) {
zip.setNamedBytes(new String(objName + "/settings.xml"), zip.setNamedBytes((objName + "/settings.xml"),
OfficeDocument.docToBytes(settingsDOM)); OfficeDocument.docToBytes(settingsDOM));
} }
if (stylesDOM != null) { if (stylesDOM != null) {
zip.setNamedBytes(new String(objName + "/styles.xml"), zip.setNamedBytes((objName + "/styles.xml"),
OfficeDocument.docToBytes(stylesDOM)); OfficeDocument.docToBytes(stylesDOM));
} }
} }
...@@ -249,8 +249,7 @@ public class EmbeddedXMLObject extends EmbeddedObject { ...@@ -249,8 +249,7 @@ public class EmbeddedXMLObject extends EmbeddedObject {
Element contentNode = manifestDoc.createElement(OfficeConstants.TAG_MANIFEST_FILE); Element contentNode = manifestDoc.createElement(OfficeConstants.TAG_MANIFEST_FILE);
contentNode.setAttribute(OfficeConstants.ATTRIBUTE_MANIFEST_FILE_TYPE, "text/xml"); contentNode.setAttribute(OfficeConstants.ATTRIBUTE_MANIFEST_FILE_TYPE, "text/xml");
contentNode.setAttribute(OfficeConstants.ATTRIBUTE_MANIFEST_FILE_PATH, contentNode.setAttribute(OfficeConstants.ATTRIBUTE_MANIFEST_FILE_PATH, (objName + "/content.xml"));
new String(objName + "/content.xml"));
root.appendChild(contentNode); root.appendChild(contentNode);
} }
...@@ -259,8 +258,7 @@ public class EmbeddedXMLObject extends EmbeddedObject { ...@@ -259,8 +258,7 @@ public class EmbeddedXMLObject extends EmbeddedObject {
Element settingsNode = manifestDoc.createElement(OfficeConstants.TAG_MANIFEST_FILE); Element settingsNode = manifestDoc.createElement(OfficeConstants.TAG_MANIFEST_FILE);
settingsNode.setAttribute(OfficeConstants.ATTRIBUTE_MANIFEST_FILE_TYPE, "text/xml"); settingsNode.setAttribute(OfficeConstants.ATTRIBUTE_MANIFEST_FILE_TYPE, "text/xml");
settingsNode.setAttribute(OfficeConstants.ATTRIBUTE_MANIFEST_FILE_PATH, settingsNode.setAttribute(OfficeConstants.ATTRIBUTE_MANIFEST_FILE_PATH, (objName + "/settings.xml"));
new String(objName + "/settings.xml"));
root.appendChild(settingsNode); root.appendChild(settingsNode);
} }
...@@ -269,16 +267,14 @@ public class EmbeddedXMLObject extends EmbeddedObject { ...@@ -269,16 +267,14 @@ public class EmbeddedXMLObject extends EmbeddedObject {
Element stylesNode = manifestDoc.createElement(OfficeConstants.TAG_MANIFEST_FILE); Element stylesNode = manifestDoc.createElement(OfficeConstants.TAG_MANIFEST_FILE);
stylesNode.setAttribute(OfficeConstants.ATTRIBUTE_MANIFEST_FILE_TYPE, "text/xml"); stylesNode.setAttribute(OfficeConstants.ATTRIBUTE_MANIFEST_FILE_TYPE, "text/xml");
stylesNode.setAttribute(OfficeConstants.ATTRIBUTE_MANIFEST_FILE_PATH, stylesNode.setAttribute(OfficeConstants.ATTRIBUTE_MANIFEST_FILE_PATH, (objName + "/styles.xml"));
new String(objName + "/styles.xml"));
} }
Element objectNode = manifestDoc.createElement(OfficeConstants.TAG_MANIFEST_FILE); Element objectNode = manifestDoc.createElement(OfficeConstants.TAG_MANIFEST_FILE);
objectNode.setAttribute(OfficeConstants.ATTRIBUTE_MANIFEST_FILE_TYPE, objType); objectNode.setAttribute(OfficeConstants.ATTRIBUTE_MANIFEST_FILE_TYPE, objType);
objectNode.setAttribute(OfficeConstants.ATTRIBUTE_MANIFEST_FILE_PATH, objectNode.setAttribute(OfficeConstants.ATTRIBUTE_MANIFEST_FILE_PATH, (objName + "/"));
new String(objName + "/"));
root.appendChild(objectNode); root.appendChild(objectNode);
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment