Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 341093 - [Autoedit] SL_COMMENT turns into ML_COMMENT
Summary: [Autoedit] SL_COMMENT turns into ML_COMMENT
Status: CLOSED FIXED
Alias: None
Product: TMF
Classification: Modeling
Component: Xtext (show other bugs)
Version: 2.0.0   Edit
Hardware: PC Mac OS X - Carbon (unsup.)
: P3 normal (vote)
Target Milestone: M7   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
: 341185 (view as bug list)
Depends on:
Blocks:
 
Reported: 2011-03-28 08:35 EDT by Sebastian Zarnekow CLA
Modified: 2017-09-19 18:09 EDT (History)
2 users (show)

See Also:
sebastian.zarnekow: indigo+

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
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