- **Needs**: -->
- **Patch**: new --> review
- **Type**: -->
- **Comment**:
Passes make, make check and a full make doc.
(although I get all those cell counts back again, I assume this would be
expected in this one case - ie. I am checking against a baseline that used .ly
and now is using .txt)
---
** [issues:#5721] lilypond-book: Rewrite processing of snippets**
**Status:** Started
**Created:** Sat Feb 01, 2020 11:55 AM UTC by Jonas Hahnfeld
**Last Updated:** Sun Feb 02, 2020 03:28 PM UTC
**Owner:** Jonas Hahnfeld
lilypond-book: Rewrite processing of snippets
The previous version relied on a stable hash() method, both for the
ordering in a set() and for the list's checksum. This probably worked
with Python 2 and up to Python 3.2, but later versions use a random
seed for invocations of hash(). This ensures different hash values for
subsequent invocations to make malicious attacks more difficult.
The new code uses hashlib.md5() and .hexdigest() instead which returns
a deterministic result across runs. It also sorts the snippets' names
which leads to more stable profiling results for 'make check'.
This change also tries to improve performance by writing snippets with
the same basename only once. Additionally it solves potential problems
if the build directory has the string '.ly' in its path.
http://codereview.appspot.com/555220043
---
Sent from sourceforge.net because testlilyissues-a...@lists.sourceforge.net is
subscribed to https://sourceforge.net/p/testlilyissues/issues/
To unsubscribe from further messages, a project admin can change settings at
https://sourceforge.net/p/testlilyissues/admin/issues/options. Or, if this is
a mailing list, you can unsubscribe from the mailing list.
_______________________________________________
Testlilyissues-auto mailing list
testlilyissues-a...@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/testlilyissues-auto