Hi,

On Wed, 2009-04-01 at 20:47 -0700, Justin Martin wrote:
> The update I'd like to propose is to the Zend Highlighter for PHP,
> specifically related to the highlight_file and highlight_string
> functions, as well as the php -s command.

I don't see the need for a rush (or in other words: 5.3 can happily live
without it imo) and for the general consideration there's still my patch
from 2005 using a <ol> to have line numbers and adding id's so lines can
directly be accessed by an URL.

http://schlueters.de/~johannes/php/zend_highlight_20050312_1.diff

About ini settings: We should avoid adding them as much as possible,
making this an ini setting gives you trouble in case the admin plays
with them for some reason and you don't overwrite them in the script, on
the other hand it's the only way to make it work with "php -s" and the
apache sapi's application/x-httpd-php-source.

So in summary: Let's think carefully about this before committing the
first idea which comes to mind.

johannes



-- 
PHP Internals - PHP Runtime Development Mailing List
To unsubscribe, visit: http://www.php.net/unsub.php

Reply via email to