I suppose, as far as "if test" expression is used everywhere in the code it's better to use IT but not Bash's square brackets (see also Git coding guideline https://github.com/git/git/blob/master/Documentation/CodingGuidelines#L84 which I prefer to follow after I sent some patches to Git codebase)