Review Request 111547: Add support for PHP 5.5's array literal dereferencing.

Milian Wolff mail at milianw.de
Wed Jul 17 22:16:03 UTC 2013


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/111547/#review36086
-----------------------------------------------------------

Ship it!


Just remove the obsolete code, then +1 for shipping it.


duchain/expressionvisitor.cpp
<http://git.reviewboard.kde.org/r/111547/#comment26738>

    this is not doing anything anymore, just remove it please


- Milian Wolff


On July 17, 2013, 2:07 p.m., Heinz Wiesinger wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/111547/
> -----------------------------------------------------------
> 
> (Updated July 17, 2013, 2:07 p.m.)
> 
> 
> Review request for KDevelop.
> 
> 
> Description
> -------
> 
> Add support for PHP 5.5's array literal dereferencing.
> 
> This would for example look like this:
> 
> $a = [1,2,3][0];
> 
> where $a = 1;
> 
> The rfc detailing the feature can be found here: https://wiki.php.net/rfc/constdereference
> 
> I had to rewrite the array handling a little bit to make the typing work as expected and adjusted the unit tests for that as well.
> 
> 
> Diffs
> -----
> 
>   duchain/expressionvisitor.h 3b951fbe878e8a9bd0d8c981a9c2f8c02bfcbc8f 
>   duchain/expressionvisitor.cpp 0a926167eeb5e493f22ac62704865ad6461b64d8 
>   duchain/tests/expressionparser.h a749e973d11505d94abf1ea43d45513ca22d5244 
>   duchain/tests/expressionparser.cpp 72970fce85153fdb1baf97163c306a0e6c312d6c 
>   parser/php.g 55b7b64de919137b03331569381dee45cdb21fb2 
> 
> Diff: http://git.reviewboard.kde.org/r/111547/diff/
> 
> 
> Testing
> -------
> 
> Unit tests and php example code
> 
> 
> Thanks,
> 
> Heinz Wiesinger
> 
>

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


More information about the KDevelop-devel mailing list