Johannes Schindelin <Johannes.Schindelin@xxxxxx> wrote: > On Wed, 14 Jan 2009, Felipe Contreras wrote: > > > "S_IFREG | mode" probably is only required for 0644 and 0755. > > Why should we want to have that patch? IOW what does it fix, and what > might it break? It cleans up the code to make it more readable. It makes no sense to be doing S_IFREG | S_IFLINK, which happens when the input is for a symlink. It doesn't break anything to do that | operation, but it also looks damn odd when reading the function. -- Shawn. -- 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
