The test version which you made available in the "https://www.tmssoftware.com/site/forum/forum_posts.asp?TID=13009&title=possible-concurrency-issue-in-fixinsight" thread throws unexpected W525 "Missing INHERITED call in constructor" warnings.
Missing INHERITED call in constructor |
begin
Create(AOwner);
...
end;
The warning is shown for the alternative constructor even though there is nothing to inherit from.
It looks like the rule is applied to every constructor. I can
understand the reasoning behind it. Wouldn't it be better to apply
this rule only when a constructor actually overrides its parent?
Or should we just turn the warning off on this line, because we know that the parent constructor is called indirectly?