Bug 222375 - [Markers] copy markers from markers view should 'pretty print'
Summary: [Markers] copy markers from markers view should 'pretty print'
Status: VERIFIED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: UI (show other bugs)
Version: 3.3   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 3.4 M6   Edit
Assignee: Tod Creasey CLA
QA Contact:
URL:
Whiteboard:
Keywords: contributed
Depends on:
Blocks:
 
Reported: 2008-03-12 06:12 EDT by Benno Baumgartner CLA
Modified: 2008-03-26 16:18 EDT (History)
1 user (show)

See Also:

Attachments
is (367 bytes, text/plain)
2008-03-12 06:12 EDT, Benno Baumgartner CLA 		no flags Details
should (536 bytes, text/plain)
2008-03-12 06:13 EDT, Benno Baumgartner CLA 		no flags Details
patch (1.20 KB, patch)
2008-03-12 09:45 EDT, Benjamin Cabé CLA 		no flags 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 Benno Baumgartner CLA 2008-03-12 06:12:29 EDT 
I20080304-1113

1. Select one or more markers in problems view
2. Ctrl-C
3. Paste to notepad
Comment 1 Benno Baumgartner CLA 2008-03-12 06:12:57 EDT 
Created attachment 92281 [details]
is
Comment 2 Benno Baumgartner CLA 2008-03-12 06:13:19 EDT 
Created attachment 92282 [details]
should
Comment 3 Benno Baumgartner CLA 2008-03-12 06:15:04 EDT 
This makes the feature pretty useless.
Comment 4 Benjamin Cabé CLA 2008-03-12 09:23:57 EDT 
Tod, I think you reasonably can add a "bugday" keyword since it should not be that hard to tackle this bug. Besides, I'd love working on providing the patch ;-)
Comment 5 Benno Baumgartner CLA 2008-03-12 09:33:00 EDT 
helpwanted?! This is a regression, it works in 3.3! Also it looks like a trivial fix, copy seams to add a '13' instead of a new line (13 is the ascii number for return)
Comment 6 Benjamin Cabé CLA 2008-03-12 09:39:44 EDT 
(In reply to comment #5)
> helpwanted?! This is a regression, it works in 3.3! Also it looks like a
> trivial fix, copy seams to add a '13' instead of a new line (13 is the ascii
> number for return)
> 

Oh, indeed!
However, in 3.3 it's not exactly your "should" that is produced by the copy action (i.e. one TAB between each field instead of spaces).
Comment 7 Benjamin Cabé CLA 2008-03-12 09:45:18 EDT 
Created attachment 92302 [details]
patch
Comment 8 Tod Creasey CLA 2008-03-12 15:15:10 EDT 
Patch released with some modifications in build >20080312
Comment 9 Tod Creasey CLA 2008-03-26 16:18:17 EDT 
Verified in I20080325-0100