Hello everyone! A few modifications were done to the patch.Below is the list of updated parts of the patch with comments about what has changed, comparing to the previous version of the patch.
The patched JDK builds fine and all the tests pass well. WEBREV-000: http://cr.openjdk.java.net/~igerasim/8198358/04/000/webrev/ WEBREV-001: http://cr.openjdk.java.net/~igerasim/8198358/04/001/webrev/ WEBREV-002: http://cr.openjdk.java.net/~igerasim/8198358/04/002/webrev/ WEBREV-003: http://cr.openjdk.java.net/~igerasim/8198358/04/003/webrev/ - Rebased to reflect recent changes to socketListen() WEBREV-004: http://cr.openjdk.java.net/~igerasim/8198358/04/004/webrev/ WEBREV-005: http://cr.openjdk.java.net/~igerasim/8198358/04/005/webrev/ WEBREV-006: http://cr.openjdk.java.net/~igerasim/8198358/04/006/webrev/ WEBREV-007: http://cr.openjdk.java.net/~igerasim/8198358/04/007/webrev/ WEBREV-008: http://cr.openjdk.java.net/~igerasim/8198358/04/008/webrev/- The check the line 400 was changed to test if (newfd < 0) and then if (newfd == -2). I don't think the later is quite accurate (cannot find in the documentation that accept can return -2), but it is how it was done in TwoStacks originally, so no regression is expected.
WEBREV-009: http://cr.openjdk.java.net/~igerasim/8198358/04/009/webrev/ - Added missing 'return -1;' at the line 188. WEBREV-010: http://cr.openjdk.java.net/~igerasim/8198358/04/010/webrev/ WEBREV-011: http://cr.openjdk.java.net/~igerasim/8198358/04/011/webrev/ WEBREV-012: http://cr.openjdk.java.net/~igerasim/8198358/04/012/webrev/ WEBREV-013: http://cr.openjdk.java.net/~igerasim/8198358/04/013/webrev/ WEBREV-014: http://cr.openjdk.java.net/~igerasim/8198358/04/014/webrev/ With kind regards, Ivan
