https://codereview.appspot.com/102760044/diff/90001/scm/document-backend.scm
File scm/document-backend.scm (right):

https://codereview.appspot.com/102760044/diff/90001/scm/document-backend.scm#newcode37
scm/document-backend.scm:37: (set! all-grob-descriptions
Doing an assoc-set! here is inefficient inside of the loop.  Instead of
(for-each
   (lambda (grob-description)
      ...
      (set! all-grob-descriptions (assoc-set! ...

one should use
(set! all-grob-descriptions
  (map!
     (lambda (grob-description)
        ;replacement for grob description)
     all-grob-descriptions))

Changing the list when doing the iteration is not well-defined anyway.

https://codereview.appspot.com/102760044/

_______________________________________________
lilypond-devel mailing list
[email protected]
https://lists.gnu.org/mailman/listinfo/lilypond-devel

Reply via email to