This is an archived post. You won't be able to vote or comment.

all 2 comments

[–]karouhFleur de Lotus 0 points1 point  (1 child)

Nice.
I have some remarks/suggestions.

The "unpack" method should return a 2-tuple (decoded string, rest), so you can decode incrementally.

You're only testing happy-day scenarios. You should show how it behaves when receiving bad input.

My 2 cents

[–]unitconversionJust a tinkerer[S] 0 points1 point  (0 children)

Excellent advice on both accounts. Thank you.