From: Karsten D. <k.d...@tu...> - 2001-03-02 17:01:26
|
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 |