Make load* methods of DifferenceEngine idempotent
authorGergő Tisza <tgr.huwiki@gmail.com>
Wed, 11 Jul 2018 08:48:49 +0000 (10:48 +0200)
committerGergő Tisza <tgr.huwiki@gmail.com>
Wed, 18 Jul 2018 06:03:30 +0000 (08:03 +0200)
commit04f16a204b89c037a2a555615ca0cd9a66837c99
tree8bd089b336d12bd67c30e9d4d055354d5d7b4c71
parente5d71f2552893f15da9c0be7dd1fa210adb7be23
Make load* methods of DifferenceEngine idempotent

These methods returned a boolean indicating whether loading the
data was successful, but then always returned true on subsequent
calls. Fix that.

This changes public methods but there's no usage in Gerrit (some
of them are called but the return value is ignored), no use case
for a caller to care, and the previous behavior has been
undocumented and unreliable, so there is no deprecation period.

Change-Id: I3998aeea66972f33274e05fa5a74d6ce7fdc56b6
includes/diff/DifferenceEngine.php