Re-re-hello Sieghard.
> I propose to add a - possibly empty, but AT LEAST commented! - default
> (else) clause.
You are right.
I will revert the last commit (that disabled the warning for many units) and
re-commit the code with warning on.
And if somebody want to do the fixes adding "else" in
Re-hello Sieghard.
> I propose to add a - possibly empty, but AT LEAST commented! - default
> (else) clause.
Yes I was thinking to do that but it is **lot of work**, +- 500 cases to
fix, sometimes in complicated code.
Fre;D
--
Sent from: http://mseide-msegui-talk.13964.n8.nabble.com/
Hello Sieghard.
I totally agree with you, sometimes the warnings helps.
I can understand that in a case of the variable is part of the procedure,
like this:
procedure test;
var
b: 1..10;
begin
case b of // here warning
1: writeln('hello');
end
end;
Here I understand
Hello fredvs,
you wrote on Fri, 27 Mar 2020 15:04:49 -0700 (MST):
> With last fpc 3.3.1 trunk, there are warnings in case like this:
>
> procedure test;
> var
> b: 1..10;
> begin
> case b of // here warning
> 1: writeln('hello');
> end
> end;
>
> " Warning: (6060) Case
With last fpc 3.3.1 trunk, there are warnings in case like this:
procedure test;
var
b: 1..10;
begin
case b of // here warning
1: writeln('hello');
end
end;
" Warning: (6060) Case statement does not handle all possible cases."
In MSEgui there is lot of code that is
Hello.
Do you have download all the files from:
https://github.com/fredvs/ideU
Click on the green button "Clone or Download".
Then unzip the file and load "ideu/src/ideU.prj"
Also use the last version of msegui:
https://github.com/mse-org/mseide-msegui
Did you try the pre-compiled release of
Hello Fred
I have some errors making new ideU.prj
sourcepage.pas(1469,10) Error: (5000) Identifier not found "fontline"
sourcepage.pas(1470,47) Error: (5000) Identifier not found "fontline"
sourcepage.pas(1472,9) Error: (5000) Identifier not found "backgroundline"
all errors are in
7 matches
Mail list logo
