TransmitterPdu has double extraction of RadioID

[arrowsmith]

Hi again Don @brutzman,

I have noticed the TransmitterPdu class extracts the RadioID field (ref) once inside the header field (see RadioCommsHeader class), and again separately in the radioNumber field. this causes an unnecessary shift of unmarshalling by 2 bytes. Removing it from the RadioCommsHeader class fixes the problem, but so does commenting out the local field to the TransmitterPdu class, and redirecting all calls to get/set of the RadioID to the embedded RadioCommsHeader instance's field. I'm not sure which way you want to go.

Let me know if you're the best POC, or if there's a preferred approach to raising these issues.

[arrowsmith]

Just in case, I'll tag the other repo members for awareness: @terry-norbraten / @blueoak / @leif81 .

[brutzman]

We think this is probably fixed by integration of pull request... Recommend adding (or providing) a test for ongoing confirmation.