kriegaex commented on code in PR #125:
URL: https://github.com/apache/xalan-java/pull/125#discussion_r1399930714


##########
serializer/src/main/java/org/apache/xml/serializer/Version.java:
##########
@@ -20,33 +20,97 @@
  */
 package org.apache.xml.serializer;
 
+
+import java.io.FileInputStream;
+import java.io.IOException;
+import java.io.InputStream;
+import java.util.Properties;
+import java.util.regex.Matcher;
+import java.util.regex.Pattern;
+
 /**
  * Administrative class to keep track of the version number of
  * the Serializer release.
  * <P>This class implements the upcoming standard of having
- * org.apache.project-name.Version.getVersion() be a standard way 
- * to get version information.</P> 
+ * org.apache.project-name.Version.getVersion() be a standard way
+ * to get version information.</P>
  * @xsl.usage general
  */
 public final class Version
 {
+  private static final String POM_PROPERTIES_JAR = 
"org/apache/xml/serializer/version.properties";
+  private static final String POM_PROPERTIES_FILE_SYSTEM = 
"serializer/target/classes" + POM_PROPERTIES_JAR;
+  private static final String VERSION_NUMBER_PATTERN = 
"^(\\d+)[.](\\d+)[.](D)?(\\d+)(-SNAPSHOT)?$";
+  private static final String NO_VERSION = "0.0.0";
+
+  private static String version = NO_VERSION;
+  private static int majorVersionNum;
+  private static int releaseVersionNum;
+  private static int maintenanceVersionNum;
+  private static int developmentVersionNum;
+
+  private static boolean snapshot;

Review Comment:
   No, it does not. I had checked that before.
   
   ```text
   --- dependency:2.8:tree (default-cli) @ serializer ---        
   xalan:serializer:jar:2.7.3                                    
   +- org.apache.bcel:bcel:jar:6.7.0:compile                     
   |  \- org.apache.commons:commons-lang3:jar:3.12.0:compile     
   +- commons-logging:commons-logging:jar:1.2:provided           
   +- javax:javaee-api:jar:6.0:provided                          
   +- javax.servlet:servlet-api:jar:2.5:provided                 
   +- xerces:xercesImpl:jar:2.12.2:provided                      
   \- xml-apis:xml-apis:jar:1.4.01:provided                      
   ```
   
   You see that I already optimised `XSLProcessorVersion` to avoid duplicate 
code. Again, I am pointing you to my PR description. Search for the paragraph 
starting with "There are 3 version classes". All those concerns you seem to 
have were already considered by me. Please do read descriptions too, not just 
review code in isolation.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@xalan.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@xalan.apache.org
For additional commands, e-mail: dev-h...@xalan.apache.org

Reply via email to