Thread (1 message) 1 message, 1 author, 2016-06-15

Re: [PATCH 1/3] make-static: master

From: Junio C Hamano <hidden>
Date: 2016-06-15 22:52:02

Ramkumar Ramachandra [off-list ref] writes:
Junio C Hamano writes:
quoted
Many symbols that are exported to the global scope do not have to be.

Signed-off-by: Junio C Hamano <redacted>
---
   To be applied on top of 3793ac5 (RelNotes/1.7.7: minor fixes, 2011-09-07)
[...]
Awesome!  I've seen many similar "make-static" patches come up on the
list, but turned down due to code churn issues.  I'm happy to finally
see it being merged.
Not so fast.

We just got a new series from Brandon Casey that adds a new caller to one
of the functions that were unused outside a file. Seeing that this happens
even inside a relative calm of pre-release feature and code freeze,
realize why we call such a change a "churn" and try to avoid it, and
imagine how much damage it would have caused if more topics were actively
being updated.

You should learn to tame your enthusiasm ;-)
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help