Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@ndtoan96
Copy link
Contributor

@sxyazi
Copy link
Owner

sxyazi commented Sep 30, 2023

This is so fast! I'm still struggling with Rust ownership, and you've already fixed it, unbelievable! 🙀

BTW, I tried so many ways and couldn't make it work. I'm curious if there's any way to handle it without moving as_bytes to the outer scope?

@ndtoan96
Copy link
Contributor Author

I whink we can call to_string_lossy in upper scope, then change the parameter type to Cow<'a, str>

@sxyazi
Copy link
Owner

sxyazi commented Sep 30, 2023

Merge first, this piece of code will soon be replaced by Lua. 🤣

@sxyazi sxyazi merged commit 8010ae6 into sxyazi:main Sep 30, 2023
@sxyazi sxyazi changed the title Fix Windows build and add github action fix: Windows build and add github action Sep 30, 2023
sxyazi pushed a commit that referenced this pull request Sep 30, 2023
@ndtoan96 ndtoan96 mentioned this pull request Sep 30, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 11, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Build breaks on Windows

2 participants