Re: [Lazarus] About a warning of unused parameter.

2018-05-02 Thread Juha Manninen via Lazarus
On Wed, May 2, 2018 at 1:31 PM, Carlos E. R. via Lazarus
 wrote:
> Then I'll wait till that update reaches me :-)

svn up
-- 
___
Lazarus mailing list
Lazarus@lists.lazarus-ide.org
https://lists.lazarus-ide.org/listinfo/lazarus


Re: [Lazarus] About a warning of unused parameter.

2018-05-02 Thread Mattias Gaertner via Lazarus
On Wed, 2 May 2018 12:31:15 +0200
"Carlos E. R. via Lazarus"  wrote:

>[...]
> Then I'll wait till that update reaches me :-)
> 
> I do not want to kill the warning for all files, only those I have reviewed.

Same here. That's why I often use the IDE directive.

Mattias
-- 
___
Lazarus mailing list
Lazarus@lists.lazarus-ide.org
https://lists.lazarus-ide.org/listinfo/lazarus


Re: [Lazarus] About a warning of unused parameter.

2018-05-02 Thread Carlos E. R. via Lazarus
On 2018-05-02 08:04, Mattias Gaertner via Lazarus wrote:
> On Wed, 2 May 2018 02:19:05 +0200 "Carlos E. R. via Lazarus" <> wrote:
> 
>> [...]
>> function TPINGSend.Checksum6(Value: AnsiString): Word;
>> begin
>>   Result := 0;
>> end;
> 
> You can:
> a) add a dummy statement: if Value='' then ;
> b) add an IDE directive: right click on this message in the Lazarus
> Messages window, Hide message by inserting IDE directive
> 
> Mattias> 
>> I see on the contextual menu of the warning that I can silence that
>> warning type on a file (all warnings of that type in that file, not just
>> the ones I know about):
>>
>> {$WARN 5024 off : Parameter "$1" not used}
>>
>> Lazarus writes "on"; tried both. Neither work, the warning still appears.
> 
> It should be "off". I fixed that in trunk.

Then I'll wait till that update reaches me :-)

I do not want to kill the warning for all files, only those I have reviewed.

-- 
Cheers / Saludos,

Carlos E. R.
(from 42.3 x86_64 "Malachite" at Telcontar)



signature.asc
Description: OpenPGP digital signature
-- 
___
Lazarus mailing list
Lazarus@lists.lazarus-ide.org
https://lists.lazarus-ide.org/listinfo/lazarus


Re: [Lazarus] About a warning of unused parameter.

2018-05-02 Thread Mattias Gaertner via Lazarus
On Wed, 2 May 2018 02:19:05 +0200
"Carlos E. R. via Lazarus"  wrote:

>[...]
> function TPINGSend.Checksum6(Value: AnsiString): Word;
> begin
>   Result := 0;
> end;

You can:
a) add a dummy statement: if Value='' then ;
b) add an IDE directive: right click on this message in the Lazarus
Messages window, Hide message by inserting IDE directive

Mattias> 
> I see on the contextual menu of the warning that I can silence that
> warning type on a file (all warnings of that type in that file, not just
> the ones I know about):
> 
> {$WARN 5024 off : Parameter "$1" not used}
> 
> Lazarus writes "on"; tried both. Neither work, the warning still appears.

It should be "off". I fixed that in trunk.

Mattias
-- 
___
Lazarus mailing list
Lazarus@lists.lazarus-ide.org
https://lists.lazarus-ide.org/listinfo/lazarus


[Lazarus] About a warning of unused parameter.

2018-05-01 Thread Carlos E. R. via Lazarus
Hi,

I'm experimenting with doing "ping host" on Lazarus using the Synapse
library (http://synapse.ararat.cz/doku.php/download). The code on my
side is very simple, I just have to "use pingsend", then basically
ping.ping('some host');

I have a little nuisance: building the included library produces a ton
of warnings. This thing was last updated on 2012 (release no. 40).

For instance, this one:

pingsend.pas(137,24) Hint: Parameter "Value" not used


The code is different on Windows or Linux, by using {$IFDEF MSWINDOWS}
clauses. On Linux, the effective code is;

function TPINGSend.Checksum6(Value: AnsiString): Word;
begin
  Result := 0;
end;

So the warning is true, but I don't see what I can do about it without
changing the function definition (I could just not call the function in
Linux). I can do:

  Result := 0;
  Value:='';

but that just changes the warning to:

pingsend.pas(137,24) Hint: Value parameter "Value" is assigned but never
used

which is of course true.



So I wonder if there is something to silence that particular warning
alone. Say, yes, I know about that one, shut up.


I see on the contextual menu of the warning that I can silence that
warning type on a file (all warnings of that type in that file, not just
the ones I know about):

{$WARN 5024 off : Parameter "$1" not used}

Lazarus writes "on"; tried both. Neither work, the warning still appears.

Ideas?

-- 
Cheers / Saludos,

Carlos E. R.
(from 42.3 x86_64 "Malachite" at Telcontar)



signature.asc
Description: OpenPGP digital signature
-- 
___
Lazarus mailing list
Lazarus@lists.lazarus-ide.org
https://lists.lazarus-ide.org/listinfo/lazarus