On Wed, 19 Dec 2018 at 01:18, brian m. carlson <sandals@xxxxxxxxxxxxxxxxxxxx> wrote: > I think this change is fine, because we initialize the value in > the_repository elsewhere, and if there's no repository, this should > never have a value other than the default anyway. Thanks, it feels good that this patch matches how you think about the `hash_algo` field. > I looked at the other patches in the series and thought they looked sane > as well. Thanks for a review, I appreciate it. Martin
