Hi Jim, Here is the updated webrev using your approach: http://cr.openjdk.java.net/~lbourges/marlin/marlin-8144938.4/
It is more close to 2nd webrev as it does not set subpathStarted to false in SEG_CLOSE case. Laurent 2016-03-17 1:49 GMT+01:00 Jim Graham <[email protected]>: > Hi Laurent, > > I'm not sure why you added the pathClosed variable back. Before your fix > we used to rely on all of the downstream consumers correctly implementing > the case of CLOSE followed by a non-MOVE command so specifically enforcing > a MOVE here should be unnecessary and just complicates the code...? > > ...jim >
