Short paths are much harder to debug for users, in the 99.99% of cases where they are not needed. If they come up in error messages, warnings, or whatever, they are far harder to read.
I would suggest to use GetShortPathName() only when absolutely necessary, that is when the unicode file path cannot be encoded in "mbcs" format (which is the language-specific encoding used by 8-bit Windows APIs).