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 » Developing U++ » UppHub » SysInfo - linking fail
Re: SysInfo [message #49095 is a reply to message #49092] Sun, 17 December 2017 09:35 Go to previous message
koldo is currently offline  koldo
Messages: 3443
Registered: August 2008
Senior Veteran
Dear Klugier

Thank you for your review.

Normally in code that has been programed years ago, after finding that something is not interesting, I usually comment this out for a while instead of removing it just if I realize after that it was interesting.

I am not used with the "auto" type. I think it hides the real type "for good and for evil". I mean, you require a little more effort to put DWORD instead of auto, but when you are reviewing the code, you cannot see the hidden details at a glance with an auto. Even in this case I prefer to put DWORD instead of an int.

About spaces instead of tabs, it is probably because of my editor does not distingish spaces and tabs (and if yes, I would disabled this feature). I do prefer tabs, but I do not thing this is interesting.


Best regards
IƱaki

[Updated on: Fri, 29 December 2017 22:46]

Report message to a moderator

 
Read Message
Read Message
Read Message
Read Message
Previous Topic: firebird error message
Next Topic: ChromiumBrowserExample compilation error due to lack of "cef_atomicops_x86_gcc.h"
Goto Forum:
  


Current Time: Mon Aug 25 20:42:37 CEST 2025

Total time taken to generate the page: 0.05797 seconds