Overview
Examples
Screenshots
Comparisons
Applications
Download
Documentation
Tutorials
Bazaar
Status & Roadmap
FAQ
Authors & License
Forums
Funding Ultimate++
Search on this site
Search in forums












SourceForge.net Logo
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 Go to previous messageGo to previous message
mr_ped is currently offline  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. Smile
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); Razz Very Happy

edit: but this is the least important "problem" from those I reported. Smile Look rather at that weird LanguageInfo without return issue.

[Updated on: Thu, 10 April 2008 09:35]

Report message to a moderator

 
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Read Message
Previous Topic: Scripts to build debian packages added to SVN repository
Next Topic: SVN Restructuration
Goto Forum:
  


Current Time: Sun Aug 24 08:14:54 CEST 2025

Total time taken to generate the page: 0.05240 seconds