-
Notifications
You must be signed in to change notification settings - Fork 27
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
223 implement assetsread and assetsread bytes #225
Conversation
Codecov Report
@@ Coverage Diff @@
## main #225 +/- ##
==========================================
+ Coverage 74.76% 74.85% +0.08%
==========================================
Files 85 87 +2
Lines 4993 5034 +41
==========================================
+ Hits 3733 3768 +35
- Misses 1174 1180 +6
Partials 86 86
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm, one comment about efficiency
}; | ||
let mut result = String::new(); | ||
for byte in src_file_bytes.iter() { | ||
result.push(*byte as char); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this seems sooooooo slow lol, is there no faster way?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure this is faster.
little harder to read but:
let res = src_file_bytes.iter().map(|x| *x as u32).collect::<Vec<u32>>();
* Impl, test, and docs. * read binary impl, test, and docs. * update lib test. * Fix test? * Might be faster.
What type of PR is this?
/kind feature
/kind eldritch-function
What this PR does / why we need it:
This PR provides the ability to read files from memory and be used by in memory processes.
Which issue(s) this PR fixes:
Fixes #223