Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
RE: [cdt-dev] RxThread.processMIOOBRecord - undesired console I/O?

Thanks, Marc.  I’ll have a look.  I suspect the easiest and least invasive way is to add the check for isEnableConsole(), but I’ll look over the fixes before recommending a change.

            -- Steve G.

 

From: cdt-dev-bounces@xxxxxxxxxxx [mailto:cdt-dev-bounces@xxxxxxxxxxx] On Behalf Of Marc Khouzam
Sent: Thursday, July 22, 2010 10:17 AM
To: 'CDT General developers list.'
Subject: RE: [cdt-dev] RxThread.processMIOOBRecord - undesired console I/O?

 

If it helps, I believe we had the same (or very similar) problem in DSF-GDB which we

solved with https://bugs.eclipse.org/bugs/show_bug.cgi?id=220959

 


From: cdt-dev-bounces@xxxxxxxxxxx [mailto:cdt-dev-bounces@xxxxxxxxxxx] On Behalf Of Steve Goodrich
Sent: Thursday, July 22, 2010 9:38 AM
To: 'CDT General developers list.'
Subject: [cdt-dev] RxThread.processMIOOBRecord - undesired console I/O?

I’m working in Helios using CDI (GDB).  I’m sending “monitor” commands to GDB, and I’m seeing the output from the commands in the console window.  I’ve disabled the console programmatically, but I’m still getting the “noise” on the output.

 

Looking at processMIOOBRecord(MIStreamRecord) in org.eclipse.cdt.debug.mi.core.RxThread, I see the following (note that the middle case ignores isEnableConsole()):

 

            if (stream instanceof MIConsoleStreamOutput) {

                        // ...

                        if (str != null && isEnableConsole()) {

                        // ...

            } else if (stream instanceof MITargetStreamOutput) {

                        // ...

                        if (str != null) {

                        // ...

            } else if (stream instanceof MILogStreamOutput) {

                        // ...

                        if (str != null && isEnableConsole()) {

 

Question: Is this desired, or could the middle case be changed to respect isEnableConsole?

 

Thanks,

            -- Steve G.


Back to the top