Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 312509

Summary:

EDCSymbolReader.java build error.

Product:

[Tools] CDT

Reporter:

Dave Korn <dave.korn.cygwin>

Component:

cdt-debug-edc

Assignee:

Ken Ryall <ken.ryall>

Status:

RESOLVED INVALID

QA Contact:

Ken Ryall <ken.ryall>

Severity:

normal

Priority:

CC:

john.cortell

Version:

0 DD 1.1

Target Milestone:

---

Hardware:

OS:

Windows XP

Whiteboard:

Attachments:

Description	Flags
Simple fix.	none

Description Dave Korn

2010-05-11 16:36:26 EDT

Build Identifier: I20100429-1549

With current CVS HEAD, I get the following error during compilation of org.eclipse.cdt.debug.edc.internal.symbols.dwarf/EDCSymbolReader.java

The type EDCSymbolReader must implement the inherited abstract method ISymbolReader.getSourceFiles(IProgressMonitor)



Reproducible: Always

Steps to Reproduce:
1. Install M7 SDK + (fixed) cdt master M7 release.
2. Check out cdt tools from dev.eclipse.org CVS
3. Build all.

Comment 1 Dave Korn

2010-05-11 16:39:07 EDT

Created attachment 168032 [details]
Simple fix.

I trivially copypasta'd the definition of the missing function from the other symbol readers that do implement it:
org.eclipse.cdt.utils.{coff/CodeViewReader,debug.dwarf/DwarfReader,debug.stabs/StabsReader}.java

It fixes the build for me.

Comment 2 John Cortell

2010-05-11 16:40:35 EDT

EDCSymbolReader does implement that method. It was added last Friday. Make sure you have the latest (rev 1.5) of EDCSymbolReader. Sounds to me you're not completely in synch with the repository.

Comment 3 Dave Korn

2010-05-11 16:46:09 EDT

(In reply to comment #2)
> EDCSymbolReader does implement that method. It was added last Friday. Make sure
> you have the latest (rev 1.5) of EDCSymbolReader. Sounds to me you're not
> completely in synch with the repository.

How bizarre.  I updated using the team synch view last night, but for some reason I still only have 1.2; yet I got a whole lot of other updates.  Sorry for the false alarm.  (I may turn out to have found a bug in the team sync view, or maybe I've mangled my cvs metadata somehow, I'll open a fresh bug if it turns out to be the former.)  Sorry for the noise.