tree 5196e8b16ce3a821cb0c64e0ff54ce6703486171
parent 0e0ca82b2e08ffdcc49833f0642bdf0ca41a98b4
author David Benjamin <davidben@google.com> 1649869069 +0200
committer Boringssl LUCI CQ <boringssl-scoped@luci-project-accounts.iam.gserviceaccount.com> 1651268725 +0000

Tidy up how ASN1_STRING_print_ex figures out the type.

Between the lookup table, the multiple layers of reuse of the "type"
variable, it is a little hard to follow what's going on with
ASN1_STRING_print_ex. Replace the lookup table with a switch-case
(implicitly handles the bounds check, and we can let the compiler figure
out the best spelling). Then, rather than returning a "character width",
which doen't represent UTF-8, just use the already-defined MBSTRING_*
constants.

(These changes should be covered by the existing ASN1Test.StringPrintEx
test.)

Change-Id: Ie3b2557bfae0f65db969e90cd0c76bc8ade963d4
Reviewed-on: https://boringssl-review.googlesource.com/c/boringssl/+/52365
Reviewed-by: Bob Beck <bbe@google.com>
Commit-Queue: Bob Beck <bbe@google.com>