To all, On 10/22/2015 03:36 PM, Joseph Myers wrote:
I think this patch is small enough, and the pieces insufficiently self-contained, that splitting it up rather than posting as one patch just makes it harder to understand. My strong preference is that the same patch that introduces a feature should also add the testcases for that feature, for example - they should not be split out (that's not even a split by reviewer, testcases are critical to reviewing functionality patches).
Okay, I'll rewrite the posting and submit it with a single patch as a response to the initial posting. I'll also re-write the comments as they have been pointed out by Nathan as being too terse. My apologies for wasting people's time. Jim