Home » Developing U++ » UppHub » Added SysExec package
|
Re: Added SysExec package [message #30816 is a reply to message #30802] |
Mon, 24 January 2011 09:54 |
|
mdelfede wrote on Sun, 23 January 2011 18:36 | Added SysExecGui package, whith gui frontend for password reading (in Linux) and same functions (without password reading) for windows, to have an uniform interface.
It uses SysExec package, adding CtrlLib dependency for gui apps; for non-gui apps, use SysExec which don't have CtrlLib dependencies.
|
Hi Max,
This is a very nice package and definitely useful. I have just one proposal: what about merging SysExec and SysExecGui into one package? I think it makes things easier for user. All you would need to do is:
1) Move files from SysExecGui to SysExec
2) Put #ifdef flagGUI guard around the code in SysExecGui.cpp
3) Put #include SysExecGui.h at the end of SysExec.h (again only #ifdef flagGUI)
4) Add package CtrlLib into SysExec with "when GUI" condition
5) Check the include paths for SysExecGui leftover and few other simple to solve details.
It would make life a little bit easier for people (like me) who sometimes write apps that can be compiled both with GUI and CLI or who start with CLI and later switch to GUI But it is just a proposal, if you think it is not a clean solution feel free to leave it as is, after all it is not that hard to make it work the same way on the user package level
Best regards,
Honza
|
|
|
|
|
Goto Forum:
Current Time: Sat May 04 15:26:18 CEST 2024
Total time taken to generate the page: 0.03846 seconds
|