On 12/11/12 2:01 PM, Alan Bateman wrote:
On 10/12/2012 17:12, Daniel Fuchs wrote:
[...]
So here is a new - and hopefully simpler webrev:

<http://cr.openjdk.java.net/~dfuchs/JDK-7169894/javax.xml.parsers/webrev.04/>


best regards,

-- daniel
This looks fine to me, just a stray <p></p> at L64.

Right - I originally did it on purpose because the doc was
looking weird without it. But now I regenerated the doc and
I don't think it looks better with it - so I will remove
the stray <p></p>. I'll just refresh webrev.04 after that
(I assume there's no need for a webrev.05 - but I'd like
to keep a public copy of the 'final' patch).

-- daniel


-Alan

Reply via email to