Improvements to PHP highlighter
|Reported by:||Owned by:||Christian Boos|
Two minor issues with the PHP highlighter (source:trunk/trac/mimeview/php.py):
- The output of
php -snincludes _two_ junk lines at the bottom, although the second one doesn't include the trailing
\n. The last line is a
</code>that definitely should be stripped. That second-to-last line is a
</span>which was messing up validation but apparently not caught because it didn't cause any weirdness.
php -snhas this nasty tendency to close tags on the line past where it'd seem logical to close them. This causes a lot of unnecessary html code, and in the interest of sending less data over the wire, this can easily be worked around. I've modified the PHPDeuglifier to scooch the close tags up to where they belong. I'm sad to see this gotcha leave, as it's what let me find #3262.
I've tested both of these points for consistency against PHP 4.3.10-16 on Debian, 4.4.1 on Windows, and 5.1.4-p10 on Gentoo.
Attaching a patch in a sec.