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

[Dev0.8.1] merged last positive-charged changes, builds on Linux/Windows, works on zand 3.1 #3

Closed
wants to merge 96 commits into from

Conversation

deguix
Copy link

@deguix deguix commented May 1, 2022

No description provided.

positively-charged and others added 28 commits January 2, 2018 13:13
@TDRRmk2
Copy link

TDRRmk2 commented May 18, 2022

I'm not sure whether to merge this or not. Problem is, there's plenty bugs in 0.8.0 that remain completely unfixed in these later commits, and not only that, but they also introduce bugs of their own.

This is one such 0.8.0 bug. #2
I later found more, particularly statically initialized strings similarly breaking on structs as if the strings weren't tagged, none of which appear to be resolved. Same problem with static arrays in scripts, and so on. I haven't done any more testing beyond these in the latest commits so it's entirely possible there's more introduced there.

And in the latest commit there's this: positively-charged#16

It would be great to have all this, but I still haven't figured out any fixes for the 0.8.0 bugs, much less this newer one introduced later on. The segfault fix and the other library changes are good, though.

@deguix deguix closed this by deleting the head repository May 1, 2024
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.

3 participants