Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 325556 - GDB 7.x MI thread details field ignored (DSF)
Summary: GDB 7.x MI thread details field ignored (DSF)
Status: RESOLVED FIXED
Alias: None
Product: CDT
Classification: Tools
Component: cdt-debug-dsf-gdb (show other bugs)
Version: 7.0   Edit
Hardware: PC All
: P3 normal with 1 vote (vote)
Target Milestone: 7.0.2   Edit
Assignee: Marc Khouzam CLA
QA Contact: Marc Khouzam CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 378152
  Show dependency tree
 
Reported: 2010-09-17 05:03 EDT by John Dallaway CLA
Modified: 2012-05-01 09:15 EDT (History)
1 user (show)

See Also:


Attachments
GDBProcesses_7_0-100917.patch (1.29 KB, patch)
2010-09-17 05:10 EDT, John Dallaway CLA
marc.khouzam: iplog+
Details | Diff
Updated fix to not use Java 1.6 constructs (1.80 KB, patch)
2010-12-16 15:36 EST, Marc Khouzam CLA
marc.khouzam: iplog-
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description John Dallaway CLA 2010-09-17 05:03:35 EDT
Build Identifier: Eclipse 3.6.0 I20100608-0911 (with cdt_7_0 branch and CDT HEAD)

Many GDB servers provide thread information details which can be very useful to the developer. When debugging with GDB 6.x, CLIInfoThreadsInfo#parseThreadInfo() folds this extra information into the OS-specific thread ID for presentation to the user in the Debug view. When debugging with GDB 7.x, the information is made available by MIThread#getDetails() but this method is not used by the higher-level code.

Workaround: Type "info threads" in the GDB console to view the thread details.

Reproducible: Always
Comment 1 John Dallaway CLA 2010-09-17 05:10:38 EDT
Created attachment 179092 [details]
GDBProcesses_7_0-100917.patch

This patch passes the GDB 7.x MI thread details into the MIThreadDMData object for presentation to the user within the Debug view. With this patch, the presentation of thread details in the Debug view matches that observed when using GDB 6.x. The patch should be usable for both CDT HEAD and the cdt_7_0 branch.
Comment 2 CDT Genie CLA 2010-12-16 15:23:06 EST
*** cdt cvs genie on behalf of mkhouzam ***
Bug 325556: GDB 7.x MI thread details field ignored (DSF)

[*] GDBProcesses_7_0.java 1.21.2.6 http://dev.eclipse.org/viewcvs/index.cgi/org.eclipse.cdt/dsf-gdb/org.eclipse.cdt.dsf.gdb/src/org/eclipse/cdt/dsf/gdb/service/GDBProcesses_7_0.java?root=Tools_Project&r1=1.21.2.5&r2=1.21.2.6
Comment 3 Marc Khouzam CLA 2010-12-16 15:36:37 EST
Created attachment 185366 [details]
Updated fix to not use Java 1.6 constructs

Small change to not use String.isEmpty()

Committed to HEAD and 7_0

John can you confirm it works as you expect?

Thanks for the contribution
Comment 4 Marc Khouzam CLA 2010-12-16 15:53:47 EST
Fixed
Comment 6 John Dallaway CLA 2010-12-17 11:56:33 EST
(In reply to comment #3)

> Committed to HEAD and 7_0
> 
> John can you confirm it works as you expect?

The final patch applied to the cdt_7_0 branch works well for me. Thank you.