X-Git-Url: https://git.donarmstrong.com/?a=blobdiff_plain;f=lily%2Fnested-property.cc;h=edb7986a51e7e5127980eb13fd84134db18805a6;hb=29d4a0cf02e2890ded85acf48865f64541d7c7d4;hp=6ce5234fc33178df530d653cde34d5d9d7278fad;hpb=edb75f07f428d68eb2f694445eef60606b7cf64a;p=lilypond.git diff --git a/lily/nested-property.cc b/lily/nested-property.cc index 6ce5234fc3..edb7986a51 100644 --- a/lily/nested-property.cc +++ b/lily/nested-property.cc @@ -125,7 +125,8 @@ nested_property_alist (SCM alist, SCM prop_path, SCM value) SCM rest = scm_cdr (prop_path); if (scm_is_pair (rest)) { - SCM where = assq_tail (key, alist); + SCM where = assoc_tail (key, alist); + if (scm_is_false (where)) return scm_acons (key, nested_create_alist (rest, value), alist); return scm_acons (key, nested_property_alist (scm_cdar (where), @@ -170,7 +171,12 @@ set_nested_property (Grob *me, SCM big_to_small, SCM value) // This converts an alist with nested overrides in it to a proper // alist. The number of nested overrides is known in advance, // everything up to the last nested override is copied, the tail is -// shared +// shared. +// +// The first nalist index has to be a symbol since the conversion +// relies on eq? comparisons, uses some special non-symbol values for +// special purposes, and does validity checking indexed by symbols. +// Subindexing can be done with equal?-comparable indexes, however. SCM nalist_to_alist (SCM nalist, int nested) @@ -205,7 +211,7 @@ nalist_to_alist (SCM nalist, int nested) scm_set_cdr_x (pair, scm_cons (elt, scm_cdr (pair))); continue; } - + assert (scm_is_symbol (key)); // plain override: apply any known corresponding partials SCM pair = assq_pop_x (key, &partials); if (scm_is_true (pair))