distal-attribute
distal-attribute
distal-attribute
distal-attribute

«Silence warnings»? Really?

User, date Message
Written by DimonSoft
9 months ago
Category: General
2 posts since Thu, 18 Jul 13
I'm a bit frightened by the description of build 4464. It says:

* Silence various compiler warnings about non initialized variables

AFAIK, HeidiSQL is written in Delphi. Delphi compiler NEVER gives false warnings. Sure, there are certain types of warnings that can be safely ignored (like platform warnings), but warnings about uninitialized variables SHOULD ALWAYS be removed by FIXING the code. Even if it seems that ignoring the warning is safe, you'll have big troubles in the future, when something changes.

Hope Heidi-developers read the forum. Guys, have you really turned off uninitialized-variables-checking or was that a set of code fixes?
Written by ansgar
9 months ago
4801 posts since Fri, 07 Apr 06
Calm down, everything's peachy...

You should have dived into the file diffs, as my commit message was misleading. Of course I fixed the code, and did not just silence these warnings by some compiler switch.

And yes, I'm constantly reading the forum, and I am leaving replies to nearly all reasonably written threads. There are a few people like kalvaro and jfalch helping out here. Thanks to both of them btw! Since at least 5 years, this takes me at least 20 to 30 minutes per day.
Written by ansgar
9 months ago
4801 posts since Fri, 07 Apr 06
Ah, forgot to mention: There is one type of warning I *did* silence per compiler switch: platform warnings. They are obsolete and just spamming my build log.
Written by DimonSoft
9 months ago
2 posts since Thu, 18 Jul 13
Thanks for your reply, I feel much better now :)
Written by ansgar
9 months ago
4801 posts since Fri, 07 Apr 06
HehesmileThanks for your hint. I'll try to be more exact when writing commit comments.
 

Please login to leave a reply, or register at first.