Newly attached patch is one I've seen before in some form, but it still needs quite a lot of work before it can be accepted. Breaks coding style, irreversibly forces the new behavior upon users, and adds changes that should be irrelevant (and wrong in the case of the commented assertion; if it's failing the assertion, there's likely a good reason for it).
Newly attached patch is one I've seen before in some form, but it still needs quite a lot of work before it can be accepted. Breaks coding style, irreversibly forces the new behavior upon users, and adds changes that should be irrelevant (and wrong in the case of the commented assertion; if it's failing the assertion, there's likely a good reason for it).