Jon Griffiths jon_p_griffiths@yahoo.com writes:
Aside from the tests, what constitutes independent? These are all new functions so there is no possibility of regressions, which is the main reason to split up patches. Ease of understanding/reviewing the patches is the other main reason, but each function is only 5-10 lines.
Since each added function requires the spec file to be updated its much easier to add the functions together (It helps me clean up the diffs from my tree to wine-cvs faster too).
It's OK to send them in the same patch, but then please also send a single Changelog entry for the whole patch, not one per file.