[gollem] directory path patch

Chuck Hagenbuch chuck at horde.org
Tue Feb 4 13:15:48 PST 2003


Quoting Gaudenz Steinlin <gaudenz at soziologie.ch>:

> If you go to the directory "/" in gollem and then click on any directory
> below it, gollem will show you "//directory". Here is a little patch to
> eliminate the additional slash.

I committed something similar. From docs/CODING_STANDARDS:

---------------------------
[15] Regular Expression Use
===========================

Always use the preg_* functions if possible instead of ereg_* (and
preg_split() instead of split()); they are included in PHP by default
and much more efficient and much faster than ereg_*.

NEVER use a regular expression to match or replace a static string.
explode() (in place of split()), str_replace(), strstr(), or strtr()
do the job much more efficiently.

-chuck

--
Charles Hagenbuch, <chuck at horde.org>
must ... find ... acorns ... *thud*


More information about the gollem mailing list