Thanks Lance, Daniel!
-Joe
On 2/14/2017 10:53 AM, Lance Andersen wrote:
Looks good… ship it.
On Feb 14, 2017, at 1:41 PM, Daniel Fuchs > wrote:
On 14/02/17 17:21, huizhe wang wrote:
Thanks!
Here's an updated webrev:
Looks good… ship it.
> On Feb 14, 2017, at 1:41 PM, Daniel Fuchs wrote:
>
> On 14/02/17 17:21, huizhe wang wrote:
>> Thanks!
>>
>> Here's an updated webrev:
>> http://cr.openjdk.java.net/~joehw/jdk9/8169450/webrev/
>
> +1
>
> -- daniel
>
>>
>> -Joe
>>
>> On
On 14/02/17 17:21, huizhe wang wrote:
Thanks!
Here's an updated webrev:
http://cr.openjdk.java.net/~joehw/jdk9/8169450/webrev/
+1
-- daniel
-Joe
On 2/14/2017 4:07 AM, Lance Andersen wrote:
Looks good overall Joe. I would agree that I would clean up the minor
comment alignment issues.
Thanks!
Here's an updated webrev:
http://cr.openjdk.java.net/~joehw/jdk9/8169450/webrev/
-Joe
On 2/14/2017 4:07 AM, Lance Andersen wrote:
Looks good overall Joe. I would agree that I would clean up the minor
comment alignment issues.
Best
Lance
On Feb 13, 2017, at 9:27 PM, huizhe wang
On 2/14/2017 3:16 AM, Daniel Fuchs wrote:
Looks good Joe.
I wonder about this though (which may be an issue for
another time):
102 [25] Eq ::= S? '=' S?
Do we support space (new line?) before and after the '=' sign?
Yes, we do. I've added a couple of test cases to cover
Thanks Christoph. Will fix those comments.
-Joe
On 2/13/2017 11:56 PM, Langer, Christoph wrote:
Hi Joe,
this looks right to me, quite straightforward.
I just have some formatting nits for the test:
1. Line 107 (XMLInputFactory xif=): space missing before '='
2. Method comments on test
Looks good overall Joe. I would agree that I would clean up the minor comment
alignment issues.
Best
Lance
> On Feb 13, 2017, at 9:27 PM, huizhe wang wrote:
>
> A quick fix for the error parsing xml declaration. This is one of the three
> outstanding issues in the xml
Looks good Joe.
I wonder about this though (which may be an issue for
another time):
102 [25] Eq ::= S? '=' S?
Do we support space (new line?) before and after the '=' sign?
best regards,
-- daniel
On 14/02/17 02:27, huizhe wang wrote:
A quick fix for the error parsing xml
Hi Joe,
this looks right to me, quite straightforward.
I just have some formatting nits for the test:
1. Line 107 (XMLInputFactory xif=): space missing before '='
2. Method comments on test methods would look nicer if formatted like:
/*
* line 1
* line 2
* ...
*/
... I want to say that the '*'