Stefan Beller <sbel...@google.com> writes:
> On Fri, Jul 31, 2015 at 6:02 PM, Junio C Hamano <gits...@pobox.com> wrote:
>> Stefan Beller <sbel...@google.com> writes:
>>
>>> -module_list()
>>> +module_list_shell()
>>> {
>>> eval "set $(git rev-parse --sq --prefix "$wt_prefix" -- "$@")"
>>> (
>>> @@ -187,6 +187,29 @@ module_list()
>>> '
>>> }
>>>
>>> +module_list()
>>> +{
>>> + # call both the old and new code
>>> + module_list_shell $@ >/u/git_submodule_module_list_shell
>>> 2>/u/git_submodule_module_list_shell2
>>> + git submodule--helper --module_list $@ >/u/git_submodule_module_list
>>> 2>/u/git_submodule_module_list2
>>
>> You seem to be discarding the double-quote around $@ in both of
>> these two places. Intended?
>
> No, not at all. This was a bit sloppy.
OK.
> This patch was rather showing off how I intend to test the previous patch.
Yeah, I can see what the code is doing, and you already saw that I
didn't disagree with the approach ;). During a reimplementation
exercise, it often is a good idea, if the code structure allows you
to, to run both implementations and compare the results---but it can
go only so far. It obviously is tricky to apply the trick to an
operation that is not idempotent to let two implementations to do it
twice in different ways and make sure they produce the same result.
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
