| Summary: | Windows PE binary parser in CDT does not return global variables | ||
|---|---|---|---|
| Product: | [Tools] CDT | Reporter: | Stefan Bylund <steby> |
| Component: | cdt-core | Assignee: | Project Inbox <cdt-core-inbox> |
| Status: | NEW --- | QA Contact: | Jonah Graham <jonah> |
| Severity: | normal | ||
| Priority: | P3 | CC: | malaperle, steby, yevshif |
| Version: | 8.0 | ||
| Target Milestone: | --- | ||
| Hardware: | PC | ||
| OS: | Windows XP | ||
| Whiteboard: | |||
| Attachments: | |||
|
Description
Stefan Bylund
Created attachment 202317 [details]
Simple test program for exemplifying the bug
Created attachment 202318 [details]
Patch for org.eclipse.cdt.core/src/org/eclipse/cdt/utils/coff/Coff.java
Created attachment 202319 [details]
Patch for org.eclipse.cdt.core/src/org/eclipse/cdt/utils/coff/parser/PEBinaryObject.java
Is there any chance of having this fix included in CDT 8.0.1? Can I do anything to facilitate that? (In reply to comment #4) > Is there any chance of having this fix included in CDT 8.0.1? Can I do anything > to facilitate that? Hi, thanks for working on that. This can't go in 8.0.1 or 8.0.2 because it introduces new API (public members in a class that's not in an internal package). You can use API Tooling to see that. http://wiki.eclipse.org/CDT/policy#Using_API_Tooling The new public members will need @since tags. Also, could you generate a single patch that can apply inside Eclipse? Here's a video that shows how to do it with EGit: http://www.youtube.com/user/cdtdoug#p/a/u/2/Lhwi3z2DyzQ |