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

«Silence warnings»? Really?

DimonSoft posted 1 year ago in General
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 posted 1 year ago
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 posted 1 year ago
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.
DimonSoft posted 1 year ago
Thanks for your reply, I feel much better now :)
ansgar posted 1 year ago
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.