Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Delphi 7 compatibility #13

Open
DelphiGit opened this issue Jul 29, 2016 · 3 comments
Open

Delphi 7 compatibility #13

DelphiGit opened this issue Jul 29, 2016 · 3 comments

Comments

@DelphiGit
Copy link

DelphiGit commented Jul 29, 2016

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. 😉

@Avemey
Copy link
Owner

Avemey commented Jul 31, 2016

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

@the-Arioch
Copy link
Contributor

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
Copy link
Owner

Avemey commented Aug 2, 2016

Added defines for FPC + Delphi unicode/Delphi 7.

@MtwStark MtwStark mentioned this issue Feb 12, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants