«Silence warnings»? Really?

[expired user #7155]'s profile image [expired user #7155] posted 11 years ago in General Permalink
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?
ansgar's profile image ansgar posted 11 years ago Permalink
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.
ansgar's profile image ansgar posted 11 years ago Permalink
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.
[expired user #7155]'s profile image [expired user #7155] posted 11 years ago Permalink
Thanks for your reply, I feel much better now :)
ansgar's profile image ansgar posted 11 years ago Permalink
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.