243200 2010-06-29 22:20:14 +0000 Code completion mis-parses //* resulting in not working 2011-06-09 04:34:40 +0000 1 1 2 Applications kdevelop Code completion git master Compiled Sources Linux RESOLVED FIXED NOR minor 4.0.1 1 mactalla.obair kdevelop-bugs-null nalvarez 0 oldest_to_newest 984452 0 mactalla.obair 2010-06-29 22:20:14 +0000 Version: SVN (using KDE 4.4.3) OS: Linux Code completion won't trigger after a comment beginning in //* until the scope changes. Reproducible: Always Steps to Reproduce: Step 1: Open a c++ file and chose a place to use code completion. Step 2: Type a bit or use ctrl-space to confirm the popup appears. Step 3: Add a comment beginning in //* within the same scope. Step 4: Try to trigger code completion after this comment (within the same scope). Actual Results: Code completion won't trigger. Expected Results: Code completion should trigger. This is a valid one-line comment, but the code completion refuses to trigger. I expect it's parsing it as a /* comment. Severity is minor as it is easy to work around, but it is non-obvious for anyone who encounters it and feels like kdevelop is broken until my typo is discovered. 1028699 1 nalvarez 2010-10-07 04:12:14 +0000 I can reproduce this in 4.1. 1028700 2 nalvarez 2010-10-07 05:12:34 +0000 I think the solution is simply to make KDevelop::clearComments clear //comments before /*comments*/. After clearing //comments, it won't find a "/*" anymore, and it won't wrongly think the cursor is inside a comment. I'll try this tomorrow. 1129130 3 olivier.jg 2011-06-09 04:34:40 +0000 Git commit 1b51218cd6122230e40698934e7bcc311a92be07 by Olivier JG. Committed on 09/06/2011 at 04:31. Pushed by olivierjg into branch 'master'. Fix clearComments() to work with "//*" and "/*//*/" stuff that it used to choke on (See testClearComments in test_cppcodecompletion) BUG: 243200 M +30 -39 language/duchain/stringhelpers.cpp http://commits.kde.org/kdevplatform/1b51218cd6122230e40698934e7bcc311a92be07