[PATCH] D13134: [analyzer] Add keyboard shortcuts to report.html
Devin Coughlin via cfe-commits
cfe-commits at lists.llvm.org
Mon Oct 5 11:10:30 PDT 2015
dcoughlin added a comment.
Thanks for the update. Unfortunately, we can't accept the mousetrap code. Any code contributed to clang needs to be under the UIUC license. The original copyright owner would need to contribute the code under that license.
I think it would be fine to revert to your previous version -- but with a FIXME stating that this doesn't work on non-QWERTY keyboards.
Other than that, this LGTM.
Repository:
rL LLVM
http://reviews.llvm.org/D13134
More information about the cfe-commits
mailing list