[prev in list] [next in list] [prev in thread] [next in thread]
List: openjdk-2d-dev
Subject: Re: [OpenJDK 2D-Dev] 8017109: Cleanup overrides warning in src/solaris/classes/sun/print/AttributeCl
From: Dan Xu <dan.xu () oracle ! com>
Date: 2013-08-15 20:44:25
Message-ID: 520D3DA9.3010404 () oracle ! com
[Download RAW message or body]
Thankyou, Jennifer.
I will push it today.
-Dan
On 08/15/2013 12:55 PM, Jennifer Godinez wrote:
> Hi Dan,
>
> Fix looks good.
>
> Jennifer
>
> On 8/12/2013 1:48 PM, Jennifer Godinez wrote:
>> Hi Dan,
>>
>> I will run some tests and let you know the results.
>> Thank you.
>>
>> Jennifer
>> On 8/8/2013 1:14 PM, Dan Xu wrote:
>>> Hi Phil and Jennifer,
>>>
>>> Thanks for your suggestions!
>>>
>>> I have updated the changes at
>>> http://cr.openjdk.java.net/~dxu/8017109/webrev.00/. Please help
>>> review and sponsor it.
>>>
>>> -Dan
>>>
>>>
>>> On 07/29/2013 11:54 AM, Dan Xu wrote:
>>>> Hi Phil and Jennifer,
>>>>
>>>> As Kurchi just left, I am following up on this warning fix. I
>>>> wonder whether you have tested out her fix. Are you able to sponsor
>>>> the changeset? Thanks!
>>>>
>>>> -Dan
>>>>
>>>>
>>>> On 06/21/2013 09:35 AM, Kurchi Subhra Hazra wrote:
>>>>> On 6/21/13 9:33 AM, Phil Race wrote:
>>>>>> On 6/21/2013 1:56 AM, Chris Hegarty wrote:
>>>>>>>
>>>>>>> Yes of course. What I was really getting at was the pre testing
>>>>>>> that a sponsor can do before pushing. Someone in the 2d area
>>>>>>> could run their usual build and test cycle before pushing.
>>>>>>
>>>>>> I had asked Jennifer to identify and recommend which tests should
>>>>>> be run.
>>>>>> But it may need someone with the right environment as well.
>>>>>>
>>>>>> -phil.
>>>>>
>>>>> Thanks. I think it were best if someone would test and sponsor
>>>>> this changeset for me.
>>>>>
>>>>> - Kurchi
>>>>>
>>>>
>>>
>>
>
[Attachment #3 (text/html)]
<html>
<head>
<meta content="text/html; charset=UTF-8" http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
<font size="-1">Thank<font size="-1"> you, Jennifer.<br>
<br>
<font size="-1">I will push it today.<br>
<br>
<font size="-1">-Dan<br>
<br>
<br>
</font></font></font></font>
<div class="moz-cite-prefix">On 08/15/2013 12:55 PM, Jennifer
Godinez wrote:<br>
</div>
<blockquote cite="mid:520D3222.5020009@oracle.com" type="cite">Hi
Dan,
<br>
<br>
Fix looks good.
<br>
<br>
Jennifer
<br>
<br>
On 8/12/2013 1:48 PM, Jennifer Godinez wrote:
<br>
<blockquote type="cite">Hi Dan,
<br>
<br>
I will run some tests and let you know the results.
<br>
Thank you.
<br>
<br>
Jennifer
<br>
On 8/8/2013 1:14 PM, Dan Xu wrote:
<br>
<blockquote type="cite">Hi Phil and Jennifer,
<br>
<br>
Thanks for your suggestions!
<br>
<br>
I have updated the changes at
<a class="moz-txt-link-freetext" \
href="http://cr.openjdk.java.net/~dxu/8017109/webrev.00/">http://cr.openjdk.java.net/~dxu/8017109/webrev.00/</a>. \
Please help review and sponsor it.
<br>
<br>
-Dan
<br>
<br>
<br>
On 07/29/2013 11:54 AM, Dan Xu wrote:
<br>
<blockquote type="cite">Hi Phil and Jennifer,
<br>
<br>
As Kurchi just left, I am following up on this warning fix.
I wonder whether you have tested out her fix. Are you able
to sponsor the changeset? Thanks!
<br>
<br>
-Dan
<br>
<br>
<br>
On 06/21/2013 09:35 AM, Kurchi Subhra Hazra wrote:
<br>
<blockquote type="cite">On 6/21/13 9:33 AM, Phil Race wrote:
<br>
<blockquote type="cite">On 6/21/2013 1:56 AM, Chris
Hegarty wrote:
<br>
<blockquote type="cite">
<br>
Yes of course. What I was really getting at was the
pre testing that a sponsor can do before pushing.
Someone in the 2d area could run their usual build and
test cycle before pushing.
<br>
</blockquote>
<br>
I had asked Jennifer to identify and recommend which
tests should be run.
<br>
But it may need someone with the right environment as
well.
<br>
<br>
-phil.
<br>
</blockquote>
<br>
Thanks. I think it were best if someone would test and
sponsor this changeset for me.
<br>
<br>
- Kurchi
<br>
<br>
</blockquote>
<br>
</blockquote>
<br>
</blockquote>
<br>
</blockquote>
<br>
</blockquote>
<br>
</body>
</html>
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic