Skip to content

Commit

Permalink
Use rbpf from git now that our fix is merged
Browse files Browse the repository at this point in the history
This means the XMP protocol is tested too.

Signed-off-by: Sean Young <[email protected]>
  • Loading branch information
seanyoung committed May 11, 2024
1 parent eb0b57b commit be49647
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 7 deletions.
2 changes: 1 addition & 1 deletion cir/src/keymap/decode.rs
Original file line number Diff line number Diff line change
Expand Up @@ -60,7 +60,7 @@ impl Keymap {
vec![nfa]
};

// TODO: merge NFAs so we end up with on DFA
// TODO: merge NFAs so we end up with one DFA
Ok(nfa.iter().map(|nfa| nfa.build_dfa(options)).collect())
}

Expand Down
2 changes: 1 addition & 1 deletion irp/Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -31,4 +31,4 @@ cir = { path = "../cir" }
rand = "0.8"
irptransmogrifier = { workspace = true }
aya-obj = { version = "0.1", features = [ "std" ] }
rbpf = "0.2"
rbpf = { git = "https://github.com/qmonnet/rbpf" }
5 changes: 0 additions & 5 deletions irp/tests/bpf_decoding.rs
Original file line number Diff line number Diff line change
Expand Up @@ -26,11 +26,6 @@ fn decode_all() {
let mut rng = rand::thread_rng();

for protocol in &mut protocols {
// TODO: See https://github.com/qmonnet/rbpf/pull/108
if protocol.name.starts_with("XMP") {
continue;
}

println!("trying {}: {}", protocol.name, protocol.irp);

if protocol.name == "NEC-Shirriff" {
Expand Down

0 comments on commit be49647

Please sign in to comment.