Comment 4 for bug 443325

Revision history for this message
Dustin Kirkland  (kirkland) wrote :

I'm looking at the diff for revision 913, and it's huge.

There's quite a bit of whitespace and non-functional changes in there.

This makes it extremely difficult for me to review and cherrypick without risking serious regressions.

Chris, in the future, could you try to fix one bug per commit, and separate non-functional or whitespace-only changes to a separate commit? That would really help your downstreams in processing your commits.

Thanks a lot.

:-Dustin