Minor issue. 1278 specifies that the beam data length is a char

open-dis / open-dis-cpp

C++ implementation of the IEEE-1278.1 Distributed Interactive Simulation (DIS) application protocol v6 and v7

BSD 2-Clause "Simplified" License

91 stars 66 forks source link

Minor issue. 1278 specifies that the beam data length is a char #68

Closed jcassagnol-public closed 2 years ago

jcassagnol-public commented 2 years ago

I'm not certain about the DIS7 implementation, but it looks like the DIS6 UaPDU has a slight problem with the Acoustic Beam Data struct. The beam data length field is listed as a 8 bit value in the 1278 spec so I changed it from a short to a char and adjusted the marshalled size function by one as well.

jcassagnol-public commented 2 years ago

Aw it looks like my commit email screwed up. I'll fix that and then force-push.