[PATCH resend2] 8145278: Fix memory leak in splitPathList

[serguei.spitsyn at oracle.com]

On 4/18/16 13:57, Daniel D. Daugherty wrote:
> On 4/18/16 2:55 PM, serguei.spitsyn at oracle.com wrote:
>> Dan,
>>
>> Thank you for the notice!
>
> No problem. I just happened to notice that my changeset notification
> tracker didn't report that one going by... and we were watching stuff
> closed on 2016.04.13 and 2016.04.14 due to the upcoming JDK9-hs-rt
> merge with JDK9-hs.

It is nice it got caught.
I could catch the absence of the integration comment in the bug report.


>
>
>> Will check and re-submit the job.
>
> Please remember to rebase to JDK9-hs. JDK9-hs-rt is now closed
> and read-only...

Yes of course, I remember it. :)


Thanks,
Sertguei

>
> Dan
>
>
>>
>> Thanks,
>> Serguei
>>
>>
>> On 4/18/16 13:40, Daniel D. Daugherty wrote:
>>> Serguei,
>>>
>>> Even though the JPRT "succeeded", the push itself failed:
>>>
>>> hg --config ui.merge=false push --repository 
>>> /net/scaaa637.us.oracle.com/export/
>>> archives/data/jprt/archive/2016/04/2016-04-13-173931.sspitsyn.8145278/sourcetree 
>>>
>>> /jdk --noninteractive --time
>>> pushing to ssh://sspitsyn@hg.openjdk.java.net/jdk9/hs-rt/jdk
>>> searching for changes
>>> remote: adding changesets
>>> remote: adding manifests
>>> remote: adding file changes
>>> remote: added 1 changesets with 1 changes to 1 files
>>> remote: [jcheck d46eaf84b346 2015-12-01 13:15:54 -0800]
>>> remote:
>>> remote: > Changeset: 14045:93826bcfbaf1
>>> remote: > Author:    Alex Henrie <alexhenrie24 at gmail.com>
>>> remote: > Date:      2015-11-17 23:10
>>> remote: >
>>> remote: > 8145278: Fix memory leak in splitPathList
>>> remote: > Reviewed-by: sspitsyn, dsamersoff, dcubed
>>> remote:
>>> remote: Invalid changeset author: Alex Henrie <alexhenrie24 at gmail.com>
>>> remote:
>>> remote: transaction abort!
>>> remote: rollback completed
>>> remote: abort: pretxnchangegroup.0.jcheck hook failed
>>> time: real 5.530 secs (user 0.350+0.000 sys 0.050+0.000)
>>> -> 1
>>>
>>>
>>> You must have jcheck disabled on the machine that you used to
>>> commit the changeset. The server side rejected the changeset
>>> because of the malformed "Author:" field.
>>>
>>> Dan
>>>
>>>
>>> On 4/18/16 2:23 PM, serguei.spitsyn at oracle.com wrote:
>>>> The fix was pushed to the jdk9/hs-rt.
>>>> It will take some time (a week or two) to be promoted to the jdk9/dev.
>>>>
>>>> Thanks,
>>>> Serguei
>>>>
>>>>
>>>> On 4/18/16 13:17, Alex Henrie wrote:
>>>>> 2016-04-14 2:35 GMT-06:00 serguei.spitsyn at oracle.com
>>>>> <serguei.spitsyn at oracle.com>:
>>>>>> Your fix has been pushed.
>>>>> Thanks! Could you send me a link to the repository that it was pushed
>>>>> to? I'm not seeing it in jdk9/dev/jdk.
>>>>>
>>>>> -Alex
>>>>
>>>>
>>>
>>
>