Ok, sorry. I didn't want to interfere with his suggestions, actually I'm fine with it.

It's just that mixing new features and reformatting of existing code makes it harder to isolate the added features. For new code, by all means follow any advice given by @lstrojny . But for existing code, we should rather have one large commit that only fixes style and whitespace without touching functionality at all.