httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Torsten Förtsch <torsten.foert...@gmx.net>
Subject Per module LogLevel parsing
Date Fri, 08 Apr 2011 11:17:09 GMT
Hi,

the new per module LogLevel syntax is documented as

  LogLevel modulename:level ...

where level is a certain value out of a predefined set. The string 
"modulename:level" is then parsed in core.c/set_loglevel by:

  level_str = ap_strchr(arg, ':');

This finds the first occurrence of a colon in the string. But shouldn't it 
rather be the last one?

  level_str = ap_strrchr(arg, ':');

The point is, if a module is written in Perl its name can very well contain 
colons.

Is it too late to request this change? Should I create a bug report? (It's 
actually not a bug as such.)

Thanks,
Torsten Förtsch

-- 
Need professional modperl support? Hire me! (http://foertsch.name)

Like fantasy? http://kabatinte.net

Mime
View raw message