Bug 304506 - Task descriptions always have a space after the tag
Summary: Task descriptions always have a space after the tag
Status: VERIFIED FIXED
Alias: None
Product: JDT
Classification: Eclipse Project
Component: Core (show other bugs)
Version: 3.6   Edit
Hardware: PC Windows XP
: P3 minor (vote)
Target Milestone: 3.6 M6   Edit
Assignee: Olivier Thomann CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-03-03 07:58 EST by Markus Keller CLA
Modified: 2010-03-09 11:58 EST (History)
2 users (show)

See Also:

Attachments
Proposed fix + updated regression tests (12.77 KB, patch)
2010-03-03 14:46 EST, Olivier Thomann CLA 		no flags Details | Diff
Proposed fix + updated regression tests (18.28 KB, patch)
2010-03-04 11:47 EST, Olivier Thomann CLA 		no flags Details | Diff
Show Obsolete (1) 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 Markus Keller CLA 2010-03-03 07:58:32 EST 
I20100302-0800

In a CU, have:
// TODO:     Buy      bananas.
//TODO       Eat bananas

The generated tasks in the Tasks view look like this:
TODO :     Buy      bananas.
TODO Eat bananas

Expected: Should look like I wrote them (first without " " before ":", second with more space). You should not trim the description string, and you should not insert a space between task tag and description.
Comment 1 Olivier Thomann CLA 2010-03-03 14:41:24 EST 
Do you also want to keep the spaces after the message ?
Comment 2 Olivier Thomann CLA 2010-03-03 14:46:42 EST 
Created attachment 160832 [details]
Proposed fix + updated regression tests

This doesn't trim the start of the message anymore, but it trims the end.
This might have effect on JDT/UI tests.
Comment 3 Dani Megert CLA 2010-03-04 03:52:17 EST 
>Do you also want to keep the spaces after the message ?
No.
Comment 4 Olivier Thomann CLA 2010-03-04 11:47:11 EST 
Created attachment 160957 [details]
Proposed fix + updated regression tests
Comment 5 Olivier Thomann CLA 2010-03-04 11:47:39 EST 
Resolved for 3.6M6.
Updated existing regression tests.
Comment 6 Frederic Fusier CLA 2010-03-09 11:58:52 EST 
Verified for 3.6M6 using I20100307-2000 build.