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

Fixed the encoding of some files #208

Merged
merged 1 commit into from
Apr 15, 2020
Merged

Fixed the encoding of some files #208

merged 1 commit into from
Apr 15, 2020

Conversation

not-matthias
Copy link
Contributor

Changed the encoding of some files from UTF-16 to UTF-8, because this breaks some tools.

@ayoubfaouzi
Copy link
Owner

Thanks for the conversion! By curiosity, what kind of tools does it break :) ?

@ayoubfaouzi ayoubfaouzi merged commit ede1e00 into ayoubfaouzi:master Apr 15, 2020
@not-matthias
Copy link
Contributor Author

not-matthias commented Apr 15, 2020

@LordNoteworthy I was trying to generate rust bindings (using this). 😄

Unfortunately al-khaser is not really designed for the use as a library, since there are many functions, that'll just print the result instead of returning it.

I'm not sure if changing these functions is something you'd want. If it is, I'm happy to submit another pull request. :)

fengjixuchui added a commit to fengjixuchui/al-khaser that referenced this pull request Apr 18, 2020
Merge pull request ayoubfaouzi#208 from not-matthias/master
@ayoubfaouzi
Copy link
Owner

ahh @not-matthias, rust binding to C libs :) Interesting!

Indeed a lib might be useful for people who integrate al-khaser as a part of their QA testing. Atm I am trying to give some priority for the issues which are open. I will definitely think about it.

Thanks.

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