Although I think this feature can indeed be useful for some, I think it must be a setting. Because when using e.g. PHP, no matter the code is in an HTML comment it will be processed, so this change gives the false impression there is no PHP script there:
<!-- some commented out <?phpecho$var?> snippet -->
IMO, this feature should be configurable through a property.
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
PHP might be introduced with <? if its short_open_tag option is enabled, but it is considered legacy feature (because it clashes with XML's <?xml) so yeah, it's probably safe to consider that PHP starts with <?php.
However if it's here only not to highlight commented <?xml ... ?>, which you pointed out yourself being not valid, I don't see much point in this "fix" and would rather see it revered.
This said I don't mind much what's done, as far as preprocessors like PHP inside XML comments can still be highlighted.
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
The is a processing instruction and it appears they cannot occur inside a comment. http://www.w3.org/TR/REC-xml/#sec-pi
Committed fix as [b811a2].
Related
Commit: [b811a2]
Although I think this feature can indeed be useful for some, I think it must be a setting. Because when using e.g. PHP, no matter the code is in an HTML comment it will be processed, so this change gives the false impression there is no PHP script there:
IMO, this feature should be configurable through a property.
Proposed patch
SciTE patch for it to forward the property from the previous patch
If PHP is always introduced with <?php then that could be treated specially without the need for another property.
PHP might be introduced with
<?if itsshort_open_tagoption is enabled, but it is considered legacy feature (because it clashes with XML's<?xml) so yeah, it's probably safe to consider that PHP starts with<?php.However if it's here only not to highlight commented
<?xml ... ?>, which you pointed out yourself being not valid, I don't see much point in this "fix" and would rather see it revered.This said I don't mind much what's done, as far as preprocessors like PHP inside XML comments can still be highlighted.
Commit [657c09] modifies previous change to not affect <?php or <? .
Related
Commit: [657c09]