Re: Optimizations for pure-height approximations. (issue1817045)

[n . puttock]

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