> > +static int record_ieot(void) > > +{ > > + int val; > > + > > Initialize stack val to zero to ensure proper default. I don't think that is needed here, as we only use `val` when we first write to it via git_config_get_bool. Did you spot this via code review and thought of defensive programming or is there a tool that has a false positive here? > > > + if (!git_config_get_bool("index.recordoffsettable", &val)) > > + return val; > > + return 0; > > +}