On 1/12/2023 2:35 PM, Victoria Dye wrote:
> Matthew John Cheetham via GitGitGadget wrote:
>> From: Matthew John Cheetham <mjcheetham@xxxxxxxxxxx>
>>
>> Extract functions for setting up listening sockets and keep-alive options
>> from `daemon.c` to new `daemon-utils.{c,h}` files. Remove direct
>> dependencies on global state by inlining the behaviour at the callsites
>> for all libified functions.
>
> Thanks for making this change, the reduced code duplication should make the
> common daemon-related code more maintainable.
>
> For reference, I used
>
> 'git blame -s -b -C -C -C master..<this patch> -- daemon-utils.c'
>
> to help identify which lines in 'daemon-utils.c' were changed from their
> original implementation in 'daemon.c'.
Neat trick! Thanks for sharing. Using --color-moved was giving similar
results, but with a lot more tracking back-and-forth to see what the
differences were.
I agree with your assessment on this patch that the differences are
valid, safe, and desired.
Thanks,
-Stolee
