<table><tr><td style="">pprkut accepted this revision.<br />pprkut added a comment.<br />This revision is now accepted and ready to land.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D6342" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>The changes look good, although I think it would be preferable to have the basic types in a separate commit from the more complicated ones like <tt style="background: #ebebeb; font-size: 13px;">iterable</tt>, <tt style="background: #ebebeb; font-size: 13px;">self</tt>, <tt style="background: #ebebeb; font-size: 13px;">object</tt>, etc.</p>

<p>Regarding the tokens, I'm afraid that's the way it is right now. There's currently no way to support more than one PHP version and traditionally we opted for the newest one.<br />
My next goal was context sensitive lexer support which should take care of the problem with using the tokens as identifiers, so I wouldn't worry too much about that.</p>

<p>IMHO, <tt style="background: #ebebeb; font-size: 13px;">self</tt> should still work as a token. We could take a look at the errors you're getting in a separate WIP diff. You could take some hints from the grammar definition in the PHP7 source and see how they did it for comparison.</p>

<p>Regarding <tt style="background: #ebebeb; font-size: 13px;">callable</tt>, I'm not sure what you mean.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R52 KDevelop: PHP Support</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D6342" rel="noreferrer">https://phabricator.kde.org/D6342</a></div></div><br /><div><strong>To: </strong>mtijink, KDevelop, pprkut<br /><strong>Cc: </strong>apol, kdevelop-devel, geetamc, Pilzschaf, akshaydeo, surgenight, arrowdodger<br /></div>