[geeklog-devel] Regex functions in Geeklog
geeklog at langfamily.ca
Mon Mar 8 12:40:52 EST 2004
I would personally hold off any new changes and I think Dirk will also feel
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
----- 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.
> 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
> > 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:
> >>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.
> >>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.
> >>geeklog-devel mailing list
> >>geeklog-devel at lists.geeklog.net
> geeklog-devel mailing list
> geeklog-devel at lists.geeklog.net
More information about the geeklog-devel