Comment # 5
on bug 8243
from Tyson Key
(In reply to comment #4)
> Comment on attachment 9847 [details]
> A version of the patch with fixed brackets
>
> Can the "commands" be switched to a single value_string, so there can just
> be a single col_set_str(pinfo->cinfo, COL_INFO, val_to_str(...)) then
> followed by a switch statement that dissects known commands further?
I assume that you're thinking of collapsing the commands, and responses tables
into a single opcodes one, too? (As suggested previously).
If so, I'll have a play with the code, and try to make a revised patch.
You are receiving this mail because:
- You are watching all bug changes.