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

Way2js3 - Part 1 - 2nd Attempt #166

Closed
wants to merge 7 commits into from
Closed

Conversation

peterdell
Copy link
Contributor

Hello tebe,
as discussed here the first part of the compatibility changes for the PAS2JS target.

Use regular records instead of variant records
Use TString instead of ^TString for names in TToken

Kind regards, Peter.

Variant records are not really required and cannot be translated into Java/Javascript
The reference is not used as reference but always as value throughout the whole code. Value assignment make memory management easier for the compiler and allow targets which do not support references to elementary types, e.g. Javascript
@zbyti
Copy link
Collaborator

zbyti commented Feb 24, 2025

please re-clone the repository and re-PR.

I tried to clear the history of the bin directory #165

@zbyti
Copy link
Collaborator

zbyti commented Feb 24, 2025

it may be best to wait with cloning until GitHub clears the repository - this can take up to 72 hours.

@zbyti
Copy link
Collaborator

zbyti commented Feb 24, 2025

ok. fork again, or get fresh clone - repo is ready.

@peterdell peterdell closed this Feb 25, 2025
@peterdell
Copy link
Contributor Author

New brach created, #167

@peterdell peterdell deleted the way2js3 branch March 2, 2025 22:55
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