On Sat, Aug 5, 2017 at 10:25 PM, Christian Couder
<[email protected]> wrote:
> On Sat, Aug 5, 2017 at 12:28 PM, Prathamesh Chavan <[email protected]> wrote:
>> On Tue, Aug 1, 2017 at 4:57 AM, Christian Couder
>> <[email protected]> wrote:
>>> On Mon, Jul 31, 2017 at 10:56 PM, Prathamesh Chavan <[email protected]>
>>> wrote:
>>>
>> We can avoid it to behave same for "" and NULL, by checking if diff_cmd
>> is "cmd_diff_files", since its value is set NULL by this case.
>>
>> ret = compute_summary_module_list(strcmp(diff_cmd, "diff-files") ?
>> NULL: sb.buf, &info);
>> strbuf_release(&sb);
>
> It looks error prone, more fagile and less efficient to me.
I think by using enum { DIFF_INDEX, DIFF_FILES },
we can avoid using strcmp() here, and make it more efficient
not only here but also in other parts of the code, since
all such steps involving strcmp could be removed.
Thanks,
Prathamesh Chavan
