Merge rust-bitcoin/rust-miniscript#520: Remove fixed TODO

cf0578798e2d774b41f5c8b72628c539ea9e2a6a 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 239efd55387f1d893e6ac4e602ca11454785c375

  Fixes #268

ACKs for top commit:
  apoelstra:
    ACK cf0578798e2d774b41f5c8b72628c539ea9e2a6a

Tree-SHA512: c55236979dfaac3e104edd35eb191be20fe45b6e994a886603d9b9c5559f49723fdaa10f73eaee7e7d6f30170f948f609e0369cd65c3569585675d432661f74a

Author: heap-coder

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,