On 2010/07/21 20:16:56, joeneeman wrote:

I can change it to 100, but it's still a bit worrying that a few
measly hash
tables end up taking so much memory (there should only be one per
staff).

Ah, don't worry, I think I see what's going on.  It's creating a new
hash table whenever the method's called:

+ if (!to_boolean (scm_hash_table_p (pure_height_cache)))

This is always true.

I think it should be

if (scm_hash_table_p (pure_height_cache) != SCM_BOOL_T)

http://codereview.appspot.com/1817045/show

_______________________________________________
lilypond-devel mailing list
lilypond-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/lilypond-devel

Reply via email to