[clang-tools-extra] r304931 - [clang-tidy] When" -fno-exceptions is used", this warning is better to be suppressed.

2017-06-07 Thread Yan Wang via cfe-commits
Author: yawanng Date: Wed Jun 7 12:41:59 2017 New Revision: 304931 URL: http://llvm.org/viewvc/llvm-project?rev=304931&view=rev Log: [clang-tidy] When" -fno-exceptions is used", this warning is better to be suppressed. Summary: clang-tidy is better not to issues this warning, which checks wher

[clang-tools-extra] r304949 - [clang-tidy] When" -fno-exceptions is used", this warning is better to be suppressed.

2017-06-08 Thread Yan Wang via cfe-commits
Author: yawanng Date: Wed Jun 7 17:39:20 2017 New Revision: 304949 URL: http://llvm.org/viewvc/llvm-project?rev=304949&view=rev Log: [clang-tidy] When" -fno-exceptions is used", this warning is better to be suppressed. Summary: "misc-noexcept-move-constructor" is better not to be issued when

[clang-tools-extra] r306165 - [clang-tidy][Part1] Add a new module Android and three new checks.

2017-06-23 Thread Yan Wang via cfe-commits
Author: yawanng Date: Fri Jun 23 16:37:29 2017 New Revision: 306165 URL: http://llvm.org/viewvc/llvm-project?rev=306165&view=rev Log: [clang-tidy][Part1] Add a new module Android and three new checks. Summary: A common source of security bugs is code that opens a file descriptors without using t

[clang-tools-extra] r306172 - [clang-tidy] doc format fix

2017-06-23 Thread Yan Wang via cfe-commits
Author: yawanng Date: Fri Jun 23 17:12:55 2017 New Revision: 306172 URL: http://llvm.org/viewvc/llvm-project?rev=306172&view=rev Log: [clang-tidy] doc format fix Summary: The url in the doc is wrong. Fix the url. Reviewers: chh Reviewed By: chh Subscribers: srhines, xazax.hun Tags: #clang-too

r309750 - [clang] Change the condition of unnecessary packed warning

2017-08-01 Thread Yan Wang via cfe-commits
Author: yawanng Date: Tue Aug 1 14:41:39 2017 New Revision: 309750 URL: http://llvm.org/viewvc/llvm-project?rev=309750&view=rev Log: [clang] Change the condition of unnecessary packed warning Summary: Change the condition of this unnecessary packed warning. The packed is unnecessary when 1. the

[clang-tools-extra] r310630 - [clang-tidy] Refactor the code and add a close-on-exec check on memfd_create() in Android module.

2017-08-10 Thread Yan Wang via cfe-commits
Author: yawanng Date: Thu Aug 10 10:18:10 2017 New Revision: 310630 URL: http://llvm.org/viewvc/llvm-project?rev=310630&view=rev Log: [clang-tidy] Refactor the code and add a close-on-exec check on memfd_create() in Android module. Summary: 1. Refactor the structure of the code by adding a base

[clang-tools-extra] r310643 - [clang-tidy] Fix a buildbot.

2017-08-10 Thread Yan Wang via cfe-commits
Author: yawanng Date: Thu Aug 10 11:19:40 2017 New Revision: 310643 URL: http://llvm.org/viewvc/llvm-project?rev=310643&view=rev Log: [clang-tidy] Fix a buildbot. Fix format in ReleaseNotes.rst. Modified: clang-tools-extra/trunk/docs/ReleaseNotes.rst Modified: clang-tools-extra/trunk/docs/

Re: [clang-tools-extra] r310630 - [clang-tidy] Refactor the code and add a close-on-exec check on memfd_create() in Android module.

2017-08-10 Thread Yan Wang via cfe-commits
; D:\buildslave\clang-x64-ninja-win7\llvm\tools\clang\include\ > clang/Basic/Diagnostic.h(1117): note: or 'const > clang::DiagnosticBuilder &clang::operator <<(const clang::DiagnosticBuilder > &,int)' > D:\buildslave\clang-x64-ninja-win7\llvm\tools\clang\include\ &

[clang-tools-extra] r310669 - [clang-tidy] Fix for buildbot.

2017-08-10 Thread Yan Wang via cfe-commits
Author: yawanng Date: Thu Aug 10 15:09:22 2017 New Revision: 310669 URL: http://llvm.org/viewvc/llvm-project?rev=310669&view=rev Log: [clang-tidy] Fix for buildbot. Summary: Fix an issue for windows. Differential Revision: https://reviews.llvm.org/D35372 Added: clang-tools-extra/trunk/clang

[clang-tools-extra] r306708 - [clang-tidy][Part2] Add a new module Android and three new checks

2017-06-29 Thread Yan Wang via cfe-commits
Author: yawanng Date: Thu Jun 29 10:40:57 2017 New Revision: 306708 URL: http://llvm.org/viewvc/llvm-project?rev=306708&view=rev Log: [clang-tidy][Part2] Add a new module Android and three new checks Summary: -- creat() should be replaced by open(). [android-creat-usage] Reviewers: chh, alexfh,

[clang-tools-extra] r306709 - [clang-tidy][Part3] Add a new module Android and three new checks.

2017-06-29 Thread Yan Wang via cfe-commits
Author: yawanng Date: Thu Jun 29 10:42:23 2017 New Revision: 306709 URL: http://llvm.org/viewvc/llvm-project?rev=306709&view=rev Log: [clang-tidy][Part3] Add a new module Android and three new checks. Summary: -- fopen() should include "e" in their mode string. [android-fopen-mode] Reviewers: c

[clang-tools-extra] r306719 - [clang-tidy] Add docs to toctree

2017-06-29 Thread Yan Wang via cfe-commits
Author: yawanng Date: Thu Jun 29 11:44:28 2017 New Revision: 306719 URL: http://llvm.org/viewvc/llvm-project?rev=306719&view=rev Log: [clang-tidy] Add docs to toctree Summary: Add .rst files to toctree. Fix buildbot error. Reviewers: chh Reviewed By: chh Subscribers: srhines, JDevlieghere, xaz

[clang-tools-extra] r306728 - [clang-tidy] Rename android-file-open-flag and fix a bug

2017-06-29 Thread Yan Wang via cfe-commits
Author: yawanng Date: Thu Jun 29 12:13:29 2017 New Revision: 306728 URL: http://llvm.org/viewvc/llvm-project?rev=306728&view=rev Log: [clang-tidy] Rename android-file-open-flag and fix a bug Summary: 1. Rename android-file-open-flag to android-cloexec-open. 2. Handle a case when the function is p

[clang-tools-extra] r307818 - [clang-tidy] Add a new Android check "android-cloexec-socket"

2017-07-12 Thread Yan Wang via cfe-commits
Author: yawanng Date: Wed Jul 12 10:43:36 2017 New Revision: 307818 URL: http://llvm.org/viewvc/llvm-project?rev=307818&view=rev Log: [clang-tidy] Add a new Android check "android-cloexec-socket" Summary: socket() is better to include SOCK_CLOEXEC in its type argument to avoid the file descripto

[clang-tools-extra] r303001 - [clang-tidy] TwineLocalCheck: add param # checking

2017-05-14 Thread Yan Wang via cfe-commits
Author: yawanng Date: Sat May 13 23:14:59 2017 New Revision: 303001 URL: http://llvm.org/viewvc/llvm-project?rev=303001&view=rev Log: [clang-tidy] TwineLocalCheck: add param # checking Summary: The statement **getArg** tries to get the first one without checking, which may cause segmentation fau