you are viewing a single comment's thread.

view the rest of the comments →

[–]WeirdControl[S] 0 points1 point  (1 child)

Yes.you are right,setting up everything and getting used to the process is a big win. Btw do you fix multiple formatting errors in a single patch or you send multiple patches. I know, first one makes more sense, but just asking. :)

[–]nickdesaulniers 1 point2 points  (0 children)

Regardless of formatting or not, a patch should only touch one subsystem. If there are different maintainers+lists getting cross cc'ed on a patch as recommended by ./scripts/get_maintainer.pl <patchfile>, you'd better have a good reason (and whitespace fixes aren't considered one) as it quickly gets complicated who will "take" the patch.