Changeset 1484

Timestamp:

04/22/11 08:41:39 (13 months ago)

Author:

jharrop

Message:

Some comments

Location:

trunk/docx4j/src/main/java/org/docx4j/openpackaging/parts

Files:

• WordprocessingML/EmbeddedPackagePart.java (modified) (1 diff)
• relationships/Namespaces.java (modified) (1 diff)

trunk/docx4j/src/main/java/org/docx4j/openpackaging/parts/WordprocessingML/EmbeddedPackagePart.java

@@ -1 +1 @@
 package org.docx4j.openpackaging.parts.WordprocessingML;
+// Ideally this part would have been created in the parts 
+// package itself, since it is also used when a 
+// chart is embedded in a pptx.  ie there is nothing
+// about it which is specific to WML.
 
 import org.apache.log4j.Logger;

trunk/docx4j/src/main/java/org/docx4j/openpackaging/parts/relationships/Namespaces.java

@@ -227 +227 @@
         
         ///xl/charts/chart1.xml
+        // Should have just called this CHART or RELS_CHART,
+        // since it is also used in PresentationML
         public final static String SPREADSHEETML_CHART =
         "http://schemas.openxmlformats.org/officeDocument/2006/relationships/chart";