2011/1/26 Ivan Maidanski <iv...@mail.ru>:
> Hi,
>
> It's ok but:
>
> 1. I'd better rewrote check for null (IMHO looks better):
>
> try {
> Pattern.quote(null);
> harness.check(true);
> } catch (NullPointerException e) {
> harness.check(false);
> }OK but other way around as we expect NPE to be thrown. > 2. Add a test for a string which already contains \Q Sure, I'll do that.
