diff options
| author | Christian Kamm <christian.d.kamm@nokia.com> | 2011-09-30 11:15:15 +0200 |
|---|---|---|
| committer | Christian Kamm <christian.d.kamm@nokia.com> | 2011-10-10 14:40:10 +0200 |
| commit | 1b0dc76564200f768010e9486c6895433027c607 (patch) | |
| tree | 0a4511d19cea41c31b7f3089368e9c0e38530658 /src/libs/qmljs/qmljscheck.cpp | |
| parent | be9d25ec64788f5c826f0f073bf086bc02ab9c31 (diff) | |
| download | qt-creator-1b0dc76564200f768010e9486c6895433027c607.tar.gz | |
QmlJS checks: Add warning about 'eval'.
Migrated from QtChecker.
Change-Id: I4b3e8993c7b9f697497d2199b24bf49379dbc1b4
Reviewed-on: http://codereview.qt-project.org/5860
Sanity-Review: Qt Sanity Bot <qt_sanity_bot@ovi.com>
Reviewed-by: Fawzi Mohamed <fawzi.mohamed@nokia.com>
Diffstat (limited to 'src/libs/qmljs/qmljscheck.cpp')
| -rw-r--r-- | src/libs/qmljs/qmljscheck.cpp | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/src/libs/qmljs/qmljscheck.cpp b/src/libs/qmljs/qmljscheck.cpp index 3c56387252..da9c5cf78d 100644 --- a/src/libs/qmljs/qmljscheck.cpp +++ b/src/libs/qmljs/qmljscheck.cpp @@ -1160,6 +1160,8 @@ bool Check::visit(CallExpression *ast) if (!name.isEmpty() && name.at(0).isUpper()) { addMessage(WarnExpectedNewWithUppercaseFunction, location); } + if (cast<IdentifierExpression *>(ast->base) && name == QLatin1String("eval")) + addMessage(WarnEval, location); return true; } |