Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 341093

Summary: [Autoedit] SL_COMMENT turns into ML_COMMENT
Product: [Modeling] TMF Reporter: Sebastian Zarnekow <sebastian.zarnekow>
Component: XtextAssignee: Project Inbox <tmf.xtext-inbox>
Status: CLOSED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: clay, karsten.thoms
Version: 2.0.0Flags: sebastian.zarnekow: indigo+
Target Milestone: M7   
Hardware: PC   
OS: Mac OS X - Carbon (unsup.)   
Whiteboard:

Description Sebastian Zarnekow CLA 2011-03-28 08:35:47 EDT
// fasdf asdf asd f|

Return at cursor position (|) yields
// fasdf asdf asd f
* |
*/
Comment 1 Sebastian Zarnekow CLA 2011-03-28 08:36:10 EDT
Reproducable in the Xtext grammar editor or with Xtend
Comment 2 Sven Efftinge CLA 2011-03-29 02:22:40 EDT
*** Bug 341185 has been marked as a duplicate of this bug. ***
Comment 3 Michael Clay CLA 2011-03-30 02:45:05 EDT
what else has to be considered here because i cannot reproduce the ML_Comment endings neither with .xtext nor .xtend?

(In reply to comment #1)
> Reproducable in the Xtext grammar editor or with Xtend
Comment 4 Sebastian Zarnekow CLA 2011-03-30 02:56:45 EDT
Xtend-input where | is the cursor position:

/**/
//package foo|

Does not happen with:

/*
 * */
//package foo|

but happens again with

/*
 **/
//package foo|

So the problem might be the double asterisk in the ml comment
Comment 5 Sebastian Zarnekow CLA 2011-03-30 10:49:05 EDT
Something else should be related to this bug:

/*******************************************************************************
 * Some text|
 *******************************************************************************/

Hitting <enter> at the cursor position yields

/*******************************************************************************
 * Some text
 * 
 */
 *******************************************************************************/

which is wrong.
Comment 6 Sebastian Zarnekow CLA 2011-04-11 04:52:45 EDT
Pushed to master.
Comment 7 Karsten Thoms CLA 2017-09-19 17:59:08 EDT
Closing all bugs that were set to RESOLVED before Neon.0
Comment 8 Karsten Thoms CLA 2017-09-19 18:09:50 EDT
Closing all bugs that were set to RESOLVED before Neon.0