Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Add UTF-8 to UCS-2 encoding and Simple File System Protocol support #7

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

Closed
FredrikAleksander wants to merge 3 commits into rust-osdev:master from FredrikAleksander:master

Conversation

Copy link
Contributor

@FredrikAleksander FredrikAleksander commented Mar 25, 2018
edited
Loading

Simple UTF-8 -> UCS-2 routine. Supports all multibyte UTF-8 sequences that are UCS-2 compatable.

Added most of the Simple File System Protocol, besides GetInfo and SetInfo on EFI_FILE_PROTOCOL. Still new to Rust, so not quite sure how to to provide those idiomatically

GabrielMajeri reacted with thumbs up emoji
@FredrikAleksander FredrikAleksander changed the title (削除) Add UTF-8 to UCS-2 encoding (削除ここまで) (追記) Add UTF-8 to UCS-2 encoding and Simple File System Protocol support (追記ここまで) Mar 25, 2018
Copy link
Collaborator

Thanks for contributing to this project.

It's easier for me to review and keep track of pull requests if they focus on one thing at a time.
You should keep your fork's master branch clean, and work on various features on different branches.

It would be great if you could break this pull request into two different pull requests, corresponding to the two features:

  • A new pull request for adding UTF-8 to UCS-2 conversion functions
  • Another one for adding filesystem support

And then we will close this one and discuss those ones.

Copy link
Contributor Author

Sure, I'll make one for ucs-2 first then i'll make anotjer one for SFS when ucs-2 is merged (sfs uses the ucs-2 function

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /