Review Request 118070: Support enumerations in plugin.qmltypes files

Commit Hook null at kde.org
Sat May 10 13:33:51 UTC 2014


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/118070/#review57676
-----------------------------------------------------------


This review has been submitted with commit 07434a43464b75d3a9419be9c727eafee8a1d9d6 by Denis Steckelmacher to branch master.

- Commit Hook


On May 10, 2014, 1:10 p.m., Denis Steckelmacher wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/118070/
> -----------------------------------------------------------
> 
> (Updated May 10, 2014, 1:10 p.m.)
> 
> 
> Review request for KDevelop.
> 
> 
> Repository: kdev-qmljs
> 
> 
> Description
> -------
> 
> This patch adds support for Enum definitions in plugin.qmltypes files. The support is still minimal because the enumeration is recognized as being an enum, but its values are not yet parsed. An upcoming patch will add support for that, by parsing the "values:" key of Enum definitions.
> 
> 
> Diffs
> -----
> 
>   duchain/declarationbuilder.cpp 212d806 
>   tests/files/plugins.qml 1944023 
> 
> Diff: https://git.reviewboard.kde.org/r/118070/diff/
> 
> 
> Testing
> -------
> 
> This patch adds a test in tests/files/plugins.qml. This test ensures that enumerations are properly recognized and that their declaration and inner context have the proper type. All the other tests still pass.
> 
> 
> Thanks,
> 
> Denis Steckelmacher
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20140510/cb10a760/attachment.html>


More information about the KDevelop-devel mailing list