Hi Claes,

Looks good, Thanks for the test.

p.s.  I probably would have used a data provider for the CS cases but its not significant.

On 5/19/20 6:37 AM, Claes Redestad wrote:
Hi Roger,

sure, added Emptiness.java with a few sanity tests.

/Claes

On 2020-05-19 00:29, Roger Riggs wrote:
Hi Claes,

Though it does not look like it could be any simpler, there should probably be a test. Checking consistency with the existing implementations of CharSequence.length()
and a custom subtype.

Thanks, Roger


On 5/18/20 4:48 PM, Claes Redestad wrote:
Hi,

let's add an isEmpty default method to CharSequence!

Webrev: http://cr.openjdk.java.net/~redestad/8215401/open.00/
RFE:    https://bugs.openjdk.java.net/browse/JDK-8215401
CSR:    https://bugs.openjdk.java.net/browse/JDK-8215402

Testing: tier1-3

/Claes


Reply via email to