jprt build failures on windows
Seán Coffey
sean.coffey at oracle.com
Tue Oct 11 04:15:53 PDT 2011
New changes look fine Chris.
regards,
Sean.
On 11/10/2011 11:51, Chris Hegarty wrote:
> There was a cut'n'paste error in the original change for CR 7098719.
> super.create(stream) should be reinstated.
>
> After the above, now the (closed) regression test amended for CR
> 7098719 fails. The reason is that TwoStacksPlainSocketImpl overrides
> the close() method, from AbstractPlainSocketImpl, and does not
> decrements the ResourceManagers count. This problem existed before CR
> 7098719, it is just now being seen as the amended test catches this
> situation. The solution is to simply invoke
> ResourceManager.afterUdpClose() for non-stream sockets, just as is
> done in AbstractPlainSocketImpl.close().
>
> Webrev for CR 7099488:
> http://cr.openjdk.java.net/~chegar/7099488/webrev.00/webrev/
>
> Thanks,
> -Chris.
>
> On 10/11/11 10:36 AM, Chris Hegarty wrote:
>> Thanks Max,
>>
>> I filed the following for this:
>> CR 7099488: "TwoStacksPlainSocketImpl should invoke
>> super.create(stream), typo in fix for 7098719"
>>
>> Too many changes in too many repos!
>>
>> -Chris.
>>
>> On 10/11/11 06:32 AM, Weijun Wang wrote:
>>> http://prt-web.us.oracle.com//archive/2011/10/2011-10-11-051159.ww155710.jdk//JobStatus.txt
>>>
>>>
>>>
>>>
>>> where T is a type-variable:
>>> T extends Object declared in class Constructor
>>> ..\..\..\src\windows\classes\java\net\TwoStacksPlainSocketImpl.java:85:
>>> error: no suitable method found for create()
>>> super.create();
>>> ^
>>> method AbstractPlainSocketImpl.create(boolean) is not applicable
>>> (actual and formal argument lists differ in length)
>>>
>>> Looks like something changed in a recent change
>>>
>>> http://hg.openjdk.java.net/jdk8/tl/jdk/rev/1e89a13d9d8f
>>>
>>> - super.create(stream);
>>> + try {
>>> + super.create();
>>>
>>> -Max
More information about the net-dev
mailing list