+1
Fixed

AFAIK Record constructors don't need a call to "inherited", yet W504 warning is shown

Krom Stern 10 years ago updated by Roman 10 years ago 8
Under review
Of course, record constructors don't need t call "inherited". And this works pretty well, when I test it. Could you please provide an example of code where FixInsight gives a warning for a record constructor? So I will be able to reproduce it and fix.
unit KM_Points;
interface

type
TKMRect = record
constructor Create;
end;

implementation

{ TKMRect }
constructor TKMRect.Create; //[FixInsight Warning] KM_Points.pas(12): W504 Missing INHERITED call in constructor/destructor
begin
end;

end.
Hm. Interesting. For me it doesn't produce a warning for this code.
OK, I will spend more time debugging this rule.
Delphi XE5 here.
I could not reproduce the issue when created new VCL App and pasted this code into the unit with Form1. Yet on my big project I got ~10 such warnings on different places. Thanks for your response, I'll see if I can test it deeper.
Analyzing new apps (just created) is an another issue
http://fixinsight.userecho.com/topic/632361-new-units-are-not-taken-into-account/
:)

That should not depend on Delphi version. But this is a good point. I'll check it with XE5.