Thread (4 messages) flat view 4 messages, 3 authors, 2016-09-04

Re: git add -p—splitting hunks, limit is too large

From: Johannes Schindelin <hidden>
Date: 2016-09-04 08:10:46

Hi Peff,

On Fri, 2 Sep 2016, Jeff King wrote:
The good news (or maybe the bad) is that "add -p" is implemented
entirely in Perl. :)
Yeah, you would definitely not call this "good news" if you were in my
shoes.

There is no question that it has grown way too unwieldy and that we (once
again, as with so many other scripts) missed the boat to convert it to C.

Scripting is nice for prototyping. But it comes at a high
portability/performance cost if taken too far. And we took it way too far.

Just look at all of those 1667 lines of git-add--interactive and weep. So
many things reimplemented in Perl instead of reusing functions in
libgit.a (or introducing them, making them usable from other parts of
Git). Wasted time is what I see there.

The worst part is that it completely violates our original "Unix
philosophy" of implementing the business logic in C and combinig it using
light-weight scripting.

And of course now the script is *so large* that nobody wants to undertake
the task of porting it to C.

Ciao,
Dscho
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help