Merge #520: Remove fixed TODO

cf05787 Remove fixed TODO (sanket1729)

Pull request description:

  This TODO was actually from 255 where the bug was in cast_unlikely, even though the issue is labeled with or_i (my bad for not checking). There is no code for `cast_unlikely` as it is treated as or_i.

  Addressed in 239efd5

  Fixes #268

ACKs for top commit:
  apoelstra:
    ACK cf05787

Tree-SHA512: c55236979dfaac3e104edd35eb191be20fe45b6e994a886603d9b9c5559f49723fdaa10f73eaee7e7d6f30170f948f609e0369cd65c3569585675d432661f74a

Author: apoelstra

src/miniscript/types/extra_props.rs

@@ -709,7 +709,6 @@ impl Property for ExtData {
                 (None, None) => None,
             },
             timelock_info: TimelockInfo::combine_or(l.timelock_info, r.timelock_info),
-            // TODO: fix elem count dissat bug
             exec_stack_elem_count_sat: cmp::max(
                 l.exec_stack_elem_count_sat,
                 r.exec_stack_elem_count_sat,