1. General
  2. Ideas
+1
Under review

allow the Suppress Warnigs comment to be in a separate line

dummzeuch 10 years ago • updated by Roman 10 years ago • 3
Currently the // FI:Wxxxx comment to suppress a warning must be in the line that contains that warning. In some cases, e.g. "W506: Emtpy ELSE block" it would be more convenient to put it into its own line. e.g.:

case _Idx of
0: Result := GetValid(Laser0OutUpMask);
1: Result := GetValid(Laser1OutUpMask);
2: Result := GetValid(Laser2OutUpMask);
else // FI:W506
// ignore
end;
should also allow:
else
// FI:W506 ignore
end;
Currently we already have an // ignore comment there so adding the FI:W506 would not be such and eye sore.
dummzeuch 10 years ago
Just to make it clear: The example above actually was a bug, but W506 masked W521: Return value of function might be undefined.
dummzeuch 10 years ago
The same applies to W519. Please allow us to put that comment into the method body:

procedure TGpsEaDataAccess.UpdateData;
begin
// FI:W519 do nothing
end;

Under review
Roman 10 years ago

Customer support service by UserEcho