# Re: \[PATCH 2/2\] sequencer.c: plug mem leak in git\_sequencer\_config

- From: Junio C Hamano \<hidden\>
- Date: 2018-06-04 04:56:04 
- List: git
- Page: https://ratatoskr.run/git/2018/06/7769856

```text
Jeff King [ref] writes:

> With that strategy, we'd have to have a big initialize_defaults()
> function. Which actually might not be _too_ bad since we now have
> common-main.c, but:
>
>   - it sucks to keep the default values far away from the declarations
>
>   - it does carry a runtime cost. Not a huge one, but it sucks to pay it
>     on every program startup, even if we're not using those variables.

True, and s/even if/even though most of the time/ the situation is
even worse X-<.
```