]> git.wincent.com - wikitext.git/commitdiff
Remove comment from _Wikitext_append_entity_from_utf32_char
authorWincent Colaiuta <win@wincent.com>
Tue, 12 May 2009 21:32:00 +0000 (23:32 +0200)
committerWincent Colaiuta <win@wincent.com>
Tue, 12 May 2009 21:32:00 +0000 (23:32 +0200)
There is no point in special casing entities like "quot", "amp"
and such in this function because neither of the two call sites
would ever pass in such a code point:

- the _Wikitext_append_sanitized_link_target already explicitly
  handles all these cases, either emitting the entities manually
  or raising an exception.

- the DEFAULT case won't ever have to process those characters
  because they would have already been tokenized otherwise and
  handled in the QUOT, AMP, LESS and GREATER cases.

Signed-off-by: Wincent Colaiuta <win@wincent.com>
ext/parser.c

index f4a41b5a69112090a1a6667ea209ed3246d3180b..8bd124ca1a8c0fd734a7a46c3487db75037e08ef 100644 (file)
@@ -700,7 +700,6 @@ uint32_t _Wikitext_utf8_to_utf32(char *src, char *end, long *width_out)
 
 void _Wikitext_append_entity_from_utf32_char(char *output, uint32_t character)
 {
-    // TODO: consider special casing some entities (ie. quot, amp, lt, gt etc)?
     char hex_string[8]  = { '&', '#', 'x', 0, 0, 0, 0, ';' };
     char scratch        = (character & 0xf000) >> 12;
     hex_string[3]       = (scratch <= 9 ? scratch + 48 : scratch + 87);