mORMot Open Source

tbo

Member

Registered: 2015-04-20

Posts: 354

Unit mormot.core.os, {$ifdef} {$else}

Delphi 11.3, mORMot2 GitHub commit 5323 (752e7f6), unit mormot.core.os

I think the second last {$else OSWINDOWS} is too much here.

{$ifdef OSWINDOWS}  // line 1030

{$ifdef UNICODE}    // line 1032
...
{$else}             // line 1039
...
{$endif UNICODE}    // line 1062
...
{$else OSWINDOWS}   // line 1082
...
{$endif OSWINDOWS}  // line 1101

With best regards
Thomas

ab

Administrator

From: France

Registered: 2010-06-21

Posts: 14,713

Website

Re: Unit mormot.core.os, {$ifdef} {$else}

I don't think so.

For windows, there is the standard SystemInfo:TSystemInfo global variable with the Win32 API fields.

For POSIX/Linux, there is a fake SystemInfo: record with minimal information.

tbo

Member

Registered: 2015-04-20

Posts: 354

Re: Unit mormot.core.os, {$ifdef} {$else}

I don't think so.

You are right. When I look at it in Delphi 11.3, it is highlighted correctly. I had looked at it with another editor and there the display was misleading. I think the spelling {$else OSWINDOWS} confused me because I just write {$else}. Sorry for the wrong warning.

With best regards
Thomas