Skip to content

Add binding for git_message_trailers #749

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.

Sign up for GitHub

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

Merged
alexcrichton merged 20 commits into from
Oct 18, 2021
Merged

Add binding for git_message_trailers #749

Changes from 1 commit
Commits
Show all changes
20 commits
Select commit Hold shift + click to select a range
15a7d4e
Add binding for `git_message_trailers`
LemmingAvalanche Sep 27, 2021
5575c3d
`cargo fmt`
LemmingAvalanche Sep 27, 2021
33ebdcd
Fix pointer mismatch error
LemmingAvalanche Oct 11, 2021
f6fd4b7
Remove a lot of unnecessary lifetimes
LemmingAvalanche Oct 12, 2021
b0015cc
Remove another unnecessary lifetime
LemmingAvalanche Oct 12, 2021
e94008a
Use `Range<usize>` instead of `usize`
LemmingAvalanche Oct 12, 2021
e830f7d
`cargo fmt`
LemmingAvalanche Oct 12, 2021
55ea909
Inline one-off struct
LemmingAvalanche Oct 12, 2021
e7b0306
Implement more iterators
LemmingAvalanche Oct 13, 2021
cd4c2db
Undo accidental and unrelated edit
LemmingAvalanche Oct 14, 2021
e450945
Less explicit lifetimes
LemmingAvalanche Oct 15, 2021
a0bd026
Don’t need `std::marker` any more
LemmingAvalanche Oct 15, 2021
34fa7b5
Correct `len(…)`
LemmingAvalanche Oct 15, 2021
023fb92
Remove unnecessary annotation
LemmingAvalanche Oct 15, 2021
16321f3
Implement `size_hint()`
LemmingAvalanche Oct 15, 2021
70b7ebd
Split into “bytes” and “string” iterators
LemmingAvalanche Oct 15, 2021
ed11c76
Remove more lifetimes
LemmingAvalanche Oct 16, 2021
958e378
Docs
LemmingAvalanche Oct 16, 2021
2508506
`cargo fmt`
LemmingAvalanche Oct 16, 2021
f35b68f
Undo accidental and unrelated edit
LemmingAvalanche Oct 14, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
cargo fmt
  • Loading branch information
@LemmingAvalanche
LemmingAvalanche committed Oct 16, 2021
commit 25085063074f16d134bda5e70df498a18e657069
9 changes: 2 additions & 7 deletions src/message.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -46,9 +46,7 @@ pub fn message_trailers_strs(message: &str) -> Result<MessageTrailersStrs, Error
/// Use this function when the message might not be UTF-8-encoded,
/// or if you want to handle the returned trailer key–value pairs
/// as bytes.
pub fn message_trailers_bytes<S: IntoCString>(
message: S,
) -> Result<MessageTrailersBytes, Error> {
pub fn message_trailers_bytes<S: IntoCString>(message: S) -> Result<MessageTrailersBytes, Error> {
_message_trailers(message.into_c_string()?).map(|res| MessageTrailersBytes(res))
}

Expand Down Expand Up @@ -331,10 +329,7 @@ Signed-off-by: Lieutenant \xe2\x28\xa1prout
";

let trailer = message_trailers_bytes(&message4[..]).unwrap();
let expected = (
&b"Signed-off-by"[..],
&b"Lieutenant \xe2\x28\xa1prout"[..],
);
let expected = (&b"Signed-off-by"[..], &b"Lieutenant \xe2\x28\xa1prout"[..]);
let actual = trailer.iter().next().unwrap();
assert_eq!(expected, actual);

Expand Down