Re: [PATCH] merge: use string_list_split() in add_strategies()

From: Junio C Hamano <gitster@pobox.com>
Date: Aug 8, 2016, 20:11 UTC
Message-ID: <xmqqfuqfdnqr.fsf@gitster.mtv.corp.google.com>
In-Reply-To: <xmqqeg5zf8mh.fsf@gitster.mtv.corp.google.com>

Junio C Hamano <gitster@pobox.com> writes:

> If the input comes from the end user, we certainly would want to
> allow "word1 word2\tword3 " as input (i.e. squishing repeated

Any intelligent reader may have guessed already, but before I stupidly told Emacs to refill the paragraph, the above example had two SPs between word1 and word2. Sorry for being sloppy.

Show 10 quoted lines

> delimiters into one without introducing an "empty" element, allowing
> more than one delimiter characters like SP and HT, and ignoring
> leading or trailing runs of delimiter characters).
>
> If the input is generated internally, perhaps we should rethink the
> interface between the function that wants to do the for-each-word
> and its caller; if the caller wants to pass multiple things to the
> callee, it should be able to do so without first having to stuff
> these multiple things into a single string, only to force the callee
> to use this helper to split them out into individual pieces.