Request Review for small change to remove 2 warnings in rmic

Sebastian Sickelmann sebastian.sickelmann at gmx.de
Thu Aug 18 04:54:39 UTC 2011


Am 18.08.2011 06:46, schrieb Sebastian Sickelmann:
> Am 18.08.2011 00:52, schrieb Mandy Chung:
>> Thumbs up from me.   Are you going to include this change with 
>> another CR that you have a fix to be pushed?
>>
> I have some other improved that are reviewed by Joe actually. But 
> there are on a completly other topic and it
> should take some days to have all prepared for the push.
>
> Is the CR to small, or what is the reason for your question?
>
> I think i can fix 2 other warnings of the same kind and fix them 
> together.
> src/solaris/classes/sun/awt/X11/XIconWindow.java:95: warning: 
> non-varargs call of varargs method with inexact argument type for last 
> parameter;
> src/share/classes/sun/tools/serialver/SerialVer.java:462: warning: 
> non-varargs call of varargs method with inexact argument type for last 
> parameter;
>
> I send in a combined webrev in shortly.
>> Mandy
>>
>> On 8/17/11 3:45 PM, Sebastian Sickelmann wrote:
>>> Hi while searching if i created warnings with an change i have seen 
>>> that there are 2 warnings in rmic that are easy to remove.
>>>
>>> Please review this hg export (no webrev unfortunately, because my 
>>> free-file-host-provider is out of service :-( )
>>>
>>> # HG changeset patch
>>> # User Sebastian
>>> # Date 1313620914 -7200
>>> # Node ID 1dd60cdcf67183afd12323258307f2d78bdd3979
>>> # Parent  07ad163881707c67a0365090ff2192878d266542
>>> Removed some compile-warnings in rmic
>>>
>>> diff -r 07ad16388170 -r 1dd60cdcf671 
>>> src/share/classes/sun/rmi/rmic/Main.java
>>> --- a/src/share/classes/sun/rmi/rmic/Main.java    Wed Aug 17 
>>> 14:18:30 2011 -0700
>>> +++ b/src/share/classes/sun/rmi/rmic/Main.java    Thu Aug 18 
>>> 00:41:54 2011 +0200
>>> @@ -879,6 +879,6 @@
>>>          args[1] = (arg1 != null ? arg1.toString() : "null");
>>>          args[2] = (arg2 != null ? arg2.toString() : "null");
>>>
>>> -        return java.text.MessageFormat.format(format, args);
>>> +        return java.text.MessageFormat.format(format, (Object[]) 
>>> args);
>>>      }
>>>  }
>>> diff -r 07ad16388170 -r 1dd60cdcf671 
>>> src/share/classes/sun/rmi/rmic/newrmic/Resources.java
>>> --- a/src/share/classes/sun/rmi/rmic/newrmic/Resources.java    Wed 
>>> Aug 17 14:18:30 2011 -0700
>>> +++ b/src/share/classes/sun/rmi/rmic/newrmic/Resources.java    Thu 
>>> Aug 18 00:41:54 2011 +0200
>>> @@ -69,7 +69,7 @@
>>>              format = "missing resource key: key = \"" + key + "\", " +
>>>                  "arguments = \"{0}\", \"{1}\", \"{2}\"";
>>>          }
>>> -        return MessageFormat.format(format, args);
>>> +        return MessageFormat.format(format, (Object[]) args);
>>>      }
>>>
>>>      /**
>>
>
My filehoster is working again.
Here is the combined webrev:

http://oss-patches.24.eu/openjdk8/Varargs_Warning/

-- Sebastian



More information about the core-libs-dev mailing list