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

ndk: Add AMidi interface #353

Open
wants to merge 56 commits into
base: master
Choose a base branch
from
Open
Changes from 1 commit
Commits
Show all changes
56 commits
Select commit Hold shift + click to select a range
2daf09c
ndk: Add AMidi interface
paxbun Oct 4, 2022
9456589
ndk: Remove needless explicit lifetimes in midi.rs
paxbun Oct 4, 2022
9cf8581
ndk: Fix mismatching doc comments in midi.rs
paxbun Oct 4, 2022
9f7c9ef
ndk: Remove get_ prefix from midi.rs
paxbun Oct 4, 2022
466bf4a
ndk: Fix doc comments of MidiOutputPort::receive
paxbun Oct 4, 2022
4013f68
ndk-sys: Add feature `midi`
paxbun Oct 5, 2022
ff6697c
ndk: Add feature `midi`
paxbun Oct 5, 2022
02630b0
ndk: Make Midi types implement Send
paxbun Oct 5, 2022
d3d616e
ndk: Update ndk/src/midi.rs
paxbun Oct 5, 2022
4f53506
ndk: Fix mismatching doc comments in midi.rs
paxbun Oct 5, 2022
15b917f
ndk: Fix link to ndk-samples with a permalink
paxbun Oct 5, 2022
5ae1cad
ndk: Replace try_from_primitives in midi.rs with try_from
paxbun Oct 5, 2022
06ebda5
ndk: Make MidiOutputPort::receive return timestamp only with MidiOpco…
paxbun Oct 5, 2022
2c92479
ndk: Add ptr exposing functions to wrappers in midi.rs
paxbun Oct 5, 2022
b2ec4ea
ndk: Raise error when 2 or more msgs received in MidiOutputPort
paxbun Oct 5, 2022
6f262cc
ndk: Remove as_ptr() from structs in midi.rs
paxbun Oct 5, 2022
01877bc
ndk: Add more strict opcode checks to MidiOutputPort
paxbun Oct 5, 2022
48b4416
ndk: Make msg for unexpected opcode in midi.rs more clear
paxbun Oct 5, 2022
aa26247
ndk: Fix vague doc comments in midi.rs
paxbun Oct 5, 2022
8235a5c
ndk: Add mod media_error
paxbun Oct 5, 2022
12df844
ndk: Remove pub use for MediaErrorResult from media/mod.rs
paxbun Oct 5, 2022
34357c7
ndk: Remove pub use from media/mod.rs
paxbun Oct 5, 2022
23b18b0
ndk: Reflect midi.rs changes to CHANGELOG.md
paxbun Oct 5, 2022
c79c8c5
fix: Make MidiDevice::from_java unsafe
paxbun Jun 15, 2023
895350b
Merge branch 'master' into feat/amidi
paxbun Jun 16, 2023
acce571
fix(ndk): Remove all occurrences of ffi::size_t from midi.rs
paxbun Jun 16, 2023
8900c1c
fix(ndk): Remove unnecessary casts from midi.rs
paxbun Jun 16, 2023
95220e7
ndk: Rework media error types
MarijnS95 Jun 15, 2023
85288eb
Merge branch 'ndk-rework-media-error' into feat/amidi
paxbun Jun 16, 2023
cc1c2e3
fix(ndk): Reflect changes in media_error to midi
paxbun Jun 16, 2023
200ab56
Merge branch 'master' into feat/amidi
paxbun Jun 23, 2023
408e460
Update comments in ndk/src/midi.rs
paxbun Aug 10, 2023
9a5744b
Update ndk/src/midi.rs
paxbun Aug 10, 2023
412b6ad
Update ndk/src/midi.rs
paxbun Aug 10, 2023
aa4467e
Update ndk/CHANGELOG.md
paxbun Aug 10, 2023
3cfaad4
fix: Remove impl Send for MidiDevice
paxbun Aug 10, 2023
2e036d2
Fix comments in ndk/src/media_error.rs
paxbun Aug 10, 2023
aa79a19
ndk: Make "midi" feature dependent on "media"
paxbun Aug 10, 2023
0341b4d
fix: Remove redundant Result mapping from midi.rs
paxbun Aug 10, 2023
c8b2b2e
ndk: Add MidiDeviceType::Unknown
paxbun Aug 10, 2023
6044fc2
ndk: Add doc-comments to MidiInputPort
paxbun Aug 16, 2023
f22054d
ndk: Add doc-comments to fields of MidiOpcode::Data
paxbun Aug 16, 2023
734b81b
ndk: Reference the Java Android MIDI docs in the doc-comments
paxbun Aug 16, 2023
7220a9a
ndk: Fix doc-comments of MidiOutputPort::receive
paxbun Aug 16, 2023
241b096
ndk: Drop Send for ndk::midi::Midi*Port
paxbun Aug 17, 2023
d9ef310
ndk: Describe about Java VM thread attachment in the safety section o…
paxbun Aug 17, 2023
31159dc
ndk: Add safe wrapper of midi
paxbun Aug 17, 2023
129edc2
ndk: Fix typos in ndk::midi::safe
paxbun Aug 17, 2023
1ea5b32
Merge branch 'master' into feat/amidi
paxbun Aug 17, 2023
83fe4b2
ndk: Fix clippy warnings in ndk::midi
paxbun Aug 17, 2023
8c64160
ndk: Make examples in ndk::midi compilable
paxbun Aug 17, 2023
5428ca7
ndk: Remove link to SafeMidiDeviceBox from the module-level doc-comme…
paxbun Aug 17, 2023
802ab4a
ndk: Add more detailed description about safety of AMidi functions to…
paxbun Aug 17, 2023
300118e
ndk: Add missing brackets to links to functions in doc-comments in nd…
paxbun Aug 17, 2023
8086a7d
ndk: Enable media_error when "midi" is enabled
paxbun Aug 17, 2023
5707555
Merge remote-tracking branch 'origin/master' into feat/amidi
paxbun Dec 14, 2023
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
Fix comments in ndk/src/media_error.rs
  • Loading branch information
paxbun committed Aug 10, 2023
commit 2e036d2e1f5fe307cd01e0c859ca022f7639c297
3 changes: 2 additions & 1 deletion ndk/src/media_error.rs
Original file line number Diff line number Diff line change
@@ -1,11 +1,12 @@
//! Bindings for NDK media status codes.
//!
//! Also used outside of `libmediandk.so` in `libmidi.so` for example.
//! Also used outside of `libmediandk.so` in `libamidi.so` for example.

#![cfg(feature = "media")]
// The cfg(feature) bounds for some pub(crate) fn uses are non-trivial and will become even more
// complex going forward. Allow them to be unused when compiling with certain feature combinations.
#![allow(dead_code)]

use std::{mem::MaybeUninit, ptr::NonNull};

MarijnS95 marked this conversation as resolved.
Show resolved Hide resolved
use thiserror::Error;
Expand Down