[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Omaha.pm] (?:pattern)

To: "Perl Mongers of Omaha, Nebraska USA" <omaha-pm@pm.org>
Subject: Re: [Omaha.pm] (?:pattern)
From: Andy Lester <andy@petdance.com>
Date: Thu, 21 Oct 2010 09:37:56 -0500
Delivered-to: mailman-omaha-pm@mailman.pm.dev
Delivered-to: omaha-pm@pm.org
In-reply-to: <C7D12528-0A0D-49F7-83CC-9E20139B5F61@mutationgrid.com>
List-archive: <http://mail.pm.org/pipermail/omaha-pm>
List-help: <mailto:omaha-pm-request@pm.org?subject=help>
List-id: "Perl Mongers of Omaha, Nebraska USA" <omaha-pm.pm.org>
List-post: <mailto:omaha-pm@pm.org>
List-subscribe: <http://mail.pm.org/mailman/listinfo/omaha-pm>, <mailto:omaha-pm-request@pm.org?subject=subscribe>
List-unsubscribe: <http://mail.pm.org/mailman/options/omaha-pm>, <mailto:omaha-pm-request@pm.org?subject=unsubscribe>
References: <C7D12528-0A0D-49F7-83CC-9E20139B5F61@mutationgrid.com>
Reply-to: "Perl Mongers of Omaha, Nebraska USA" <omaha-pm@pm.org>

On Oct 21, 2010, at 9:32 AM, Jay Hannah wrote:

> Since the author isn't using the backreferences anyway, why not just
> 
>   /^INS_(CHC|CRS|PLV)$/
> 
> Isn't that the same thing and easier to read?
> 
> I should probably write some little programs to test it, but I'm feeling lazy.  :)

There's presumably a speed-up because it's not dumping the match into $1.  You'd have to benchmark to find out just how much it's saving.

The bigger change is getting rid of the outer grouping parens which are not even needed for grouping.

xoox,
Andy

--
Andy Lester => andy@petdance.com => www.techworklove.com => AIM:petdance

Follow-Ups:
- Re: [Omaha.pm] (?:pattern)
  - From: Jay Hannah <jhannah@mutationgrid.com>

References:
- [Omaha.pm] (?:pattern)
  - From: Jay Hannah <jhannah@mutationgrid.com>

Prev by Date: [Omaha.pm] (?:pattern)
Next by Date: Re: [Omaha.pm] (?:pattern)
Previous by thread: [Omaha.pm] (?:pattern)
Next by thread: Re: [Omaha.pm] (?:pattern)
Index(es):
- Date
- Thread