[Libreoffice-commits] core.git: xmerge/source

Alexander Wilms f.alexander.wilms at gmail.com
Wed Feb 26 04:57:49 PST 2014


 xmerge/source/xmerge/java/org/openoffice/xmerge/Convert.java                                   |    8 +-
 xmerge/source/xmerge/java/org/openoffice/xmerge/converter/xml/ParaStyle.java                   |    2 
 xmerge/source/xmerge/java/org/openoffice/xmerge/converter/xml/sxc/DocumentMergerImpl.java      |    4 -
 xmerge/source/xmerge/java/org/openoffice/xmerge/converter/xml/sxc/SxcDocumentDeserializer.java |    4 -
 xmerge/source/xmerge/java/org/openoffice/xmerge/converter/xml/sxc/SxcDocumentSerializer.java   |    2 
 xmerge/source/xmerge/java/org/openoffice/xmerge/converter/xml/sxc/SxcPluginFactory.java        |    4 -
 xmerge/source/xmerge/java/org/openoffice/xmerge/converter/xml/xslt/PluginFactoryImpl.java      |    2 
 xmerge/source/xmerge/java/org/openoffice/xmerge/merger/merge/CharacterBaseParagraphMerge.java  |    8 +-
 xmerge/source/xmerge/java/org/openoffice/xmerge/test/Driver.java                               |    2 
 xmerge/source/xmerge/java/org/openoffice/xmerge/util/registry/ConverterInfo.java               |   10 +--
 xmerge/source/xmerge/java/org/openoffice/xmerge/util/registry/ConverterInfoMgr.java            |   30 +++++-----
 xmerge/source/xmerge/java/org/openoffice/xmerge/util/registry/ConverterInfoReader.java         |   10 +--
 12 files changed, 43 insertions(+), 43 deletions(-)

New commits:
commit c2a0a2bb49ac09e9fe63317cedaba85096b33cd3
Author: Alexander Wilms <f.alexander.wilms at gmail.com>
Date:   Tue Feb 25 23:12:58 2014 +0100

    Remove visual noise from xmerge
    
    Change-Id: I74b5e30d08ade403ec35298d5c783f02b22e886a
    Reviewed-on: https://gerrit.libreoffice.org/8343
    Reviewed-by: Caolán McNamara <caolanm at redhat.com>
    Tested-by: Caolán McNamara <caolanm at redhat.com>

diff --git a/xmerge/source/xmerge/java/org/openoffice/xmerge/Convert.java b/xmerge/source/xmerge/java/org/openoffice/xmerge/Convert.java
index 9912cb6..883d137 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 80d42f5..986e539 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 fcdaba4..fd01ec8 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 d7e6b55..55f9c39 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 cf4f0b4..c9bee87 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 ac4cc33..39ec87b 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 878ba9c..39aa2b3 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 a303794..0b48d7a 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 bd6ada6..a527556 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 bf88d21..d6b171c 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 245fb4a..c53437a 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 b4083f8..caaffd6 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();
     }
 


More information about the Libreoffice-commits mailing list