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

Purge TLS from platform code. #1024

Open
SiegeLord opened this Issue Feb 23, 2019 · 2 comments

Comments

Projects
None yet
2 participants
@SiegeLord
Copy link
Member

SiegeLord commented Feb 23, 2019

Back when I reviewed the modifications made by Factorio, this was the biggest change I noticed. Roughly speaking this means that we access TLS inside the top-level functions, and then pass the values into the platform implementations. I think this will result in both faster and easier-to-understand code.

@dos1

This comment has been minimized.

Copy link
Contributor

dos1 commented Feb 25, 2019

the modifications made by Factorio

Are they available somewhere?

@SiegeLord

This comment has been minimized.

Copy link
Member Author

SiegeLord commented Feb 25, 2019

They are closed source, but perhaps if we asked them nicely they may give us the Allegro directory.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.