Thread (1 message) 1 message, 1 author, 2018-06-04

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

Jeff King [off-list 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-<.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help