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

Bug 323997

Summary: edc windows agent - add capability to use new dialog() in ILogging
Product: [Tools] CDT Reporter: Kirk Beitz <kirk.beitz>
Component: cdt-debug-edcAssignee: Ken Ryall <ken.ryall>
Status: RESOLVED FIXED QA Contact: Ken Ryall <ken.ryall>
Severity: normal    
Priority: P3 CC: cdtdoug
Version: 7.0   
Target Milestone: ---   
Hardware: PC   
OS: All   
Whiteboard:
Bug Depends on: 322609    
Bug Blocks: 323998    
Attachments:
Description Flags
add writeln() & dialog() support to LoggingService.cpp + .h cdtdoug: iplog-, kirk.beitz: review?

Description Kirk Beitz CLA 2010-08-30 13:02:53 EDT
the ILogging service has added dialog() since 2.0 .

This change provides a way for the windows debug agent to make use of the service.

See the attached patch for the difference with the LoggingService.cpp & LoggingService.h as they exist on top of the changes to be submitted for ecl.bz 322609 .

Once ecl.bz 322609 has been committed and is available on the mirror service for download and patch creation, i will create a new patch and attach it to this ticket.
Comment 1 Kirk Beitz CLA 2010-08-30 16:36:02 EDT
Created attachment 177783 [details]
add writeln() & dialog() support to LoggingService.cpp + .h

patch is ready for review, has been tested with local change to edc windows debugger (not to be committed) to open warning with details at startup.
Comment 2 Ken Ryall CLA 2010-09-01 01:00:14 EDT
Committed to HEAD
Comment 4 Glyn Normington CLA 2010-09-03 03:52:23 EDT
I'm intrigued why 323609 was blocking this bug, especially as this bug was resolved before 323609. Please could you briefly summarise your use of Virgo? Thanks!
Comment 5 Kirk Beitz CLA 2010-09-03 04:46:42 EDT
typo - depends on should have been 322609, not 323609