Thread (99 messages) flat view 99 messages, 7 authors, 2016-12-15

Re: [PATCH v9 4/5] http: create function to get curl allowed protocols

From: Brandon Williams <hidden>
Date: 2016-12-14 18:01:03

On 12/14, Jeff King wrote:
On Tue, Dec 13, 2016 at 05:40:36PM -0800, Brandon Williams wrote:
quoted
Move the creation of an allowed protocols whitelist to a helper
function.
This is "what" but not "why". You can figure it out if you see the next
patch, but it's often nice to make a brief mention, like:

  This will be useful when we need to compute the set of allowed
  protocols differently for normal and redirect cases.
Commit message writing is hard (at least for me :).  I'll update the
message to indicate the why.

-- 
Brandon Williams
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help