Hi

the latest version (177) of fpc in svn does not compile on Mac OS X 10.3.9.

The make all ends with following:

/Users/j/fpc/fpc-svn/compiler/ppcppc -Xs -O1 -n -S2 -Fu/Users/j/fpc/fpc-svn/rtl/units/powerpc-darwin
-Fu/Users/j/fpc/fpc-svn/packages/base/ibase/units/powerpc-darwin -Fu/Users/j/fpc/fpc-svn/packages/base/postgres/units/powerpc-darwin -Fu/Users/j/fpc/fpc-svn/packages/base/mysql/units/powerpc-darwin -Fu/Users/j/fpc/fpc-svn/packages/base/sqlite/units/powerpc-darwin -FE. -FU/Users/j/fpc/fpc-svn/fcl/units/powerpc-darwin -dpowerpc -dRELEASE db.pp
dataset.inc(468,54) Error: Incompatible type for arg no. 1: Got "TDateTimeAlias", expected "Int64"
db.pp(2025) Fatal: There were 1 errors compiling module, stopping db.pp(2025) Error: Compilation aborted
make[3]: *** [db.ppu] Error 1
make[2]: *** [db_all] Error 2
make[1]: *** [fcl_all] Error 2
make: *** [build-stamp.powerpc-darwin] Error 2

>

there is an inconsistency:

  TDateTimeRec = record
    case TFieldType of
      ftDate: (Date: Longint);
      ftTime: (Time: Longint);
      ftDateTime: (DateTime: TDateTimeAlias);
  end;
   TTimeStamp = record
      Time: integer;   { Number of milliseconds since midnight }
      Date: integer;   { One plus number of days since 1/1/0001 }
   end ;

TTimeStamp,Date is integer, and TDateTimeRec.ftDate is longint.

on AMD 64 integerlongint?

Alex

integer is 32 bit.

Peter

IMHO we don't need date on 64 bit:

   TDateTimeRec = record
     case TFieldType of
       ftDate: (Date: integer);
       ftTime: (Time: integer);
       ftDateTime: (DateTime: TDateTimeAlias);
   end;

I don't understand the code exactly. (It seems to assume that the DateTime field contains miliseconds since a certains startdate?) It is about this line:

         TimeStamp := MSecsToTimeStamp(Source.DateTime);

DataTime is a double. MSecsToTimeStamp expects a comp. Maybe a double can be converted to a comp on i386, but not on powerpc.

Vincent.