search

Avemey / zexmlss

ZEXMLSS Lazarus/Delphi component for read/write ods, excel xml, xlsx
http://avemey.com/zexmlss/index.php
Other
72 stars 51 forks source link

Delphi 7 compatibility #13

Open DelphiGit opened 8 years ago

DelphiGit commented 8 years ago

Hi, I'm trying the numberformats-branch and in unit "zeodfs.pas" the procedure _ReadAutoFilter() contains two lines with calls to ReplaceStr(). I think it's better to use AnsiReplaceStr(), that way the code compiles in Delphi 7 too. Also, from what I can tell ReplaceStr() itself calls AnsiReplaceStr() with the same parameters. It probably won't result in a huge speed improvement though. :wink:

Avemey commented 8 years ago

Hi! Ok, I will check AnsiReplaceStr/ReplaceStr for FPC2.6/FPC3 and delphi 7/XE in some days.

the-Arioch commented 8 years ago

in xe2 they both present, they in general extended non-Ansi functions to tunnel back to Ansi-xxx ( except some few behaviour-dependent ones like UpperCase which intentionally ASCII-7-only)

Avemey commented 8 years ago

Added defines for FPC + Delphi unicode/Delphi 7.