suggestion : in case of linux could the "`wx-config --libs` and `wx-config --cflags`" be put at the project level instead of at the targets 'debug and release' level. Since it applies unchanged to both.
So in case they must be changed (OpenSuse wx-config-2.8) there's only one place to adjust.
Nice catch. Thanks for pointing it. :)
I'll make necessary changes.
in my case I created a little dialog app. So I ended up with 1 wxs file. But that file was NOT added to the project, so that means if I change it and then build, it does NOT get autosaved !!!
Could you fix this too ?
Quote from: killerbot on July 06, 2007, 02:48:37 PM
in my case I created a little dialog app. So I ended up with 1 wxs file. But that file was NOT added to the project, so that means if I change it and then build, it does NOT get autosaved !!!
Could you fix this too ?
I'll try to fix this too. :)
I have a few more questions or maybe remarks about the wx project wizard (focusing on the wxSmith track) :
When I have the wizard create me a wxSmith project (dialog based) I have in the
blablaMain.cpp, the following includes :
//(*InternalHeaders(blablaDialog)
#include <wx/bitmap.h>
#include <wx/font.h> //wxFont
#include <wx/fontenum.h>
#include <wx/fontmap.h>
#include <wx/image.h>
#include <wx/intl.h> // _()
#include <wx/settings.h> // wxSystemSettings, wxSystemSettings
#include <wx/string.h> // wxString
//*)
I *think* that the following are NOT needed :
1) wx/bitmap.h
2) wx.fontenum.h
3) wx/fontmap.h
4) wx/image.h
Is this correct ?
A remark about readability : several methods their arguments are all concatenated without any whitespace in between, which makes it hard to read. Is it possible to insert a space (no tab ;-) ) ?
Turn :
Create(parent,id,_("wxWidgets app"),wxDefaultPosition,wxDefaultSize,wxDEFAULT_DIALOG_STYLE,_T("wxDialog"));
into :
Create(parent, id, _("wxWidgets app"), wxDefaultPosition, wxDefaultSize, wxDEFAULT_DIALOG_STYLE, _T("wxDialog"));
I believe Byo would be the best person to answer both the questions. :)
The sample code is a wxSmith generated code. I am unsure whether the generated code can be customised or not.
BTW, I have one question for you.
Quote from: killerbot on July 06, 2007, 02:48:37 PM
in my case I created a little dialog app. So I ended up with 1 wxs file. But that file was NOT added to the project, so that means if I change it and then build, it does NOT get autosaved !!!
On which platform did you face this issue? :)
Quote from: Biplab on July 07, 2007, 12:12:50 PM
The sample code is a wxSmith generated code. I am unsure whether the generated code can be customised or not.
Yes, wxSmith will regenerate that code frequently, so simply changing that code will not help.
@Biplab : on linux
well I should restate it : I looked at my project file (cbp) :
<Unit filename="UtcTranslatorApp.cpp" />
<Unit filename="UtcTranslatorApp.h" />
<Unit filename="UtcTranslatorMain.cpp" />
<Unit filename="UtcTranslatorMain.h" />
<Unit filename="wxsmith/UtcTranslatordialog.wxs" />
So it seems it is added to the project. Seems more a bug of CB then, that a build time it refuses to save it !!!
Quote from: killerbot on July 07, 2007, 11:29:53 AM
I *think* that the following are NOT needed :
1) wx/bitmap.h
2) wx.fontenum.h
3) wx/fontmap.h
4) wx/image.h
Is this correct ?
Right, those are not needed, but including them conditionally will require some more sophisticated include-generating system checking whether wxBitmap / wxImage etc is really used in generated code. Generated code is correct and will compile fine so I've marked it to post-release feature.
Quote
A remark about readability : several methods their arguments are all concatenated without any whitespace in between, which makes it hard to read. Is it possible to insert a space (no tab ;-) ) ?
Quite easy to do but time-consuming. I just wonder why someone would like to read wxSmith's generated code ;) Since it's automatically generated, people may thread it as some black-box (even though it's content is available) which just do what it should. Currently I don't have enough time to switch. So this is probably also post-release task.
Regards
BYO
QuoteQuite easy to do but time-consuming. I just wonder why someone would like to read wxSmith's generated code Wink Since it's automatically generated, people may thread it as some black-box (even though it's content is available) which just do what it should. Currently I don't have enough time to switch. So this is probably also post-release task.
If you can pinpoint me the code, I am happy to help ;-)
Quote from: killerbot on July 09, 2007, 01:18:52 PM
QuoteQuite easy to do but time-consuming. I just wonder why someone would like to read wxSmith's generated code Wink Since it's automatically generated, people may thread it as some black-box (even though it's content is available) which just do what it should. Currently I don't have enough time to switch. So this is probably also post-release task.
If you can pinpoint me the code, I am happy to help ;-)
Search for OnBuildCreatingCode functions in classes inside wxSmith/wxwidgets/defitems directory. Each of these functions is responsible for generating code and usually it's done through
Codef function. This function works simillar to printf but is dedicated to code generation. So when you have f. ex. :
Code << Codef(Language,_T("%C(%W,%I,%P,%S,%T,%N);\n"));
Just change it to:
Code << Codef(Language,_T("%C(%W, %I, %P, %S, %T, %N);\n"));
That's all :)
(Note that some funcitons may use easier version of Codef - without streaming operator and Language argument, but it works the same).
Regards
BYO