Attention is currently required from: fixeria, pespin.

View Change

1 comment:

• File pySim/ts_51_011.py:

  • Patch Set #2, Line 366: 'dialing_nr'/ExtendedBcdAdapter(BcdAdapter(Rpad(Bytes(10)))),

    But then it's not really a BcdAdapter, if it simply translates the */# characters? I don't want to b […]

    I thought about the naming when writing the code, and the problem is that it *is* and adapter in the construct world. It also *does* convert BCD. Just not from bytes to BCD but it translates a BCD string to an Extended BCD string.

    So if you really wanted to be clear, you would have to rename the old BcdAdpater to BytesToBcdAdapter, and then the new one to BcdToExtdBcdAdapter or soemthing like that.

    In the end I didn't come up with a clean solution that doesn't need to rename everything.

    I'm also not convinced why we would always only directly convert between bytes and extended BCD like you presented. There may be BCD data which is already in form of a string of digits whcih we want to interpret as extended BCD.

To view, visit change 31066. To unsubscribe, or for help writing mail filters, visit settings.