<table><tr><td style="">mtijink created this revision.<br />mtijink added a project: KDevelop.<br />Restricted Application added a subscriber: kdevelop-devel.
</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/D6643" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>This diff adds support for php return typehints (e.g. <tt style="background: #ebebeb; font-size: 13px;">function foo(): string</tt>) to the parser. Nothing is done yet to use this in the TypeBuilder, that will come later.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>The newly added test checks if the return typehint is correctly tokenized. Parsing itself is not tested, but the old tests work (so no regressions) and opening files with return typehints now works without parse errors. Tests will be added after using the return typehints in the TypeBuilder.</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/D6643" rel="noreferrer">https://phabricator.kde.org/D6643</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>completion/context.cpp<br />
parser/php.g<br />
parser/phplexer.cpp<br />
parser/test/lexertest.cpp<br />
parser/test/lexertest.h</div></div></div><br /><div><strong>To: </strong>mtijink, KDevelop<br /><strong>Cc: </strong>kdevelop-devel, KDevelop, geetamc, Pilzschaf, akshaydeo, surgenight, arrowdodger<br /></div>