mORMoti18n - LANGUAGE_NONE

Leslie7 · 2022-04-01 05:59:59

Hi,

I am trying to use mORMoti18n the first time.

Versions:
FPC/Lazarus fixes branch -up to date
mORMot 1.18 -up to date

mORMoti18n.pas(324,15) Error: range check error while evaluating constants (255 must be between 0 and 59)

const
/// value stored into a TLanguages enumerate to mark no language selected yet
LANGUAGE_NONE = TLanguages(255);

ab · 2022-04-01 06:12:10

The unit is expected to compile with range-checking OFF.
It is ensured in Synopse.inc conditionals.

Leslie7 · 2022-04-01 06:35:27

I have all units in the project with {$I Synopse.inc}

Checked Synopse.inc, it is the same as in the repository

(********************** Shared Conditionals **********************)

{$ifdef PUREPASCAL}
  {$define NODELPHIASM}
  {$define FPC_OR_PUREPASCAL}
{$else}
{$endif PUREPASCAL}

{$H+} // we use long strings
{$R-} // disable Range checking in our code

There are no range checking defines in project options or in the source files.

Is there something else I can check?

Last edited by Leslie7 (2022-04-01 07:02:07)

Leslie7 · 2022-04-01 08:04:36

I have tried it with a freshly created application in Lazarus and the problem is the same.
Is it possible that the compiler's behavior has changed?

Last edited by Leslie7 (2022-04-01 08:05:38)

ab · 2022-04-01 08:34:16

Sounds like the FPC trunk did change its behavior about range and type checking: it is as if they added some new checks at runtime.
Do have anyone any information about that?

IMHO this is unacceptable because it would break a lot of existing code.
It also break what Delphi and FPC did for decades... weird...
I suspect they will go back and disable this "feature".
Don't use FPC trunk until it is stabilized.

ab · 2022-04-01 08:40:57

It may be linked to https://wiki.freepascal.org/User_Change … elphi_mode

Which is wrong in its statement: Delphi has no trouble with LANGUAGE_NONE = TLanguages(255), from Delphi 1 to latest Delphi 11.1.

Where to report the problem?

Leslie7 · 2022-04-01 08:50:17

This is most likely the cause.
I will do some digging.

AFAIK we are using the same "fixes" branch .
At which commit is yours?

Last edited by Leslie7 (2022-04-01 09:03:20)

ab · 2022-04-01 09:31:41

I am using plain FPC 3.2.0 as in the mORMot documentation.

pvn0 · 2022-04-01 09:56:08

ab wrote:

Where to report the problem?

fpc-devel mailing list is very active and probably will get noticed sooner then creating a git issue.
https://www.freepascal.org/maillist.html

ab · 2022-04-01 11:45:00

I have posted on https://forum.lazarus.freepascal.org/in … ic=58942.0

Leslie7 · 2022-04-03 16:14:31

https://gitlab.com/freepascal.org/fpc/s … 372da876e6
https://gitlab.com/freepascal.org/fpc/s … 372da876e6

This was amazingly fast fix during a weekend from the FPC devs!
I have applied the changes to the fixes branch locally and the range checking errors are gone, but unfortunately there other compiler errors like this with ResStringRec.Identifier:

mORMoti18n.pas(901,19) Error: Illegal qualifier

function LoadResString(ResStringRec: PResStringRec): string;
var Buffer: array [0..4095] of Char; // char = use the generic string type
    i: integer;
begin
  if ResStringRec=nil then begin
    result := '';
    Exit;
  end;
  if ResStringRec.Identifier[<64*1024 then begin

ab · 2022-04-04 06:30:21

mORMoti18n is not FPC/LCL compatible, IIRC.

Leslie7 · 2022-04-04 13:03:17

Ok, got it. The easily predictable question is: any plans?

ab · 2022-04-04 20:41:16

In mormot 2, yes, soon.

Leslie7 · 2022-04-05 14:29:05

Excellent, this is a really important part of the library.

mORMot Open Source

#1 2022-04-01 05:59:59

mORMoti18n - LANGUAGE_NONE

#2 2022-04-01 06:12:10

Re: mORMoti18n - LANGUAGE_NONE

#3 2022-04-01 06:35:27

Re: mORMoti18n - LANGUAGE_NONE

#4 2022-04-01 08:04:36

Re: mORMoti18n - LANGUAGE_NONE

#5 2022-04-01 08:34:16

Re: mORMoti18n - LANGUAGE_NONE

#6 2022-04-01 08:40:57

Re: mORMoti18n - LANGUAGE_NONE

#7 2022-04-01 08:50:17

Re: mORMoti18n - LANGUAGE_NONE

#8 2022-04-01 09:31:41

Re: mORMoti18n - LANGUAGE_NONE

#9 2022-04-01 09:56:08

Re: mORMoti18n - LANGUAGE_NONE

#10 2022-04-01 11:45:00

Re: mORMoti18n - LANGUAGE_NONE

#11 2022-04-03 16:14:31

Re: mORMoti18n - LANGUAGE_NONE

#12 2022-04-04 06:30:21

Re: mORMoti18n - LANGUAGE_NONE

#13 2022-04-04 13:03:17

Re: mORMoti18n - LANGUAGE_NONE

#14 2022-04-04 20:41:16

Re: mORMoti18n - LANGUAGE_NONE

#15 2022-04-05 14:29:05

Re: mORMoti18n - LANGUAGE_NONE

Board footer