On 02/03/2018, Jonathan Nieder <jrnieder@xxxxxxxxx> wrote: > Is this reproducible for you? Yes. It seems to occur consistently, given the same input. > Do you have more details about how I can reproduce it? Unfortunately, the particular git repo I encountered it on is private, otherwise I would point you to it. I haven't attempted to create a MWE. Am I correct that doing so is now not needed, in the light of Jeff King's email below? > What arch are you on? What perl version do you use? Can you report > this using "reportbug git"? All perfectly decent questions. For a modicum of security/privacy, I would prefer to avoid answering them unless necessary. Am I right in thinking that these answers are no longer needed, in the light of Jeff King's email below? On 02/03/2018, Jeff King <peff@xxxxxxxx> wrote: > 3. Your invocation in particular is a problem because it uses > --word-diff, which will not have a one-to-one line correspondence > with the bare diff. > > add--interactive handles pretty-printing by running the diff > command twice: once with no special options, and once with > "--color" and piped through the diffFilter. It assumes that the two > match each other line for line, so it shows you the "DISPLAY" > variant, but then ultimately applies the "TEXT" variant. > > And that last one is the cause of the errors you see: > >> Use of uninitialized value $_ in print at >> /usr/lib/git-core/git-add--interactive line 1371, <STDIN> line 74. > > The "DISPLAY" run for your case generates fewer lines than the "TEXT" > run, and we complain on trying to show those extra lines.
