I think Objects.equals(Object, Object) should be used when comparing result and expected. Otherwise, the fix looks OK to me.

Masayoshi

On 5/8/2013 7:10 AM, Naoto Sato wrote:
On 5/6/13 9:30 PM, Masayoshi Okutsu wrote:
Not sure how the test can detect bugs in the provider of the host
adapter.

The test case only checks the Java side code paths, i.e., whether the returned display name from the HOST adapter is honored over JRE's one or not. In the case of "mk", that detects the failure.

BTW, some lines are very long requiring a wide screen monitor.

Fixed: http://cr.openjdk.java.net/~naoto/8013233/webrev.01/

Naoto


Masayoshi

On 5/7/2013 3:50 AM, Naoto Sato wrote:
Hello,

Please review the following changeset:

http://cr.openjdk.java.net/~naoto/8013233/webrev.00/

for the following bug:

http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=8013233

The original test case assumed Windows ver 6.0 or upper always contain
the localized display name for Macedonian, which wasn't the case in
some installation. Removed the hard coded name and replaced them with
the actual returned name from the host adapter.

Naoto



Reply via email to