Community
Participate
Working Groups
Build Identifier: 2.2.0.v20100616 If have multi line comment entered at PDT Editor at some cases entered parameters there are read like a PHPDoc comment Reproducible: Always Steps to Reproduce: 1.Have the following code at PDT <?php /*** * Main description * @param test * @author test */ function test (){ } $test = te| //invoke content assist at | sign 2. At CA look at suggested for test() tooltip 3. Hover over test() definition and look again tooltip Expected result: This is not a PHPDoc comment so at tool tips must have only function name without any additional information - it must be the same like use /* for comment start Actual result: At tool tips have additional information about functions comment - can see whole information like a PHPDoc, but this is not a PHPDoc - look attached
Created attachment 172210 [details] Wrong tooltip suggestion
It's nice catch, however PHPDocumentor manual doesn't say that such comments are not phpdoc blocks. They say that "every DocBlock comment is a C-style comment with two leading asterisks (*)". So imho we neither should be making any assumptions or guesses about user intentions to not break compatibility with PHPDocumentor. For a counter-example, one could prefer following commenting style and argue that it's valid phpdoc: /******************* * My super-important method with lots of asterisks. *******************/ makes sense Teo?
Hi Teo It is my fault that I mislead you,I asked Roy about this bug,and he also thought /*** * Main description * @param test * @author test */ is a phpdoc,so the problem is the above phpdoc is highlighted wrongly:)
ah so now the fix is to make comments starting with "/***" blue, instead of green, like on Teo's screenshot, yes? updating title a little to match our new target :-)
My proposed fix is available: https://git.eclipse.org/r/#/c/35756/ Thierry.
Merged: https://git.eclipse.org/c/pdt/org.eclipse.pdt.git/commit/?id=5abc11257f541474bb1f86ac1437b9254b9d7ad6 Thanks Thierry!
Comment is now blue instead of green. Closing