Home » U++ Library support » U++ Widgets - General questions or Mixed problems » tutorial check completed - "bugs" and bug
| tutorial check completed - "bugs" and bug [message #4680] |
Sun, 20 August 2006 11:55  |
Werner
Messages: 234
Registered: May 2006
Location: Cologne / Germany
|
Experienced Member |
|
|
There is a minor "bug" in tutorial "Gui14", "Gui15", "Gui20a", "Gui20b", and "Gui20c":
The respective line
typedef MyApp[Window] CLASSNAME;
is unnecessary.
Tutorial "Ntl01" is broken:
Ntl01.upp line #5 reads
but should read
Werner
|
|
|
|
|
|
| Re: tutorial check completed - "bugs" and bug [message #4705 is a reply to message #4692] |
Sun, 20 August 2006 21:33  |
Werner
Messages: 234
Registered: May 2006
Location: Cologne / Germany
|
Experienced Member |
|
|
| luzr wrote on Sun, 20 August 2006 20:23 |
| Werner wrote on Sun, 20 August 2006 05:55 |
There is a minor "bug" in tutorial "Gui14", "Gui15", "Gui20a", "Gui20b", and "Gui20c":
The respective line
typedef MyApp[Window] CLASSNAME;
is unnecessary.
|
That is true, but in this case, I think I will keep it as it is. Adding this typedef for GUI class is a good practice in U++.... (because you soon will need some THISBACK anyway 
Mirek
|
Apologies for being insistent  , but might I suggest that you reconsider your decision  ?
After all the tutorials are not production code, or are they? They are meant to demonstrate a certain topic. And here the KISS principle applies: Keep it short and simple! If you do prefer to retain the line in question - and it goes without saying that your "good practice" argument is compelling - please consider to document this line. There are probably a lot of users like me who are not professional programmers and have to strive hard to analyze code undocumented to a large extent  .
Werner
|
|
|
|
| Re: tutorial check completed - "bugs" and bug [message #4712 is a reply to message #4705] |
Sun, 20 August 2006 21:44 |
 |
fudadmin
Messages: 1321
Registered: November 2005
Location: Kaunas, Lithuania
|
Ultimate Contributor
Administrator |
|
|
| Werner wrote on Sun, 20 August 2006 20:33 |
| luzr wrote on Sun, 20 August 2006 20:23 |
| Werner wrote on Sun, 20 August 2006 05:55 |
There is a minor "bug" in tutorial "Gui14", "Gui15", "Gui20a", "Gui20b", and "Gui20c":
The respective line
typedef MyApp[Window] CLASSNAME;
is unnecessary.
|
That is true, but in this case, I think I will keep it as it is. Adding this typedef for GUI class is a good practice in U++.... (because you soon will need some THISBACK anyway
Mirek
|
Apologies for being insistent , but might I suggest that you reconsider your decision ?
After all the tutorials are not production code, or are they? They are meant to demonstrate a certain topic. And here the KISS principle applies: Keep it short and simple! If you do prefer to retain the line in question - and it goes without saying that your "good practice" argument is compelling - please consider to document this line. There are probably a lot of users like me who are not professional programmers and have to strive hard to analyze code undocumented to a large extent .
Werner
|
May I suggest, that it should just mentioned somewhere with big letters that "typedef is not "a must" but "a good practice" for THISBACK macro callbacks"
[Updated on: Sun, 20 August 2006 21:48] Report message to a moderator |
|
|
|
|
|
Goto Forum:
Current Time: Mon Apr 28 16:43:19 CEST 2025
Total time taken to generate the page: 0.00863 seconds
|
Members
Pages
Help
Register
Login
Home
