From 80aee3ba189d50bb9c059cc397b04fcf32b79a4c Mon Sep 17 00:00:00 2001 From: alanb Date: Wed, 2 Mar 2011 16:56:07 +0000 Subject: [PATCH] 7021987: native2ascii "file cannot be read" error message is broken Reviewed-by: lancea, mchung --- src/share/classes/sun/tools/native2ascii/Main.java | 14 ++++++-------- test/sun/tools/native2ascii/Native2AsciiTests.sh | 11 ++++++++++- 2 files changed, 16 insertions(+), 9 deletions(-) diff --git a/src/share/classes/sun/tools/native2ascii/Main.java b/src/share/classes/sun/tools/native2ascii/Main.java index 2d127344b..d19d27a64 100644 --- a/src/share/classes/sun/tools/native2ascii/Main.java +++ b/src/share/classes/sun/tools/native2ascii/Main.java @@ -94,7 +94,7 @@ public class Main { * Run the converter */ public synchronized boolean convert(String argv[]){ - Vector v = new Vector(2); + List v = new ArrayList<>(2); File outputFile = null; boolean createOutputFile = false; @@ -115,7 +115,7 @@ public class Main { usage(); return false; } - v.addElement(argv[i]); + v.add(argv[i]); } } if (encodingString == null) @@ -126,11 +126,11 @@ public class Main { initializeConverter(); if (v.size() == 1) - inputFileName = (String)v.elementAt(0); + inputFileName = v.get(0); if (v.size() == 2) { - inputFileName = (String)v.elementAt(0); - outputFileName = (String)v.elementAt(1); + inputFileName = v.get(0); + outputFileName = v.get(1); createOutputFile = true; } @@ -363,9 +363,7 @@ public class Main { private String formatMsg(String key, String arg) { String msg = getMsg(key); - String[] args = new String[1]; - args[0] = arg; - return MessageFormat.format(msg, (Object)args); + return MessageFormat.format(msg, arg); } diff --git a/test/sun/tools/native2ascii/Native2AsciiTests.sh b/test/sun/tools/native2ascii/Native2AsciiTests.sh index 8c32cf842..82ee282bd 100644 --- a/test/sun/tools/native2ascii/Native2AsciiTests.sh +++ b/test/sun/tools/native2ascii/Native2AsciiTests.sh @@ -24,7 +24,7 @@ # # @test -# @bug 4630463 4630971 4636448 4701617 4721296 4710890 6247817 +# @bug 4630463 4630971 4636448 4701617 4721296 4710890 6247817 7021987 # @summary Tests miscellaneous native2ascii bugfixes and regressions @@ -100,6 +100,15 @@ rm -f x.* $N2A -reverse -encoding MS932 $TESTSRC/A2N_4701617 x.out check 4701617 $TESTSRC/A2N_4701617.expected x.out +# Check that the inputfile appears in the error message when not found + +badin="DoesNotExist" +$N2A $badin x.out | grep "$badin" > /dev/null +if [ $? != 0 ]; then + echo "\"$badin\" expected to appear in error message" + exit 1 +fi + # for win32 only ensure when output file pre-exists that # native2ascii tool will simply overwrite with the expected # output file (fixed bugID 4710890) -- GitLab