On Wed, May 17, 2023 at 01:48:46PM +0200, Patrick Steinhardt wrote: > Drop the `NULL` check for `remote_ref` in `update_local_ref()`. The > function only has a single caller, and that caller always passes in a > non-`NULL` value. > > This fixes a false positive in Coverity. Thanks for following up on this. I was thinking of just reposting it, thinking there weren't that many other cleanups to do in fetch.c. But you found quite a few. :) The patch looks obviously good to me. -Peff
