--- Log opened Thu May 13 00:00:53 2021 04:53 -!- harrigan [~harrigan@ptr-93-89-242-235.ip.airwire.ie] has quit [Read error: Connection reset by peer] 04:55 -!- harrigan [~harrigan@ptr-93-89-242-235.ip.airwire.ie] has joined ##miniscript 07:42 < darosior> andytoshi sanket1729 jeremyrubin: so i finally got to finish the patch for the thresh() bug. The compilation to 'or' really isn't nice but at least it makes us compatible with the specs and the C++ implem.. https://github.com/rust-bitcoin/rust-miniscript/pull/242 08:08 < dr-orlovsky> andytoshi: currently rust-bminiscript allows policy/moniscript threshold number to be usize, while in practice I think it should be limited by u16. Any reasons for having it defined with usize? The same applies to Policy::Or probabilities and other usize-containing instructions 12:18 < andytoshi> it's a count 12:18 < andytoshi> so it's a usize 12:32 < sgeisler> Especially since you probably compare t to n, which is implicitly the amount of conditions provided, which should be a usize and casting is annoying. rust-miniscript in general is not optimized for minimal memory footprint 😅 14:58 -!- notmandatory [~notmandat@shindig.notmandatory.org] has quit [Ping timeout: 258 seconds] 15:27 -!- notmandatory [~notmandat@shindig.notmandatory.org] has joined ##miniscript 16:42 -!- shesek [~shesek@unaffiliated/shesek] has quit [Remote host closed the connection] 16:42 -!- shesek [~shesek@164.90.217.137] has joined ##miniscript 16:42 -!- shesek [~shesek@164.90.217.137] has quit [Changing host] 16:42 -!- shesek [~shesek@unaffiliated/shesek] has joined ##miniscript 23:35 -!- jb55 [~jb55@gateway/tor-sasl/jb55] has quit [Remote host closed the connection] 23:35 -!- jb55 [~jb55@gateway/tor-sasl/jb55] has joined ##miniscript --- Log closed Fri May 14 00:00:54 2021