Home » Community » U++ community news and announcements » Upp 2008.1beta2 v2 released
Re: Upp 2008.1beta2 v2 released [message #15221 is a reply to message #15216] |
Thu, 10 April 2008 09:33   |
mr_ped
Messages: 826 Registered: November 2005 Location: Czech Republic - Praha
|
Experienced Contributor |
|
|
luzr wrote on Thu, 10 April 2008 02:59 |
mr_ped wrote on Tue, 08 April 2008 08:13 |
Also the -Wall switch for GCC throws many warnings in Core package, some of them can be easily fixed (like for example:
C:\upp\uppsrc/Core/Parser.h:2: warning: suggest parentheses around && within ||
|
Sorry. I am not gonna to fix this. I am no slave of some idiot who does not remember that && has higher priority than || and thus adds such idiotic warning.
What will be next? What about (x * y) + 10?
The whole purpose of operator priorities is to reduce the number of parenthesis. Do you really think this code is more readable
return ((c >= 'a') && (c <= 'z')) || ((c >= 'A') && (c <= 'Z')) || (c == '_');
??
Mirek
|
Actually, I'm one of those idiots, and I prefer exactly that return like you wrote it. 
I have 2 reasons to do so... from my historic experience I don't believe compilers (it's not like I hit a bug in them every month or so, actually probably less than 1 bug per year, but it made my faith gone), and for me it's easier to "parse" parentheses in head, than to think about interaction between operators, as the parentheses is one *single* rule, and there are *many* operators. I got so far that in my source I already see ((a * b) + c); 
edit: but this is the least important "problem" from those I reported. Look rather at that weird LanguageInfo without return issue.
[Updated on: Thu, 10 April 2008 09:35] Report message to a moderator
|
|
|
 |
|
Upp 2008.1beta2 v2 released
By: unodgs on Tue, 25 March 2008 23:52
|
 |
|
Re: Upp 2008.1beta2 v2 released
By: Tom1 on Wed, 26 March 2008 09:55
|
 |
|
Re: Upp 2008.1beta2 v2 released
By: mirek on Wed, 26 March 2008 09:59
|
 |
|
Re: Upp 2008.1beta2 v2 released
By: mdelfede on Wed, 26 March 2008 20:34
|
 |
|
Re: Upp 2008.1beta2 v2 released
By: mr_ped on Mon, 07 April 2008 12:28
|
 |
|
Re: Upp 2008.1beta2 v2 released
By: mr_ped on Mon, 07 April 2008 12:37
|
 |
|
Re: Upp 2008.1beta2 v2 released
By: mr_ped on Tue, 08 April 2008 14:13
|
 |
|
Re: Upp 2008.1beta2 v2 released
By: mdelfede on Wed, 09 April 2008 10:30
|
 |
|
Re: Upp 2008.1beta2 v2 released
By: mirek on Thu, 10 April 2008 02:59
|
 |
|
Re: Upp 2008.1beta2 v2 released
By: mr_ped on Thu, 10 April 2008 09:33
|
 |
|
Re: Upp 2008.1beta2 v2 released
By: unodgs on Thu, 10 April 2008 12:48
|
 |
|
Re: Upp 2008.1beta2 v2 released
By: mdelfede on Thu, 10 April 2008 15:02
|
 |
|
Re: Upp 2008.1beta2 v2 released
By: bonami on Thu, 10 April 2008 05:13
|
 |
|
Re: Upp 2008.1beta2 v2 released
By: mirek on Thu, 10 April 2008 14:09
|
 |
|
Re: Upp 2008.1beta2 v2 released
By: bonami on Fri, 11 April 2008 11:57
|
 |
|
Re: Upp 2008.1beta2 v2 released
By: Tom1 on Thu, 10 April 2008 15:52
|
 |
|
Re: Upp 2008.1beta2 v2 released
By: mirek on Thu, 10 April 2008 17:44
|
 |
|
Re: Upp 2008.1beta2 v2 released
By: mr_ped on Tue, 03 June 2008 10:21
|
 |
|
Re: Upp 2008.1beta2 v2 released
By: mirek on Tue, 03 June 2008 13:54
|
 |
|
Re: Upp 2008.1beta2 v2 released
By: zsolt on Tue, 03 June 2008 15:06
|
 |
|
Re: Upp 2008.1beta2 v2 released
By: mirek on Tue, 03 June 2008 18:38
|
Goto Forum:
Current Time: Sun Aug 24 08:14:54 CEST 2025
Total time taken to generate the page: 0.05240 seconds
|