qt/qtdeclarative.git/tools/qmllint/checkidentifiers.h, branch dev Qt Declarative (Quick 2) qmllint: Remove checkidentifiers 2021-08-05T09:44:41+00:00 Maximilian Goldstein max.goldstein@qt.io 2021-08-04T12:57:29+00:00 8600202e7621164e30a28511ab92e926cba57d49 Since all of the warnings have been moved out of checkidentifiers we can now simply remove it. Change-Id: Ie59e306b69ca397c1e191a0fe379bc084f98cf73 Reviewed-by: Fabian Kosmale <fabian.kosmale@qt.io> 
Since all of the warnings have been moved out of checkidentifiers we can now simply remove it.

Change-Id: Ie59e306b69ca397c1e191a0fe379bc084f98cf73
Reviewed-by: Fabian Kosmale <fabian.kosmale@qt.io>
qmllint: Move a lot of warning logic to qmlcompiler 2021-06-09T10:46:02+00:00 Maximilian Goldstein max.goldstein@qt.io 2021-06-07T12:25:01+00:00 3c95bf5b6df7cfd12af3755d11b34db721e23f19 Moves the majority of qmllint warning logic to qmlcompiler, making them available to all tools using the library. The end goal is to get rid off the additional AST visitor in qmllint altogether (We can't quite yet until we have ported over type interference). This also prepares qmlcompiler to move to a two pass approach which isn't fully implemented in here yet due to the size of the change. Pick-to: 6.2 Change-Id: Id2e108340d26a75085ce6ed97d56dec03ea3a12d Reviewed-by: Fabian Kosmale <fabian.kosmale@qt.io> 
Moves the majority of qmllint warning logic to qmlcompiler, making them available to all tools using the library.
The end goal is to get rid off the additional AST visitor in qmllint altogether (We can't quite yet until we have ported over type interference).
This also prepares qmlcompiler to move to a two pass approach which isn't fully implemented in here yet due to the size of the change.

Pick-to: 6.2
Change-Id: Id2e108340d26a75085ce6ed97d56dec03ea3a12d
Reviewed-by: Fabian Kosmale <fabian.kosmale@qt.io>
qmlcompiler/qmllint: Use unified logger 2021-03-29T11:31:01+00:00 Maximilian Goldstein max.goldstein@qt.io 2021-03-25T14:34:42+00:00 954071f372b70f3656386793e2fca382c1b0a1c0 Improves the logging situation greatly by allowing all logging messages to be assigned different severities, highlighting the code that caused them and by now ensuring a qmllint warning will always result in a non-zero exit code. A later patch will expose more of these options to the user. Change-Id: Id9b036fe3ba80dd18e9f8cb1b05efa891713d5a8 Reviewed-by: Ulf Hermann <ulf.hermann@qt.io> 
Improves the logging situation greatly by allowing all logging messages to be assigned different severities,
highlighting the code that caused them and by now ensuring a qmllint warning will always result in a non-zero exit code.

A later patch will expose more of these options to the user.

Change-Id: Id9b036fe3ba80dd18e9f8cb1b05efa891713d5a8
Reviewed-by: Ulf Hermann <ulf.hermann@qt.io>
QmlCompiler: Rename QmlJSImporter 2020-10-05T12:59:03+00:00 Ulf Hermann ulf.hermann@qt.io 2020-10-02T12:01:06+00:00 7fba749edc4b0b69df67316a4714337963b4f754 Add an extra 'Q' for consistency. Change-Id: Idd34e0c898c9a7ac70b1ec3856a8af6829ec6fb7 Reviewed-by: Fabian Kosmale <fabian.kosmale@qt.io> 
Add an extra 'Q' for consistency.

Change-Id: Idd34e0c898c9a7ac70b1ec3856a8af6829ec6fb7
Reviewed-by: Fabian Kosmale <fabian.kosmale@qt.io>
QmlCompiler: Rename metatypes_p.h 2020-10-05T12:58:42+00:00 Ulf Hermann ulf.hermann@qt.io 2020-10-01T12:28:08+00:00 2059c8be904541a31e75c1de9d3b5afa37c6954d Those are specific to QML/JS. Change-Id: I45a5d4eb6c53bd5ca4026e042af83c4afaa4953c Reviewed-by: Fabian Kosmale <fabian.kosmale@qt.io> 
Those are specific to QML/JS.

Change-Id: I45a5d4eb6c53bd5ca4026e042af83c4afaa4953c
Reviewed-by: Fabian Kosmale <fabian.kosmale@qt.io>
QmlCompiler: Rename ScopeTree to QQmlJSScope 2020-10-05T12:58:36+00:00 Ulf Hermann ulf.hermann@qt.io 2020-10-01T12:23:27+00:00 d200ccf92017ebc10a4ccdb5d944e1d803b87c1d That is a better name. Change-Id: I34a6867692a236dd16ed8e3a68866f994eab02d2 Reviewed-by: Fabian Kosmale <fabian.kosmale@qt.io> 
That is a better name.

Change-Id: I34a6867692a236dd16ed8e3a68866f994eab02d2
Reviewed-by: Fabian Kosmale <fabian.kosmale@qt.io>
Long live libQtQmlCompiler! 2020-10-05T12:58:32+00:00 Ulf Hermann ulf.hermann@qt.io 2020-10-01T10:40:01+00:00 767dd738d3de9306062707fe05d32c91ed755da3 Move all the code from tools/shared into src/qmlcompiler and build a static library from it so that we can re-use it in external tools. Change-Id: I7c8d8e59063dc7c711f4072f103a01095e6f5997 Reviewed-by: Fabian Kosmale <fabian.kosmale@qt.io> 
Move all the code from tools/shared into src/qmlcompiler and build a
static library from it so that we can re-use it in external tools.

Change-Id: I7c8d8e59063dc7c711f4072f103a01095e6f5997
Reviewed-by: Fabian Kosmale <fabian.kosmale@qt.io>
qmllint: Remove member access chains from ScopeTree 2020-10-02T13:33:23+00:00 Ulf Hermann ulf.hermann@qt.io 2020-09-30T13:31:43+00:00 b2e1183cb7f4f552bb4409172f82327111f754bb What we need there is a proper type inference. This should be added separately. For now, keep the member access chains local in qmllint and don't pollute ScopeTree with them. Change-Id: I9f50aa4e54b285bd93e7bd4cd17797509df0c168 Reviewed-by: Fabian Kosmale <fabian.kosmale@qt.io> 
What we need there is a proper type inference. This should be added
separately. For now, keep the member access chains local in qmllint and
don't pollute ScopeTree with them.

Change-Id: I9f50aa4e54b285bd93e7bd4cd17797509df0c168
Reviewed-by: Fabian Kosmale <fabian.kosmale@qt.io>
qmllint: Remove ScopeTree::m_unmatchedSignalHandlers 2020-10-02T13:33:17+00:00 Ulf Hermann ulf.hermann@qt.io 2020-09-30T13:03:04+00:00 65b6365ffbde369894f7af1ed735ea34a584d357 Rather, immediately output the warnings when we encounter them. Change-Id: I4bd1a5491a47cb0232bfdd4e325f8f122593aa39 Reviewed-by: Maximilian Goldstein <max.goldstein@qt.io> Reviewed-by: Fabian Kosmale <fabian.kosmale@qt.io> 
Rather, immediately output the warnings when we encounter them.

Change-Id: I4bd1a5491a47cb0232bfdd4e325f8f122593aa39
Reviewed-by: Maximilian Goldstein <max.goldstein@qt.io>
Reviewed-by: Fabian Kosmale <fabian.kosmale@qt.io>
qmllint: Unify injected and "normal" JavaScript identifiers 2020-10-02T13:33:10+00:00 Ulf Hermann ulf.hermann@qt.io 2020-09-30T12:44:24+00:00 7957752447fbd2581521b13156ef29ffc80ad7bf The specifics of how to warn about the injected identifiers are moved out of ScopeTree as that is not related to the structure of the scopes. Change-Id: I26418c3fa492da8339abf045a4034a8464b7bbb8 Reviewed-by: Fabian Kosmale <fabian.kosmale@qt.io> 
The specifics of how to warn about the injected identifiers are moved
out of ScopeTree as that is not related to the structure of the scopes.

Change-Id: I26418c3fa492da8339abf045a4034a8464b7bbb8
Reviewed-by: Fabian Kosmale <fabian.kosmale@qt.io>