Coverage decreased (-1.2%) to 94.542% when pulling a96eff4 on bouweandela:yapf-diff into 489fbd9 on google:master.

This looks good. It needs a bit of cleanup. The top-level comment in yapf-diff shouldn't be about clang-diff, and it needs the copyright header. Should also be mentioned in CHANGES.

Hold on. Ignore what I said about the header stuff. Copyright stuff is more complex. :-) I'll need to check about whether we can do this.

Thanks! Do you have any news about the legal question already? The alternative would be to implement something from scratch, but because the script is fairly simple the difference between the two options might be pretty small in the end.

Thanks! Do you have any news about the legal question already?

I've gotten the info on how to proceed. We need to ensure that the licensing for this tool is retained. So please add it to a subdirectory called third_party. The structure will look something like this:

yapf/
  |
  ` third_party/
    |
    ` yapf_diff/
      |
      - LICENSE
      |
      - ... (whatever files belong here)

The LICENSE file should have a copy of the full license text under which clang-format was released. (Assuming this is clang's general license.) Because we're modifying the name, please add a comment at the top of yapf-diff that it's a modified copy of clang-format.