[spinel] simplify functions spinel_*_to_cstr() (#6199)

Browse Source

spinel.c provides functions to convert spinel values to printable
strings. These functions are written using huge switch case statements.
This patch replaces these statements with simple loops over arrays
containing the strings.

Thus, the new code takes 3 time less code than the previous. In add, the
value and the string are no more repeated so the new is less error
prone.

Note: I have used the following commands for the conversion:
  sed -nre 's/case SPINEL_CMD_(.*):/SPINEL_COMMAND_CSTR(\1),/p' spinel.c
  sed -nre 's/case SPINEL_PROP_(.*):/SPINEL_PROP_CSTR(\1),/p' spinel.c
  ...

...

This commit is contained in:

Jérôme Pouiller

2021-02-25 07:53:27 -08:00

committed by GitHub

parent ee815fed0b

commit 2195f39e94

1 changed files with 448 additions and 1655 deletions

Show all changes Ignore whitespace when comparing lines Ignore changes in amount of whitespace Ignore changes in whitespace at EOL

Download Patch File Download Diff File Expand all files Collapse all files

src/lib/spinel/spinel.c

+448 -1655

File diff suppressed because it is too large Load Diff