
CONST missing for unmodified dynamic array parameter
Like <not const> string parameters, <not const> dynamic array parameters also generate try/finally block to reference counting.

OK Button should have a shortcut
The OK-Button of the config dialog is the default button, but is doesn't have a shortcut (&OK). It should have one because if the memo has the focus, pressing enter cannot close the dialog.

Option to pick flag missed inherited calls to TInterfacedObject and TObject
Currently the checking of inherited calls is disabled if the class inherits from either TInterfacedObject or TObject (because of too many false positives by some users).
What I am asking for is an option (checkbox?) to enable checking for constructors inherited calls.

RAD 2007 [FixInsight Internal] Fatal parser error $I directive
{$I file.inc};

Extension to "W519 Method '%s' is empty" for methods which only contain "inherited"
While W519 is flagged when a method is empty. I am suggesting that methods the only call "inherited" should (optionally) be flagged as empty.
Examples:
The should be flagged as empty
procedure TMyForm.OnKeyDown(Sender : TObject; var Key: Word; Shift: TShiftState); begin inherited; // will call the inherited version of "OnKeyDown(Sender, Key, Shift);" end;
Should not be flagged as empty:
procedure TMyForm.OnKeyDown(Sender : TObject; var Key: Word; Shift: TShiftState); begin inherited OnSpecialKeyDown(Sender, Key, Shift); end;Likewise, both the following should be flagged as empty:
constructor TMyObject.Create; begin inherited; end;
constructor TMyObject.Create; begin inherited Create; // inherited call method name is the same. end;

CONST missing for unmodified interface parameter 'Foo'
Like <not const> string parameters, <not const> interface parameters also generate try/finally block to reference counting.

W519 can no longer be disabled for a method
With FI 2015.11 W519 (Method is empty) can no longer be disabled with the // FI:W519 comment. I already had it disabled for e.g. the following method like this:
procedure TDummySplashForm.SetAbortVisible(_Value: Boolean); //FI:W519 - do nothing
It worked with the previous version, but now the warning gets displayed again. I also tried to put the comment into the method body but that didn't work either.

I'm sorry for the inconvenience, but a line where this rule triggers has been changed (there was a number of users requests). You need to move your comment on the 'begin' line.
To figure out where to put your ignore-comment, double click on a warning in Messages window and see what line is highlighted.
For instance,
procedure TDummySplashForm.SetAbortVisible(_Value: Boolean); begin //FI:W519 - do nothing // do nothing end;

Empty virtual methods don't flag as "W519 Method '%s' is empty"
When an empty method has been made virtual, it doesn't flag a "W519 Method '%s' is empty".
I would like an option (checkbox?) to enable this warning for virtual methods.

Bug with multi monitor setup

CONST missing for unmodified TGUID parameter
TGUID is too a large data to pass on stack as value.
Customer support service by UserEcho