]> git.tue.mpg.de Git - paraslash.git/commit
mood: Fix compute_score().
authorAndre Noll <maan@tuebingen.mpg.de>
Sat, 22 Oct 2022 17:59:52 +0000 (19:59 +0200)
committerAndre Noll <maan@tuebingen.mpg.de>
Mon, 24 Oct 2022 14:54:48 +0000 (16:54 +0200)
commitbbf102f76c471be5e6b5d6c76e010c4c49f031de
treeca2de93bde65d972f26f09cd7f45cc1a3e09f110
parent9c31cf0d4bcf1abd9767c77d68d90c9007b4f114
mood: Fix compute_score().

This fixes a bug which was introduced 10 months ago in merge
commit 88bf6848d1c (Merge branch 'refs/heads/t/rm_v1_moods'). This
merge conflicted in mood.c, and the conflicting hunks that touched
compute_score() were resolved incorrectly. Put simply, we kept the
old code and disregarded the correction factors that were introduced
by the other side of the merge. As a result, at mood load time the
correction factors were initialized correctly but not taken into
account for computing the score of the admissible files.

The fact that the bug went unnoticed for so long can only mean that
the correction factors don't make much of a difference in practice.
However, the bug was found because one particular mood behaved
unexpectedly, likely because its admissible files consisted of a
bunch of very new files among many very old ones.
mood.c