Feature #782

Debug in separate console as option in TheIDE

Added by Zbigniew Rebacz almost 10 years ago. Updated over 9 years ago.

Status:ApprovedStart date:05/16/2014
Priority:NormalDue date:
Assignee:Miroslav Fidler% Done:

0%

Category:IDESpent time:-
Target version:-

Description

In my opinion user should decide how he/she wants to debug his/her applications. For example some time ago I used OpenSceneGraph (Pure 3D GUI app) with Core components inside IDE and I was obliged to debug in console.

BTW, Is this code correct? What dose forceconsole flag mean? (ide/Debug.cpp)

bool console = FindIndex(SplitFlags(mainconfigparam, true), "GUI") < 0 || forceconsole;
if (console && !debug_console_apps_in_console)
    console = false;

In this case flag "debug_console_apps_in_console" has got the biggest priority.

At the end I would like to know if the following name for option is appropriate: "Debug console applications in separate console". The second probleme here is following variable name: "debug_console_apps_in_console".

Debug.cpp Magnifier (7.4 KB) Zbigniew Rebacz, 05/16/2014 04:17 PM

DebugInConsoleOption.png (65.8 KB) Zbigniew Rebacz, 05/16/2014 04:17 PM

ide.lay (44.3 KB) Zbigniew Rebacz, 05/16/2014 04:17 PM

ide.h Magnifier (30.7 KB) Zbigniew Rebacz, 05/16/2014 04:17 PM

idewin.cpp Magnifier (20.3 KB) Zbigniew Rebacz, 05/16/2014 04:17 PM

Config.cpp Magnifier (5.69 KB) Zbigniew Rebacz, 05/16/2014 04:17 PM

RunLayoutImprovements.png (33.4 KB) Zbigniew Rebacz, 06/23/2014 09:33 PM

ide.lay - RunLayout improvements (44.4 KB) Zbigniew Rebacz, 06/23/2014 09:33 PM

Debug.cpp Magnifier - Conditional compilation (7.47 KB) Zbigniew Rebacz, 06/23/2014 09:33 PM

History

#1 Updated by Miroslav Fidler almost 10 years ago

Well, I am sorry for the delay, took me a while to figure out what the goal is here.

Please correct me if I am wrong:

You want the option to force ide to run application without "GUI" flag without opening the console. Am I correct?

Well, putting that into main global environment is not that good idea IMO. 'forceconsole', which you ask about, is "Debug"/"Run options" "Always open console (POSIX)" Option. If we are about to implement what you need, I would say we should put "Always run without console" there. Is that OK? (No need to post a patch, I would faster fix that myself...)

#2 Updated by Miroslav Fidler almost 10 years ago

  • Assignee changed from Miroslav Fidler to Zbigniew Rebacz

#3 Updated by Zbigniew Rebacz almost 10 years ago

  • Assignee changed from Zbigniew Rebacz to Miroslav Fidler
You want the option to force ide to run application without "GUI" flag without opening the console. Am I correct?

Yes, I would like to have such feature.
Well, putting that into main global environment is not that good idea IMO. 'forceconsole', which you ask about, is "Debug"/"Run options" "Always open console (POSIX)" Option. If we are about to implement what you need, I would say we should put "Always run without console" there. Is that OK? (No need to post a patch, I would faster fix that myself...)

OK, but will this option work only with selected package? I mean that there will not be global option for turning off console.

#4 Updated by Miroslav Fidler almost 10 years ago

  • Status changed from Patch ready to Ready for QA
  • Assignee changed from Miroslav Fidler to Zbigniew Rebacz

Like this?

#5 Updated by Zbigniew Rebacz over 9 years ago

OK, It seems that it works great, but I have several comments.

First of all we should change a little bit "RunLayout" to make it more polished. (Comp image - RunLayoutImprovements)

The second thing is lable "Run console (POSIX)". If it is only POSIX functionality why not to hide this options by conditional compilation (ide/Debug.cpp):

#ifndef PLATFORM_POSIX
    dlg.consolemode.Hide();
    dlg.console_lable.Hide();
#endif

Please notice that console_lable is new variable from improved layout. But, I would like to notice that console always works on Windows, so I don't think that this option is necessary. What do you think about this???

I enclose following files (all from ide package):
RunLayoutImprovements.png - Comp image
ide.lay - RunLayout improvements
Debug.cpp - conditional compilation

#6 Updated by Miroslav Fidler over 9 years ago

  • Status changed from Patch ready to Approved

Also available in: Atom PDF