Summary

colorize variable messup

Metadata

Description

 |06:19| ( zbigniew) #;2> (html-colorize 'scheme "(foo")
 |06:19| ( zbigniew) "<span class=\"paren2\">(<span class=\"default\">foo</span></span></span></span>"
 |06:19| ( zbigniew) #;3> (html-colorize 'scheme "(foo")
 |06:19| ( zbigniew) "<span class=\"paren3\">(<span class=\"default\">foo</span></span></span></span></span></span>"
 |06:19| ( zbigniew) #;4> (html-colorize 'scheme "(foo")
 |06:19| ( zbigniew) "<span class=\"paren4\">(<span class=\"default\">foo</span></span></span></span></span></span></span></span>"
 |06:19| ( zbigniew) ...
 |06:19| ( zbigniew) #;11> (html-colorize 'scheme "(foo")
 |06:19| ( zbigniew) "<span class=\"paren5\">(<span
 class=\"default\">foo</span></span></span></span></span></span></span></span></span></span></span></span></span></span></span></span></span></span></span></span></span></span>"

It looks like this is caused by the big LET around the coloring type definition in define-coloring-type*. It shouldn't be a static variable but one per usage of the coloring type (or reset each when a new string is parsed)

Changes and comments

[2010-09-01 19:25:38 UTC] sjamaan changed status from new to closed

[2010-09-01 19:25:38 UTC] sjamaan set resolution to fixed

[2010-09-01 19:25:38 UTC] sjamaan wrote:

Wrapped it in a lambda :)

[2011-06-01 09:00:41 UTC] felix changed milestone from 4.7.0 to 4.8.0

[2011-06-01 09:00:41 UTC] felix wrote:

Milestone 4.7.0 deleted

[2012-09-24 21:47:48 UTC] felix changed milestone from 4.8.0 to 4.9.0

[2012-09-24 21:47:48 UTC] felix wrote:

Milestone 4.8.0 deleted