Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 321422 - Breakpoint position in Xtend and Xpand is not right calculated
Summary: Breakpoint position in Xtend and Xpand is not right calculated
Status: CLOSED FIXED
Alias: None
Product: M2T
Classification: Modeling
Component: Xpand (show other bugs)
Version: 1.0.0   Edit
Hardware: PC Mac OS X - Carbon (unsup.)
: P3 normal (vote)
Target Milestone: SR1   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-07-31 05:22 EDT by Benjamin Schwertfeger CLA
Modified: 2013-02-21 08:10 EST (History)
1 user (show)

See Also:
sebastian.zarnekow: helios+

Attachments
Patch for correct line calculation. Position must be inside the text area. (1.33 KB, patch)
2010-07-31 05:22 EDT, Benjamin Schwertfeger CLA 		sven.efftinge: iplog+
Details | Diff
View All 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 Benjamin Schwertfeger CLA 2010-07-31 05:22:28 EDT 
Created attachment 175639 [details]
Patch for correct line calculation. Position must be inside the text area.

Double click on the gray border of a text editor calulates the position of the breakpoint in the wrong way (calculation is aborted and last line is set). The enables setting the breakpoint only at the first element in the file.

Right click somewhere in the editor and select "Toggle Breakpoint" works for me. Some problem is written as a comment to #267779. But the root problem there seems to be different.

I'll append a patch for line calculation in org.eclipse.xpand.shared.ui, which solves the breakpoint problem.
Comment 1 Benjamin Schwertfeger CLA 2010-08-03 05:33:09 EDT 
As a work around you can right click in the text editor on the line for the break point and select toggle breakpoint. In this case the break point is set correctly.
Comment 2 Sebastian Zarnekow CLA 2010-08-14 08:09:04 EDT 
looks good to me. Patch applied to mwe as well.
Comment 3 Karsten Thoms CLA 2013-02-21 08:10:45 EST 
Bug resolved before Xpand 1.2 release date => Closing