On Fri, Mar 02, 2001 at 04:24:13PM +0100, Karsten Dambekalns wrote:
> Hi!
>=20
> I was about to try to update from CVS, but it seems that the machine is d=
own
> (browse CVS isn't working either). Anyone else experiencing this?
Answering myself... it is up and running again. And the code is changing
steadily!
A few remarks: On some places nl2br was changed to ereg_replace to get
XHTML-style <br /> tags. Is this communicated to someone over at the PHP
developer's side? Would be nice if this would become the default output of
PHP's nl2br function.
But then: why ereg_replace()? preg_replace() is faster than that, and in th=
is
simple case str_replace() woulb be even faster and no less appropriate! And
if you change that, then why leave nl2br() in just a few lines below in teh
code?
And some of the new code was absolutely "de-indented" (does everyone
understand what I want to say with that?)
No offense :-)
Regards,
Karsten
--=20
Why do we have to hide from the police, daddy?
Because we use emacs, son. They use vi.
-----------------------------
mailto:k.d...@tu... w=B3: http://www.k-fish.de/
gpg: http://www.k-fish.de/mykeys.gpg
|
