On Mon, Dec 26, 2016 at 5:13 PM, Junio C Hamano <gitster@xxxxxxxxx> wrote: > Stefan Beller <sbeller@xxxxxxxxxx> writes: > >> As the upcoming series will add a lot of functions to the submodule >> header, let's first make the header consistent to the rest of the project >> by adding the extern keyword to functions. > > This may be the right thing to do in the longer term but a patch > like this adds a lot of unnecessary work on the integrator when > there are _other_ topics that are in flight. I'll see how bad the > conflicts are while merging the topic to 'pu'. > > Thanks. Maybe we can roll this patch on its own, so in case the latter patches need rerolls we still have this one and can build on top? Thanks, Stefan