[JAXP] RFR: 8153082: Update XSTL compiler to generate classes that invoke addReads
Daniel Fuchs
daniel.fuchs at oracle.com
Mon Jul 25 17:40:31 UTC 2016
On 25/07/16 18:38, Langer, Christoph wrote:
> Hi,
>
> looking good. I believe you could also remove the comment sections after the headers like:
> /*
> * $Id: Constants.java,v 1.7 2006/06/19 19:49:04 spericas Exp $
> */
>
> But no need for new webrev for that, of course :-)
Thanks Christoph! I was unsure whether to keep it or remove it.
I will remove it before pushing.
best regards,
-- daniel
>
> Best regards
> Christoph
>
>> -----Original Message-----
>> From: Daniel Fuchs [mailto:daniel.fuchs at oracle.com]
>> Sent: Montag, 25. Juli 2016 19:22
>> To: Langer, Christoph <christoph.langer at sap.com>; Joe Wang
>> <huizhe.wang at oracle.com>; Alan Bateman <Alan.Bateman at oracle.com>
>> Cc: core-libs-dev <core-libs-dev at openjdk.java.net>
>> Subject: Re: [JAXP] RFR: 8153082: Update XSTL compiler to generate classes
>> that invoke addReads
>>
>> Hi,
>>
>> Here is the later version of the fix:
>>
>> - Header files fixed
>> - Bytecode 1.1 compatible
>>
>> http://cr.openjdk.java.net/~dfuchs/webrev_8153082/webrev.02/
>>
>> cheers,
>>
>> -- daniel
>>
>> On 25/07/16 16:16, Langer, Christoph wrote:
>>> Hi Daniel,
>>>
>>> yes, I just recognized the header is different everywhere. It is a good idea to
>> consult Joe on this - he was also giving me hints on how to do it correctly when I
>> was touching JAXP.
>>>
>>> Best regards
>>> Christoph
>>>
>>>> -----Original Message-----
>>>> From: Daniel Fuchs [mailto:daniel.fuchs at oracle.com]
>>>> Sent: Montag, 25. Juli 2016 16:43
>>>> To: Langer, Christoph <christoph.langer at sap.com>
>>>> Cc: Joe Wang <huizhe.wang at oracle.com>; core-libs-dev <core-libs-
>>>> dev at openjdk.java.net>
>>>> Subject: Re: [JAXP] RFR: 8153082: Update XSTL compiler to generate classes
>>>> that invoke addReads
>>>>
>>>> Hi Christoph,
>>>>
>>>> On 22/07/16 20:23, Langer, Christoph wrote:
>>>>> Hi Daniel,
>>>>>
>>>>> looks good to me.
>>>>>
>>>>> Maybe you'll want to take the chance to update the apache headers in the
>>>> xalan files?
>>>>>
>>>>
>>>> Thanks for your review!
>>>>
>>>> I'm only an occasional wanderer in JAXP land - which files do you see
>>>> have an outdated header?
>>>>
>>>> I'll try to ping Joe Wang to get the correct headers.
>>>>
>>>> best regards,
>>>>
>>>> -- daniel
>
More information about the core-libs-dev
mailing list