Re: [RFC PATCH v4 1/3] Add support for nested aliases

Tim Schumacher <timschumi@xxxxxx> · Sun, 16 Sep 2018 09:46:58 +0200

On 08.09.18 15:28, Duy Nguyen wrote:
On Sat, Sep 8, 2018 at 12:44 AM Tim Schumacher <timschumi@xxxxxx> wrote:
+               /*
+                * It could be an alias -- this works around the insanity
                  * of overriding "git log" with "git show" by having
                  * alias.log = show
                  */

I think this comment block is about the next two lines you just
deleted. So delete it to instead of fixing style.

I think that comment is talking about the code that is handing the alias,
so it still would be valid.
The check might have peen placed in between to keep it logically grouped.

-               if (done_alias)
-                       break;
                 if (!handle_alias(argcp, argv))
                         break;
                 done_alias = 1;
         }