> Earlier patches in this series moved a couple of conditions from the > recursive name_rev() function into its caller name_ref(), for no other > reason than to make eliminating the recursion a bit easier to follow. > > Since the previous patch name_rev() is not recursive anymore, so let's > move all those conditions back into name_rev(). I don't really see the need for this code movement, to be honest. There is no big difference in doing the checks in one place or the other, and if you ask me, it might even be better to do it in the caller of name_rev(), and leave name_rev() to handle only the naming.
