{"push_id":1257014691,"size":42,"distinct_size":42,"ref":"refs/heads/master","head":"c79add58fbc7ec00e20d9a57b62d5a21237e96ae","before":"c8894871ecdf553d42bf4b19d2ae45952a0fd732","commits":[{"sha":"1442c174636642b1ba3ed8b760a4232395a708be","author":{"name":"Kirill Bobyrev","email":"d95d4ecac5f95e2917ce773e25c156945dee52cc@gmail.com"},"message":"[clang-tidy] enhance modernize-use-bool-literals to check ternary operator\n\nmodernize-use-bool-literals doesn't checks operands in ternary operator.\n\nFor example:\n\n``` c++\nstatic int Value = 1;\n\nbool foo() {\n  bool Result = Value == 1 ? 1 : 0;\n  return Result;\n}\n\nbool boo() {\n  return Value == 1 ? 1 : 0;\n}\n```\n\nThis issue was reported in bug 28854. The patch fixes it.\n\nReviewers: alexfh, aaron.ballman, Prazek\n\nSubscribers: Prazek, Eugene.Zelenko\n\nDifferential Revision: https://reviews.llvm.org/D23243\n\n\ngit-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@278022 91177308-0d34-0410-b5e6-96231b3b80d8","distinct":true,"url":"https://api.github.com/repos/mpark/clang-tools-extra/commits/1442c174636642b1ba3ed8b760a4232395a708be"},{"sha":"6cc750aa849fc8a59436ca224fcd54c369afe81a","author":{"name":"Kirill Bobyrev","email":"d95d4ecac5f95e2917ce773e25c156945dee52cc@gmail.com"},"message":"[clang-rename] fix bug with initializer lists\n\nClang-rename is currently not able to find a symbol in initializer list. This\npatch fixes described issue.\n\nReviewers: alexfh\n\nDifferential Revision: https://reviews.llvm.org/D23193\n\n\ngit-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@278099 91177308-0d34-0410-b5e6-96231b3b80d8","distinct":true,"url":"https://api.github.com/repos/mpark/clang-tools-extra/commits/6cc750aa849fc8a59436ca224fcd54c369afe81a"},{"sha":"91df466b1d3874bc2486e05c703d05bf91a85550","author":{"name":"Eric Liu","email":"502159d2cfb403927c8f2f98cea0e8056f5456e3@google.com"},"message":"Fix clang-tidy crash when a single fix is applied on multiple files.\n\nSummary:\ntooling::Replacements only holds replacements for a single file, so\nthis patch makes Fix a map from file paths to tooling::Replacements so that it\ncan be applied on multiple files.\n\nReviewers: hokein, alexfh\n\nSubscribers: Prazek, cfe-commits\n\nDifferential Revision: https://reviews.llvm.org/D23257\n\ngit-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@278101 91177308-0d34-0410-b5e6-96231b3b80d8","distinct":true,"url":"https://api.github.com/repos/mpark/clang-tools-extra/commits/91df466b1d3874bc2486e05c703d05bf91a85550"},{"sha":"b656961abb40263a1e3e58bd66aa8e5944618fc0","author":{"name":"Haojian Wu","email":"7e7692cf4d6f5aa42f2756afc956f887e634bcb4@google.com"},"message":"[include-fixer] Support processing multiple files in one run.\n\nSummary:\nPreviously, if we pass multiple files or a file pattern (e.g. /path/to/*.cc) to\ninclude-fixer, include-fixer will apply all replacements to the first argument,\nwhich probably causes crashes.\n\nWith this patch, include-fixer can process multiple files now.\n\nVim and Emacs integration are tested manually.\n\nReviewers: bkramer\n\nSubscribers: cfe-commits\n\nDifferential Revision: https://reviews.llvm.org/D23266\n\ngit-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@278102 91177308-0d34-0410-b5e6-96231b3b80d8","distinct":true,"url":"https://api.github.com/repos/mpark/clang-tools-extra/commits/b656961abb40263a1e3e58bd66aa8e5944618fc0"},{"sha":"436c113c94afc77b9425e892314e0b1a3a0514ce","author":{"name":"Simon Pilgrim","email":"5d82cab8c849387bc8a42783c2e56535c94fed74@redking.me.uk"},"message":"Fix Wdocumentation unknown parameter warning\n\ngit-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@278111 91177308-0d34-0410-b5e6-96231b3b80d8","distinct":true,"url":"https://api.github.com/repos/mpark/clang-tools-extra/commits/436c113c94afc77b9425e892314e0b1a3a0514ce"},{"sha":"de4d175511f2afe5ec7fa951c909095e49e7f694","author":{"name":"Kirill Bobyrev","email":"d95d4ecac5f95e2917ce773e25c156945dee52cc@gmail.com"},"message":"[clang-rename] cleanup: use isWritten\n\nnit: use isWritten and const auto *Initializer in\nNamedDeclFindingASTVisitor::VisitCXXConstructorDecl method.\nTest plan: make -j8 check-clang-tools (passed)\n\nPatch by Alexander Shaposhnikov!\n\nReviewers: omtcyfz\n\nDifferential Revision: https://reviews.llvm.org/D23298\n\n\ngit-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@278112 91177308-0d34-0410-b5e6-96231b3b80d8","distinct":true,"url":"https://api.github.com/repos/mpark/clang-tools-extra/commits/de4d175511f2afe5ec7fa951c909095e49e7f694"},{"sha":"0b3abfc6b8cbbd6349b395a946b82b8874aaf4c9","author":{"name":"Miklos Vajna","email":"46d22af2e51772639f8eb47345a095e41a2510c5@vmiklos.hu"},"message":"clang-rename rename-all: support reading old/newname pairs from a YAML file\n\nThis is handy in case by the time clang-rename is invoked, an external\ntool already genereated a list of oldname -> newname pairs to handle.\n\nReviewers: omtcyfz\n\nDifferential Revision: https://reviews.llvm.org/D23198\n\ngit-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@278145 91177308-0d34-0410-b5e6-96231b3b80d8","distinct":true,"url":"https://api.github.com/repos/mpark/clang-tools-extra/commits/0b3abfc6b8cbbd6349b395a946b82b8874aaf4c9"},{"sha":"be1c7fcb6d08be80929a646dca950ffb8a69484a","author":{"name":"Eugene Zelenko","email":"45390981a69f6cfaa6a29b408dcc553d55acc3a1@gmail.com"},"message":"[Documentation] Fix spelling mistakes in docs/clang-tidy/index.rst.\n\n\ngit-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@278198 91177308-0d34-0410-b5e6-96231b3b80d8","distinct":true,"url":"https://api.github.com/repos/mpark/clang-tools-extra/commits/be1c7fcb6d08be80929a646dca950ffb8a69484a"},{"sha":"eee1248fc7bd05468017b05b6b78ace71b8d5c96","author":{"name":"Miklos Vajna","email":"46d22af2e51772639f8eb47345a095e41a2510c5@vmiklos.hu"},"message":"clang-rename YAML reader: address post-commit comments\n\ngit-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@278201 91177308-0d34-0410-b5e6-96231b3b80d8","distinct":true,"url":"https://api.github.com/repos/mpark/clang-tools-extra/commits/eee1248fc7bd05468017b05b6b78ace71b8d5c96"},{"sha":"ef2ac9ecf4492efb79df67f0f385b7b8a3d33d98","author":{"name":"Kirill Bobyrev","email":"d95d4ecac5f95e2917ce773e25c156945dee52cc@gmail.com"},"message":"[clang-rename] merge tests when possible\n\nThe only difference between some tests is -offset passed to clang-rename. It\nmakes sense to merge them into a single file and add multiple tool invocations.\n\nReviewers: alexfh\n\nDifferential Revision: https://reviews.llvm.org/D23158\n\n\ngit-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@278221 91177308-0d34-0410-b5e6-96231b3b80d8","distinct":true,"url":"https://api.github.com/repos/mpark/clang-tools-extra/commits/ef2ac9ecf4492efb79df67f0f385b7b8a3d33d98"},{"sha":"b876aa29e5f9d8be168ae503c766e9ec6a3fc664","author":{"name":"Kirill Bobyrev","email":"d95d4ecac5f95e2917ce773e25c156945dee52cc@gmail.com"},"message":"[clang-rename] remove bunch of deprecated tests\n\n\ngit-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@278223 91177308-0d34-0410-b5e6-96231b3b80d8","distinct":true,"url":"https://api.github.com/repos/mpark/clang-tools-extra/commits/b876aa29e5f9d8be168ae503c766e9ec6a3fc664"},{"sha":"ac3a60dfedfde26e0d463b7083765aa2c6130547","author":{"name":"Kirill Bobyrev","email":"d95d4ecac5f95e2917ce773e25c156945dee52cc@gmail.com"},"message":"[clang-rename] fix test introduced in r278221\n\n\ngit-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@278225 91177308-0d34-0410-b5e6-96231b3b80d8","distinct":true,"url":"https://api.github.com/repos/mpark/clang-tools-extra/commits/ac3a60dfedfde26e0d463b7083765aa2c6130547"},{"sha":"d8b7d2f625129a5466da4a567d9557f061cc7b2e","author":{"name":"Kirill Bobyrev","email":"d95d4ecac5f95e2917ce773e25c156945dee52cc@gmail.com"},"message":"[clang-tidy] minor improvements in modernise-deprecated-headers check\n\nThis patch introduces a minor list of changes as proposed by Richard Smith in\nthe mailing list.\n\nSee original comments with an impact on the future check state below:\n\n[comments.begin\n\n> +                          {\"complex.h\", \"ccomplex\"},\n\nIt'd be better to convert this one to <complex>, or leave it alone.\n<ccomplex> is an unnecessary wart.\n\n(The contents of C++11's <complex.h> / <ccomplex> / <complex> (all of\nwhich are identical) aren't comparable to C99's <complex.h>, so if\nthis was C++98 code using the C99 header, the code will be broken with\nor without this transformation.)\n\n> +                          {\"iso646.h\", \"ciso646\"},\n\nJust delete #includes of this one. <ciso646> does nothing.\n\n> +              {\"stdalign.h\", \"cstdalign\"},\n> +              {\"stdbool.h\", \"cstdbool\"},\n\nWe should just delete these two includes. These headers do nothing in C++.\n\ncomments.end]\n\nReviewers: alexfh, aaron.ballman\n\nDifferential Revision: https://reviews.llvm.org/D17990\n\n\n\ngit-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@278254 91177308-0d34-0410-b5e6-96231b3b80d8","distinct":true,"url":"https://api.github.com/repos/mpark/clang-tools-extra/commits/d8b7d2f625129a5466da4a567d9557f061cc7b2e"},{"sha":"17cce5095d99a144750024f8191bb95c43d2dd8d","author":{"name":"Eugene Zelenko","email":"45390981a69f6cfaa6a29b408dcc553d55acc3a1@gmail.com"},"message":"[Documentation] Fix grammar mistakes in docs/clang-tidy/index.rst spotted by Alexander Kornienko.\n\n\ngit-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@278255 91177308-0d34-0410-b5e6-96231b3b80d8","distinct":true,"url":"https://api.github.com/repos/mpark/clang-tools-extra/commits/17cce5095d99a144750024f8191bb95c43d2dd8d"},{"sha":"f8b522ccc586bb6d0878a535a2f5f43af97fd9b9","author":{"name":"Kirill Bobyrev","email":"d95d4ecac5f95e2917ce773e25c156945dee52cc@gmail.com"},"message":"[clang-tidy] enhance readability-else-after-return\n\n`readability-else-after-return` only warns about `return` calls, but LLVM Coding\nStandars stat that `throw`, `continue`, `goto`, etc after `return` calls are\nbad, too.\n\nReviwers: alexfh, aaron.ballman\n\nDifferential Revision: https://reviews.llvm.org/D23265\n\n\ngit-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@278257 91177308-0d34-0410-b5e6-96231b3b80d8","distinct":true,"url":"https://api.github.com/repos/mpark/clang-tools-extra/commits/f8b522ccc586bb6d0878a535a2f5f43af97fd9b9"},{"sha":"6181df16e61b016428fd2dd1b6539319086c82ec","author":{"name":"Eugene Zelenko","email":"45390981a69f6cfaa6a29b408dcc553d55acc3a1@gmail.com"},"message":"[Release Notes] Consistency in Clang-tidy entries' style.\n\n\ngit-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@278262 91177308-0d34-0410-b5e6-96231b3b80d8","distinct":true,"url":"https://api.github.com/repos/mpark/clang-tools-extra/commits/6181df16e61b016428fd2dd1b6539319086c82ec"},{"sha":"eef43f8dbf90422dc7ba00c1e16909c7b1caa391","author":{"name":"Eugene Zelenko","email":"45390981a69f6cfaa6a29b408dcc553d55acc3a1@gmail.com"},"message":"[Documentation] Highlighting consistency and spelling mistake fix in Clang-tidy readability-else-after-return description.\n\n\ngit-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@278263 91177308-0d34-0410-b5e6-96231b3b80d8","distinct":true,"url":"https://api.github.com/repos/mpark/clang-tools-extra/commits/eef43f8dbf90422dc7ba00c1e16909c7b1caa391"},{"sha":"a79c5697ef41b71840b68fcf9e5ea12de57fd64d","author":{"name":"Eugene Zelenko","email":"45390981a69f6cfaa6a29b408dcc553d55acc3a1@gmail.com"},"message":"[Documentation] Fix style and grammar mistake in Clang-tidy readability-else-after-return description spotted by Alexander Kornienko.\n\n\ngit-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@278279 91177308-0d34-0410-b5e6-96231b3b80d8","distinct":true,"url":"https://api.github.com/repos/mpark/clang-tools-extra/commits/a79c5697ef41b71840b68fcf9e5ea12de57fd64d"},{"sha":"425f7d595f024ba30df57b3a11d0cfab42a607b0","author":{"name":"Eugene Zelenko","email":"45390981a69f6cfaa6a29b408dcc553d55acc3a1@gmail.com"},"message":"[Documentation] Improve consistency.\n\nMention Clang-rename Emacs integration in release notes.\n\n\ngit-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@278295 91177308-0d34-0410-b5e6-96231b3b80d8","distinct":true,"url":"https://api.github.com/repos/mpark/clang-tools-extra/commits/425f7d595f024ba30df57b3a11d0cfab42a607b0"},{"sha":"baf54f3c8543cd8f7a24862d8a2cad8cde19747e","author":{"name":"Kirill Bobyrev","email":"d95d4ecac5f95e2917ce773e25c156945dee52cc@gmail.com"},"message":"[clang-tidy] fix readability-else-after-return test\n\nAs pointed by Yung Douglas, exceptions in\ntest/clang-tidy/readability-else-after-return.cpp are causing PS4 bots to be\nred, because exceptions aren't enabled by default on PS4 target. This patch is a\nfix.\n\n\ngit-svn-id: https://llvm.org/svn/llvm-project/clang-tools-extra/trunk@278324 91177308-0d34-0410-b5e6-96231b3b80d8","distinct":true,"url":"https://api.github.com/repos/mpark/clang-tools-extra/commits/baf54f3c8543cd8f7a24862d8a2cad8cde19747e"}]}
{"actor":{"display_login":"mpark"}}