Unnecessary structs in packet-assa_r3.c?

[Evan Huus <eapache () gmail com>]

In packet-assa_r3.c, there are several structures:

configMap_t
commandDissectorParser_t
commandMfgDissectorParser_t
responseTypeDissectorParser_t
upstreamCommandDissectorParser_t
mfgFieldDissectorParser_t

each with two elements, where the first of those elements is (to the best
I've been able to determine) never used. The structures are used to define
arrays, and the first element of each structure in the array is being set
to the index of that element in the array, and then never referenced again.

I believe all of these structures can be removed, and all of the arrays can
be converted to arrays of the type of the second element in each structure,
which will simplify the code and provide some space savings for the
compiled wireshark binary (the arrays are quite big). I just wanted to
check before I did the work and posted a patch, as the current set-up is
sufficiently weird for me to wonder if I'm missing something.

Thanks,
Evan

___________________________________________________________________________
Sent via:    Wireshark-dev mailing list <wireshark-dev () wireshark org>
Archives:    http://www.wireshark.org/lists/wireshark-dev
Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev
             mailto:wireshark-dev-request () wireshark org?subject=unsubscribe