I think unless it's obviously insane, it doesn't matter too much as long as everyone does the same thing (or everyone does something that can be munged into the same thing by the VCS). I once found a piece of code that variously used 2, 3, 4, 5, 6, and 8 spaces for indenting. That was fun to understand.