1. General
  2. Bugs
0
Not a bug

W519 can no longer be disabled for a method

Anonymous 10 years ago • updated by Roman 10 years ago • 2

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

begin
// do nothing
end;

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.

Answer

Answer
Not a bug
Roman 10 years ago

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;
dummzeuch 10 years ago

Just in case the wrapping above wasn't clear: The comment is to the right of the line containing the procedure declaration, not in the line below that.

Answer
Not a bug
Roman 10 years ago

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;

Customer support service by UserEcho