"Torgil Svensson" <torgil.svensson@xxxxxxxxx> writes: > On 5/21/07, Martin Waitz <tali@xxxxxxxxxxxxxx> wrote: >> Unify naming of plumbing dirlink/gitlink concept: >> >> perl -pi -e 's/dirlink/gitlink/g' -e 's/DIRLNK/GITLINK/g' >> --- > > Does this mean that the link doesn't necessarily has to be represented > as a directory (even though current code does it) ? I do not think the patch has that much deep meaning. Personally I think the patch is similar to renaming "cache" used in the code to "index" --- needless code shuffling that does not buy you anything. As an internal symbol, S_IFDIRLNK makes _much_ more sense than S_IFGITLINK, exactly because it is a mixture of S_IFDIR and S_IFLNK. The only excuse that we _might_ want to consider the patch is that S_IFDIRLNK has not lived that long, compared to the use of word "cache" in our codebase. It's only two weeks old -- so if we were to apply this patch, we would need to do that right away, or it will be never. But I am not _so_ enthused about the renaming in the code. - To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
