diff options
author | Alexander Wilms <f.alexander.wilms@gmail.com> | 2014-02-25 23:12:58 +0100 |
---|---|---|
committer | Caolán McNamara <caolanm@redhat.com> | 2014-02-26 06:57:21 -0600 |
commit | c2a0a2bb49ac09e9fe63317cedaba85096b33cd3 (patch) | |
tree | 5b6cc07f5758aedec75435e1d87943ef77c4642e | |
parent | 2ef9df8aee696993d9eb235fd36caa4cbcfd3624 (diff) |
Remove visual noise from xmerge
Change-Id: I74b5e30d08ade403ec35298d5c783f02b22e886a
Reviewed-on: https://gerrit.libreoffice.org/8343
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>
12 files changed, 43 insertions, 43 deletions
diff --git a/xmerge/source/xmerge/java/org/openoffice/xmerge/Convert.java b/xmerge/source/xmerge/java/org/openoffice/xmerge/Convert.java index 9912cb642e25..883d137bd90c 100644 --- a/xmerge/source/xmerge/java/org/openoffice/xmerge/Convert.java +++ b/xmerge/source/xmerge/java/org/openoffice/xmerge/Convert.java @@ -192,7 +192,7 @@ public class Convert implements Cloneable { if (toOffice) { // From device format to Office format - // + DocumentDeserializerFactory myDocDeserializerFactory = ci.getDocDeserializerFactory(); DocumentDeserializer deser = @@ -206,7 +206,7 @@ public class Convert implements Cloneable { } else { // From Office format to device format - // + DocumentSerializerFactory myDocSerializerFactory = ci.getDocSerializerFactory(); @@ -244,7 +244,7 @@ public class Convert implements Cloneable { if (toOffice) { // From device format to Office format - // + DocumentDeserializerFactory myDocDeserializerFactory = ci.getDocDeserializerFactory(); DocumentDeserializer deser = @@ -260,7 +260,7 @@ public class Convert implements Cloneable { } else { // From Office format to device format - // + DocumentSerializerFactory myDocSerializerFactory = ci.getDocSerializerFactory(); 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 80d42f51d05d..986e53910e85 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 @@ -231,7 +231,7 @@ public class ParaStyle extends Style implements Cloneable { // Look for children. Only ones we care about are "style:properties" // nodes. If any are found, recursively traverse them, passing // along the style element to add properties to. - // + if (node.hasChildNodes()) { NodeList children = node.getChildNodes(); int len = children.getLength(); diff --git a/xmerge/source/xmerge/java/org/openoffice/xmerge/converter/xml/sxc/DocumentMergerImpl.java b/xmerge/source/xmerge/java/org/openoffice/xmerge/converter/xml/sxc/DocumentMergerImpl.java index fcdaba41edb9..fd01ec88111c 100644 --- a/xmerge/source/xmerge/java/org/openoffice/xmerge/converter/xml/sxc/DocumentMergerImpl.java +++ b/xmerge/source/xmerge/java/org/openoffice/xmerge/converter/xml/sxc/DocumentMergerImpl.java @@ -133,12 +133,12 @@ public class DocumentMergerImpl implements DocumentMerger { Node workSheet= workSheetList2.item(j); // try to match the workSheet - // + Node matchingWorkSheet = matchWorkSheet(workSheet, workSheetList1); // add the new WorkSheet to the original document iff match not // found - // + if (matchingWorkSheet == null) { Node cloneNode = XmlUtil.deepClone(officeBody, workSheet); officeBody.appendChild(cloneNode); diff --git a/xmerge/source/xmerge/java/org/openoffice/xmerge/converter/xml/sxc/SxcDocumentDeserializer.java b/xmerge/source/xmerge/java/org/openoffice/xmerge/converter/xml/sxc/SxcDocumentDeserializer.java index d7e6b55d2fa7..55f9c3917a23 100644 --- a/xmerge/source/xmerge/java/org/openoffice/xmerge/converter/xml/sxc/SxcDocumentDeserializer.java +++ b/xmerge/source/xmerge/java/org/openoffice/xmerge/converter/xml/sxc/SxcDocumentDeserializer.java @@ -167,7 +167,7 @@ public abstract class SxcDocumentDeserializer implements OfficeConstants, // use any value for password here. If StarCalc XML supports // passwords in the future, we should try to get the correct // password value here. - // + decoder = createDecoder(workbookName, worksheetNames, "password"); Debug.log(Debug.TRACE, "<?xml version=\"1.0\" encoding=\"UTF-8\"?>"); @@ -438,7 +438,7 @@ public abstract class SxcDocumentDeserializer implements OfficeConstants, // The number of columns in the spreadsheet int lastColumn = decoder.getNumberOfColumns(); - // + Node autoStylesNode = null; // Loop over all cells in the spreadsheet diff --git a/xmerge/source/xmerge/java/org/openoffice/xmerge/converter/xml/sxc/SxcDocumentSerializer.java b/xmerge/source/xmerge/java/org/openoffice/xmerge/converter/xml/sxc/SxcDocumentSerializer.java index cf4f0b401677..c9bee878fd23 100644 --- a/xmerge/source/xmerge/java/org/openoffice/xmerge/converter/xml/sxc/SxcDocumentSerializer.java +++ b/xmerge/source/xmerge/java/org/openoffice/xmerge/converter/xml/sxc/SxcDocumentSerializer.java @@ -717,7 +717,7 @@ public abstract class SxcDocumentSerializer implements OfficeConstants, } else { // Text node, Date node, or Time node - // + Debug.log(Debug.INFO, "TextNode, DateNode, TimeNode or BooleanNode\n"); // This handles the case where we have style information but no content diff --git a/xmerge/source/xmerge/java/org/openoffice/xmerge/converter/xml/sxc/SxcPluginFactory.java b/xmerge/source/xmerge/java/org/openoffice/xmerge/converter/xml/sxc/SxcPluginFactory.java index ac4cc33b5dbf..39ec87bfdf0a 100644 --- a/xmerge/source/xmerge/java/org/openoffice/xmerge/converter/xml/sxc/SxcPluginFactory.java +++ b/xmerge/source/xmerge/java/org/openoffice/xmerge/converter/xml/sxc/SxcPluginFactory.java @@ -54,7 +54,7 @@ public abstract class SxcPluginFactory throws IOException { // read zipped XML stream - // + SxcDocument doc = new SxcDocument(name); doc.read(is); return doc; @@ -64,7 +64,7 @@ public abstract class SxcPluginFactory throws IOException { // read zipped XML stream - // + SxcDocument doc = new SxcDocument(name); doc.read(is,isZip); return doc; diff --git a/xmerge/source/xmerge/java/org/openoffice/xmerge/converter/xml/xslt/PluginFactoryImpl.java b/xmerge/source/xmerge/java/org/openoffice/xmerge/converter/xml/xslt/PluginFactoryImpl.java index 878ba9c42c52..39aa2b3a9c63 100644 --- a/xmerge/source/xmerge/java/org/openoffice/xmerge/converter/xml/xslt/PluginFactoryImpl.java +++ b/xmerge/source/xmerge/java/org/openoffice/xmerge/converter/xml/xslt/PluginFactoryImpl.java @@ -153,7 +153,7 @@ public final class PluginFactoryImpl extends PluginFactory } catch (Exception e) { // It is okay for the property file to not exist. - // + } return ext; } diff --git a/xmerge/source/xmerge/java/org/openoffice/xmerge/merger/merge/CharacterBaseParagraphMerge.java b/xmerge/source/xmerge/java/org/openoffice/xmerge/merger/merge/CharacterBaseParagraphMerge.java index a303794c67e5..0b48d7a78655 100644 --- a/xmerge/source/xmerge/java/org/openoffice/xmerge/merger/merge/CharacterBaseParagraphMerge.java +++ b/xmerge/source/xmerge/java/org/openoffice/xmerge/merger/merge/CharacterBaseParagraphMerge.java @@ -143,7 +143,7 @@ public final class CharacterBaseParagraphMerge String tmpString; // Handle situation where getNodeValue returns null - // + if (orgNode.getNodeValue() != null) tmpString = orgNode.getNodeValue(); else @@ -167,11 +167,11 @@ public final class CharacterBaseParagraphMerge int lastDiffPosition = -1; // starting to diff - // + for (int j = startDiffNum; j < endDiffNum; j++) { // copy any contents before the diff - // + if (diffs[j].getOrgPosition() > orgTextPosition) { // need to flush first if (cacheLength > 0) { @@ -199,7 +199,7 @@ public final class CharacterBaseParagraphMerge // for any deleted characters, just skip without copy // but still need to take care the cached characters - // + if (diffs[j].getOperation() == Difference.DELETE) { orgTextPosition++; diff --git a/xmerge/source/xmerge/java/org/openoffice/xmerge/test/Driver.java b/xmerge/source/xmerge/java/org/openoffice/xmerge/test/Driver.java index bd6ada6ca315..a527556a288c 100644 --- a/xmerge/source/xmerge/java/org/openoffice/xmerge/test/Driver.java +++ b/xmerge/source/xmerge/java/org/openoffice/xmerge/test/Driver.java @@ -68,7 +68,7 @@ public final class Driver { public static void main(String args[]) { // Register jarfiles - // + String propFile = "ConverterInfoList.properties"; ConverterInfoList cil = null; try { diff --git a/xmerge/source/xmerge/java/org/openoffice/xmerge/util/registry/ConverterInfo.java b/xmerge/source/xmerge/java/org/openoffice/xmerge/util/registry/ConverterInfo.java index bf88d21671c1..d6b171c8050d 100644 --- a/xmerge/source/xmerge/java/org/openoffice/xmerge/util/registry/ConverterInfo.java +++ b/xmerge/source/xmerge/java/org/openoffice/xmerge/util/registry/ConverterInfo.java @@ -44,7 +44,7 @@ public class ConverterInfo { static { // This needs to be updated to reflect all valid office types. - // + validOfficeTypes = new String[2]; validOfficeTypes[0] = SxwType; validOfficeTypes[1] = SxcType; @@ -110,7 +110,7 @@ public class ConverterInfo { piClassLoader = this.getClass().getClassLoader(); // Get instance of the PluginFactory. - // + try { URL jarURL = new URL(jarName); URLClassLoader loader = new URLClassLoader(new URL[] { jarURL }, @@ -123,7 +123,7 @@ public class ConverterInfo { piPluginFactory = ( PluginFactory ) construct.newInstance(arguments); // See which interfaces the plug-in PluginFactory supports. - // + Class<?>[] cl = piPluginFactory.getClass().getInterfaces(); for (int i=0; i < cl.length; i++) { @@ -185,7 +185,7 @@ public class ConverterInfo { piClassLoader = this.getClass().getClassLoader(); // Get instance of the PluginFactory. - // + try { URL jarURL = new URL(jarName); URLClassLoader loader = new URLClassLoader(new URL[] { jarURL }, @@ -198,7 +198,7 @@ public class ConverterInfo { piPluginFactory = ( PluginFactory ) construct.newInstance(arguments); // See which interfaces the plug-in PluginFactory supports. - // + Class<?>[] cl = piPluginFactory.getClass().getInterfaces(); for (int i=0; i < cl.length; i++) { diff --git a/xmerge/source/xmerge/java/org/openoffice/xmerge/util/registry/ConverterInfoMgr.java b/xmerge/source/xmerge/java/org/openoffice/xmerge/util/registry/ConverterInfoMgr.java index 245fb4ac4419..c53437aa48fe 100644 --- a/xmerge/source/xmerge/java/org/openoffice/xmerge/util/registry/ConverterInfoMgr.java +++ b/xmerge/source/xmerge/java/org/openoffice/xmerge/util/registry/ConverterInfoMgr.java @@ -56,7 +56,7 @@ public final class ConverterInfoMgr { ConverterInfo converterInfo; // Validate - // + if (ci.getDisplayName() == null) { RegistryException re = new RegistryException( "Converter must have valid name."); @@ -80,7 +80,7 @@ public final class ConverterInfoMgr { // Verify there is no converter with the same Display Name in // the registry. - // + Iterator<ConverterInfo> ciEnum = converterInfoList.iterator(); while (ciEnum.hasNext()) { converterInfo = ciEnum.next(); @@ -93,7 +93,7 @@ public final class ConverterInfoMgr { // Since this is a adding to a static Vector, make sure this // add method call is synchronized. - // + synchronized (converterInfoList) { converterInfoList.add(ci); } @@ -216,7 +216,7 @@ public final class ConverterInfoMgr { } // Loop over elements comparing with deviceFromMime - // + Iterator<ConverterInfo> ciEnum = converterInfoList.iterator(); while (ciEnum.hasNext()) { @@ -225,7 +225,7 @@ public final class ConverterInfoMgr { Iterator<String> fromEnum = converterInfo.getDeviceMime(); // Loop over the deviceMime types. - // + while (fromEnum.hasNext()) { String fromDeviceInfo = fromEnum.next(); if (deviceMime.trim().equals(fromDeviceInfo) && @@ -259,7 +259,7 @@ public final class ConverterInfoMgr { ConverterInfo[] converterInfo = new ConverterInfo[2]; // Loop over elements comparing with deviceFromMime - // + Iterator<ConverterInfo> cifEnum = converterInfoList.iterator(); while (cifEnum.hasNext()) { @@ -269,7 +269,7 @@ public final class ConverterInfoMgr { // Loop over the deviceMime types looking for a deviceFromMime // match. - // + while (fromEnum.hasNext()) { String fromDeviceInfo = fromEnum.next(); @@ -277,7 +277,7 @@ public final class ConverterInfoMgr { // Found a a match for deviceFrom. Now loop over the // elements comparing with deviceToMime - // + Iterator<ConverterInfo> citEnum = converterInfoList.iterator(); while (citEnum.hasNext()) { @@ -287,14 +287,14 @@ public final class ConverterInfoMgr { // Loop over deviceMime types looking for a // deviceToMime match. - // + while (toEnum.hasNext()) { String toDeviceInfo = toEnum.next(); if (deviceToMime.trim().equals(toDeviceInfo) && fromOfficeInfo.equals(toOfficeInfo)) { // Found a match - // + return (converterInfo); } } @@ -343,12 +343,12 @@ public final class ConverterInfoMgr { System.out.println(""); // Quit - // + if (c == 'Q') { exitFlag = true; // Load by Jarfile - // + } else if (c == 'L') { System.out.println("Enter path to jarfile: "); @@ -374,7 +374,7 @@ public final class ConverterInfoMgr { } // Unload by Display Name or Jarfile - // + } else if (c == 'T') { if (validate== true){ System.out.println("Validation switched off"); @@ -414,7 +414,7 @@ public final class ConverterInfoMgr { } // Find Office Mime - // + } else if (c == 'F' || c == 'C') { String findMimeOne = null; @@ -467,7 +467,7 @@ public final class ConverterInfoMgr { } // View - // + } else if (c == 'V') { Iterator<ConverterInfo> ciEnum = ConverterInfoMgr.getConverterInfoEnumeration(); diff --git a/xmerge/source/xmerge/java/org/openoffice/xmerge/util/registry/ConverterInfoReader.java b/xmerge/source/xmerge/java/org/openoffice/xmerge/util/registry/ConverterInfoReader.java index b4083f865324..caaffd614f8c 100644 --- a/xmerge/source/xmerge/java/org/openoffice/xmerge/util/registry/ConverterInfoReader.java +++ b/xmerge/source/xmerge/java/org/openoffice/xmerge/util/registry/ConverterInfoReader.java @@ -89,19 +89,19 @@ public class ConverterInfoReader { jarfilename = jar; // Get Jar via URL - // + url = new URL("jar:" + jar + "!/META-INF/converter.xml"); jarConnection = (JarURLConnection)url.openConnection(); jarentry = jarConnection.getJarEntry(); jarfile = jarConnection.getJarFile(); // Build the InputSource - // + istream = jarfile.getInputStream(jarentry); isource = new InputSource(istream); // Get the DOM builder and build the document. - // + builderFactory = DocumentBuilderFactory.newInstance(); //DTD validation @@ -109,12 +109,12 @@ public class ConverterInfoReader { System.out.println("Validating xml..."); builderFactory.setValidating(true); } - // + builder = builderFactory.newDocumentBuilder(); document = builder.parse(isource); // Parse the document. - // + parseDocument(); } |