[Compiler] Add basic types and QtQml module.
ClosedPublic

Authored by akreuzkamp on Sep 11 2015, 11:02 AM.

Details

Summary

Adds a module loader for the QtQml module and adds a static class that
provides all the built in types.

T488

Diff Detail

Repository
R18 QMLWeb
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
akreuzkamp updated this revision to Diff 853.Sep 11 2015, 11:02 AM
akreuzkamp retitled this revision from to [Compiler] Add basic types and QtQml module..
akreuzkamp updated this object.
akreuzkamp edited the test plan for this revision. (Show Details)
akreuzkamp added reviewers: jangmarker, pavelvasev.
akreuzkamp added a task: Restricted Maniphest Task.Sep 11 2015, 12:14 PM
akreuzkamp updated this object.
jangmarker requested changes to this revision.Sep 11 2015, 12:31 PM
jangmarker edited edge metadata.
jangmarker added inline comments.
src/qmljsc/moduleloading/javascriptmoduleloader.cpp
416–417

I'd prefer the variable name type instead of t

src/qmljsc/moduleloading/qtqmlmoduleloader.cpp
50

I'd prefer
p -> property
m -> method
s -> signal

src/qmljsc/moduleloading/qtqmlmoduleloader.h
40

documentation needs update

tests/auto/qmljsc/testmodules.cpp
227

I'd prefer
p => property
m => method
s => signal

This revision now requires changes to proceed.Sep 11 2015, 12:31 PM
akreuzkamp updated this revision to Diff 854.Sep 11 2015, 12:48 PM
akreuzkamp edited edge metadata.

Fixed docu and one variable name.

jangmarker accepted this revision.Sep 11 2015, 12:51 PM
jangmarker edited edge metadata.

thanks

This revision is now accepted and ready to land.Sep 11 2015, 12:51 PM
This revision was automatically updated to reflect the committed changes.
This revision was automatically updated to reflect the committed changes.