Duy Nguyen writes:
> Off topic. This reminds me of an old patch about apply and ita [1] but
> that one is not the same here
... Yeah, and re-reading that one, I think that sort-of makes
sense. I am hesitant to take it out of context, though. I wonder
how it would interact with that broken mode
On Fri, Oct 7, 2016 at 2:15 AM, Junio C Hamano wrote:
> Duy Nguyen writes:
>
>> On Tue, Oct 4, 2016 at 11:15 PM, Junio C Hamano wrote:
>>> Duy Nguyen writes:
>>>
We don't use it internally _yet_. I need to go through all the
external diff code and see --shift-ita should be there. The
Duy Nguyen writes:
> On Tue, Oct 4, 2016 at 11:15 PM, Junio C Hamano wrote:
>> Duy Nguyen writes:
>>
>>> We don't use it internally _yet_. I need to go through all the
>>> external diff code and see --shift-ita should be there. The end goal
>>> is still changing the default behavior and getting
On Tue, Oct 4, 2016 at 11:15 PM, Junio C Hamano wrote:
> Duy Nguyen writes:
>
>> We don't use it internally _yet_. I need to go through all the
>> external diff code and see --shift-ita should be there. The end goal
>> is still changing the default behavior and getting rid of --shift-ita,
>
> I d
Duy Nguyen writes:
> We don't use it internally _yet_. I need to go through all the
> external diff code and see --shift-ita should be there. The end goal
> is still changing the default behavior and getting rid of --shift-ita,
I do not agree with that endgame, and quite honestly I do not want
t
On Thu, Sep 29, 2016 at 2:28 AM, Junio C Hamano wrote:
> After reading the three patches through, however, I do not think we
> use the command line option anywhere. I'm inclined to say that we
> shouldn't add it at all. Or at least do so in a separate follow-up
> patch "now we have an internal m
Junio C Hamano writes:
> As I already said, --shift-ita is not quite descriptive and I think
> it should be renamed to something else, but I kept that in the
> following attempt to rewrite:
> ...
Please do not use that verbatim; it was full of typo and grammo.
> After reading the three patches
Nguyễn Thái Ngọc Duy writes:
> The original commit d95d728aba06a34394d15466045cbdabdada58a2 was
> reverted in commit 78cc1a540ba127b13f2f3fd531777b57f3a9cd46 because we
> were (and still are) not ready for a new world order. A lot more
> investigation must be done to see what is impacted. See th
The original commit d95d728aba06a34394d15466045cbdabdada58a2 was
reverted in commit 78cc1a540ba127b13f2f3fd531777b57f3a9cd46 because we
were (and still are) not ready for a new world order. A lot more
investigation must be done to see what is impacted. See the 78cc1a5 for
details.
This patch takes
9 matches
Mail list logo