Looks good to me.

It's nice to see this simplification, especially the simplification in the builders to eliminate the computation of the isLastContent parameter.

And yeah, it's sort of funny that blockListLast was was same style as blockList.

-- Jon

On 6/5/19 1:57 AM, Hannes Wallnöfer wrote:
Please review:

JBS: https://bugs.openjdk.java.net/browse/JDK-8221709
Webrev: http://cr.openjdk.java.net/~hannesw/8221709/webrev.00/

The patch touches a lot of files, but it’s repetitive and conceptually simple.

Funnily enough there is no need for the :last-child selector as blockListLast 
used the same style as plain blockList.

Hannes

Reply via email to