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

Make Delphi Happy, about those conditionals #86

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

TommiPrami
Copy link

@TommiPrami TommiPrami commented Sep 2, 2022

Can't understand why in earth delphi even cares of second condition because of "{$if defined(fpc) and" should be enough,
If there is plane ">=3" then Delphi complains with :

[dcc32 Warning] FLRE.pas(5493): W1023 Comparing signed and unsigned types - widened both operands

hope this works OK in the FPC also

  - Can't understand why in earth delphi even cares of second condition because of {$if defined(fpc) and should be enough, if thee is plau >=3 then Delphi complains with : [dcc32 Warning] FLRE.pas(5493): W1023 Comparing signed and unsigned types - widened both operands, hope this works OK in the FPC also
@TommiPrami
Copy link
Author

It fixes the previous issue I posted: #76

@BeRo1985
Copy link
Owner

Nope, FPC raises then a error then, at least stable FPC versions, while it seems to be fixed in the non-stable FPC dev version. Thus, I would wait here until FPC >=3.4 as stable version will be released,

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

Successfully merging this pull request may close these issues.

2 participants