Changeset 288 for trunk/openjdk/jdk/make/tools

Timestamp:

Apr 30, 2011, 1:39:32 AM (14 years ago)

Author:

dmik

Message:

jdk: Fixed: the dtdbuilder tool would write the binary DTD to stdout causing "\n" to "\r\n" expansion on OS/2 (Win32 too?) and thus breaking the resulting file which in turn would completely break support of HTML text in Swing components [vendor bug].

File:

• trunk/openjdk/jdk/make/tools/src/build/tools/dtdbuilder/DTDBuilder.java (modified) (3 diffs)

trunk/openjdk/jdk/make/tools/src/build/tools/dtdbuilder/DTDBuilder.java

@@ -30 +30 @@
 import java.io.File;
 import java.io.FileInputStream;
+import java.io.FileOutputStream;
 import java.io.IOException;
 import java.io.FileNotFoundException;
@@ -289 +290 @@
         }
 
+        if (argv.length < 2) {
+            System.err.println("Must specify arguments: <dtd_file> <output_file>");
+            return;
+        }
+
         DTDBuilder dtd = null;
         try {
@@ -302 +308 @@
         }
         try {
-            DataOutputStream str = new DataOutputStream(System.out);
+            DataOutputStream str = new DataOutputStream(new FileOutputStream(argv[1]));
             dtd.save(str, argv[0]);
             str.close();
         } catch (IOException ex) {
-            ex.printStackTrace();
+            System.err.println("Could not save BDTD file "+argv[1]);
+            e.printStackTrace(System.err);
             System.exit(1);
         }