On Thu, Aug 08, 2013 at 12:38:54PM +0200, Jacob Erlbeck wrote:
all looks good. thank you for the patches. Some cosmetic nitpick.
> +
> + OSMO_ASSERT(!strcmp(encoded_hex, osmo_hexdump_nospc(coded, y)));
we tend to use strcmp == 0.
> +
> gsm_7bit_decode_ussd(decoded, coded, y * 8 / 7);
> y = strlen(decoded);
> printf("decoded = %s\n\n", osmo_hexdump((uint8_t *)decoded, y));
> +
> + OSMO_ASSERT(!strncmp(text, decoded, strlen (text)));
strlen(text) e.g. no space between the opening parethesis
> + OSMO_ASSERT(!strcmp(appended_after_decode, decoded + strlen (text)));
I am going to apply all three patches now.
