Samuel Kerschbaumer wrote:
Hmm I think the purpose of this code was to support 'u' for the milliseconds in the date format, since date() supports u only since PHP 5.2.2.
Yes, the code is there to support milliseconds in the date format.
Since Log4PHP now focus only on PHP5 support these lines are unnecessary and should be removed.

After testing changing the code to use the documented "u" modifier directly in the date() function I realize that this won't work.

See: http://bugs.php.net/bug.php?id=42921 for details.

We can only remove the regular expression here if we can to skip support for a "u" modifier in our date format.

Knut

Reply via email to