Hi Andrew,
Thanks for spotting that I inverted the logic mistakenly.
Regards.
Wed, 26 Jan 2011 22:15:52 +0200 Pekka Enberg <penb...@cs.helsinki.fi>:
> On Wed, 2011-01-26 at 20:10 +0000, Dr Andrew John Hughes wrote:
> > On 22:41 Wed 26 Jan , Ivan Maidanski wrote:
> > > 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);
> > > }
> > >
> >
> > Yeah I like this version better too. The current one reads rather oddly.
> > However, you seem to have inverted the logic; harness.check(true) should
> > be called when the NPE is given.
> >
> > If an NPE should be thrown for null values, that should be documented in
> > the Classpath patch too.
>
> Here's a new version. I'll update the Javadoc in the patch. Andrew, if
> your OK with the Mauve test case, feel free to commit it to CVS. I'm
> still waiting for my Mauve CVS account.
>
> Pekka
>
> // Tags: JDK1.5
>
> // Copyright (C) 2011 Pekka Enberg
>
> // This file is part of Mauve.
>
> // Mauve is free software; you can redistribute it and/or modify
> // it under the terms of the GNU General Public License as published by
> // the Free Software Foundation; either version 2, or (at your option)
> // any later version.
>
> // Mauve is distributed in the hope that it will be useful,
> // but WITHOUT ANY WARRANTY; without even the implied warranty of
> // MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
> // GNU General Public License for more details.
>
> // You should have received a copy of the GNU General Public License
> // along with Mauve; see the file COPYING. If not, write to
> // the Free Software Foundation, 59 Temple Place - Suite 330,
> // Boston, MA 02111-1307, USA.
>
> package gnu.testlet.java.util.regex.Pattern;
>
> import gnu.testlet.*;
> import java.util.regex.*;
>
> public class quote implements Testlet
> {
> private TestHarness harness;
>
> public void test (TestHarness harness)
> {
> harness.check(Pattern.quote("hello"), "\\Qhello\\E");
> harness.check(Pattern.quote("\\Q"), "\\Q\\Q\\E");
>
> try
> {
> Pattern.quote(null);
> harness.check(false);
> }
> catch (NullPointerException e)
> {
> harness.check(true);
> }
> }
> }
