[geeklog-devtalk] geeklog-devel digest, Vol 1 #287 - 3 msgs

[geeklog-devel-request at lists.geeklog.net]

Send geeklog-devel mailing list submissions to
	geeklog-devel at lists.geeklog.net

To subscribe or unsubscribe via the World Wide Web, visit
	http://lists.geeklog.net/listinfo/geeklog-devel
or, via email, send a message with subject or body 'help' to
	geeklog-devel-request at lists.geeklog.net

You can reach the person managing the list at
	geeklog-devel-admin at lists.geeklog.net

When replying, please edit your Subject line so it is more specific
than "Re: Contents of geeklog-devel digest..."


Today's Topics:

   1. Regex functions in Geeklog (Tony Bibbs)
   2. Re: Regex functions in Geeklog (Tony Bibbs)
   3. Re: Regex functions in Geeklog (Blaine Lang)

--__--__--

Message: 1
Date: Mon, 08 Mar 2004 08:05:02 -0600
From: Tony Bibbs <tony at tonybibbs.com>
To: Geeklog <geeklog-devel at lists.geeklog.net>
Subject: [geeklog-devel] Regex functions in Geeklog
Reply-To: geeklog-devel at lists.geeklog.net

An anonymous coward wrote:

<snip>
I was looking through the Geeklog PHP code and saw it uses functions
like eregi and ereg_replace in places, when it should be using
preg_match or str_replace, which are much faster.  I've been coding in
PHP since '99 and I've benchmarked the preg and string functions against
the ereg functions and the ereg functions are always slower and should
never be used.  You should use strstr or stristr instead of ereg or
preg_match if you don't need a regular expression.  And you should use
str_replace instead of preg_replace or ereg_replace if you don't need a
regular expression.

Please consider making these changes in future versions of the code.
</snip>

I've forward this to the list as an FYI.  This guy isn't saying 
something we haven't heard before.  BTW, for anybody watching this list, 
anonymously sent emails annoy me (and other developers I'm sure).  Use 
geeklog-devtalk or geeklog-users for stuff like this.

--Tony

--__--__--

Message: 2
Date: Mon, 08 Mar 2004 11:33:18 -0600
From: Tony Bibbs <tony at tonybibbs.com>
To: Geeklog <geeklog-devel at lists.geeklog.net>, pyrox_pro at hotmail.com
Subject: Re: [geeklog-devel] Regex functions in Geeklog
Reply-To: geeklog-devel at lists.geeklog.net

Yep, an RC3 is out now.  I doubt you get the fixes in for the final 
1.3.9 release but we can try (Dirk, you have an ETA yet?).  Just send a 
note to geeklog-devel once you think you got it.

--Tony

Justin Carlson wrote:

> Is it ereg_replace or eregi_replace?

> I will assume that since this guy has been coding since `99 that it is just

> ereg_replace().

> 

> I could possibly make the adjustments.

> You guys have a RC getting ready to be put out right?

> 

> ----- Original Message ----- 

> From: "Tony Bibbs" <tony at tonybibbs.com>

> To: "Geeklog" <geeklog-devel at lists.geeklog.net>

> Sent: Monday, March 08, 2004 8:05 AM

> Subject: [geeklog-devel] Regex functions in Geeklog

> 

> 

> 

>>An anonymous coward wrote:

>>

>><snip>

>>I was looking through the Geeklog PHP code and saw it uses functions

>>like eregi and ereg_replace in places, when it should be using

>>preg_match or str_replace, which are much faster.  I've been coding in

>>PHP since '99 and I've benchmarked the preg and string functions against

>>the ereg functions and the ereg functions are always slower and should

>>never be used.  You should use strstr or stristr instead of ereg or

>>preg_match if you don't need a regular expression.  And you should use

>>str_replace instead of preg_replace or ereg_replace if you don't need a

>>regular expression.

>>

>>Please consider making these changes in future versions of the code.

>></snip>

>>

>>I've forward this to the list as an FYI.  This guy isn't saying

>>something we haven't heard before.  BTW, for anybody watching this list,

>>anonymously sent emails annoy me (and other developers I'm sure).  Use

>>geeklog-devtalk or geeklog-users for stuff like this.

>>

>>--Tony

>>_______________________________________________

>>geeklog-devel mailing list

>>geeklog-devel at lists.geeklog.net

>>http://lists.geeklog.net/listinfo/geeklog-devel

>>


--__--__--

Message: 3
From: "Blaine Lang" <geeklog at langfamily.ca>
To: <geeklog-devel at lists.geeklog.net>
Subject: Re: [geeklog-devel] Regex functions in Geeklog
Date: Mon, 8 Mar 2004 12:40:52 -0500
Reply-To: geeklog-devel at lists.geeklog.net

I would personally hold off any new changes and I think Dirk will also feel
the same.
Only bug fixes to 1.3.9 changes should be done at this time.

We can discuss making the regex function changes for 1.3.10

Blaine
----- Original Message ----- 
From: "Tony Bibbs" <tony at tonybibbs.com>
To: "Geeklog" <geeklog-devel at lists.geeklog.net>; <pyrox_pro at hotmail.com>
Sent: Monday, March 08, 2004 12:33 PM
Subject: Re: [geeklog-devel] Regex functions in Geeklog



> Yep, an RC3 is out now.  I doubt you get the fixes in for the final

> 1.3.9 release but we can try (Dirk, you have an ETA yet?).  Just send a

> note to geeklog-devel once you think you got it.

>

> --Tony

>

> Justin Carlson wrote:

> > Is it ereg_replace or eregi_replace?

> > I will assume that since this guy has been coding since `99 that it is

just

> > ereg_replace().

> >

> > I could possibly make the adjustments.

> > You guys have a RC getting ready to be put out right?

> >

> > ----- Original Message ----- 

> > From: "Tony Bibbs" <tony at tonybibbs.com>

> > To: "Geeklog" <geeklog-devel at lists.geeklog.net>

> > Sent: Monday, March 08, 2004 8:05 AM

> > Subject: [geeklog-devel] Regex functions in Geeklog

> >

> >

> >

> >>An anonymous coward wrote:

> >>

> >><snip>

> >>I was looking through the Geeklog PHP code and saw it uses functions

> >>like eregi and ereg_replace in places, when it should be using

> >>preg_match or str_replace, which are much faster.  I've been coding in

> >>PHP since '99 and I've benchmarked the preg and string functions against

> >>the ereg functions and the ereg functions are always slower and should

> >>never be used.  You should use strstr or stristr instead of ereg or

> >>preg_match if you don't need a regular expression.  And you should use

> >>str_replace instead of preg_replace or ereg_replace if you don't need a

> >>regular expression.

> >>

> >>Please consider making these changes in future versions of the code.

> >></snip>

> >>

> >>I've forward this to the list as an FYI.  This guy isn't saying

> >>something we haven't heard before.  BTW, for anybody watching this list,

> >>anonymously sent emails annoy me (and other developers I'm sure).  Use

> >>geeklog-devtalk or geeklog-users for stuff like this.

> >>

> >>--Tony

> >>_______________________________________________

> >>geeklog-devel mailing list

> >>geeklog-devel at lists.geeklog.net

> >>http://lists.geeklog.net/listinfo/geeklog-devel

> >>

> _______________________________________________

> geeklog-devel mailing list

> geeklog-devel at lists.geeklog.net

> http://lists.geeklog.net/listinfo/geeklog-devel




--__--__--

_______________________________________________
geeklog-devel mailing list
geeklog-devel at lists.geeklog.net
http://lists.geeklog.net/listinfo/geeklog-devel


End of geeklog-devel Digest