bugCERTI - Bugs: bug #27711, HLAoctetPairLE and HLAoctetPairBE...

 
 

bug #27711: HLAoctetPairLE and HLAoctetPairBE may be 32bits on unix

Submitted by:  Eric NOULARD <erk>
Submitted on:  Thu 15 Oct 2009 03:56:43 PM UTC  
 
Category: libHLASeverity: 3 - Normal
Item Group: HLA compliance errorStatus: Ready For Test
Privacy: PublicAssigned to: Petr Gotthard <gotthardp>
Open/Closed: ClosedRelease: CVS
Reproducibility: Every TimeFixed Release: 3.3.3

Add a New Comment(Rich Markup)
   

You are not logged in

Please log in, so followups can be emailed to you.

 

Sun 18 Oct 2009 08:24:47 PM UTC, comment #3:

Thanks.

Petr Gotthard <gotthardp>
Project MemberIn charge of this item.
Sun 18 Oct 2009 05:22:27 PM UTC, comment #2:

Go ahead for the commit now.
This one should be harmless :-)

I'm late for 3.3.3 because in the first place I thought
I should be able to fix the "late arriving federate bug"
but it is not that simple to fix.

Eric NOULARD <erk>
Project Administrator
Sun 18 Oct 2009 11:25:52 AM UTC, comment #1:

Thank you, Eric.

The bugfix is ready for commit. I just replaced wchar_t --> uint16_t because I didn't find a word in the standard saying this is a character type or what UTF encoding should be used.

Should I commit now, or wait until 3.3.3 is out?

Petr Gotthard <gotthardp>
Project MemberIn charge of this item.
Thu 15 Oct 2009 03:56:43 PM UTC, original submission:

The OPLE and OPLE are currently stored in a wchar_t whose
size may 4 octets on unix.

see: http://en.wikipedia.org/wiki/Wide_character

from the standard
IEEE 1516.2, Section 4.12.3, Table 23—Basic data representation table
this is wrong those types should be encoded on 2 octets.

I discover this fact when adding test0 to
libHLA/TestHLAtypes.cc

I did checked the failing test code into CVS HEAD
but commenting the failing part because I have no
time yet to fix it.

Petr I do assign to the bug to you,
if you don't have time to fix it just unassign the bug.

Eric NOULARD <erk>
Project Administrator

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach File(s):
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by gotthardp (Posted a comment)
  • -unavailable- added by erk (Submitted the item)
  •  

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 5 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Wed 21 Oct 2009 09:50:45 AM UTCerkOpen/ClosedOpen=>Closed
      Fixed ReleaseUnknown=>3.3.3
    Sun 18 Oct 2009 08:25:03 PM UTCgotthardpStatusFixed=>Ready For Test
    Sun 18 Oct 2009 08:24:47 PM UTCgotthardpStatusIn Progress=>Fixed
    Sun 18 Oct 2009 11:25:52 AM UTCgotthardpStatusNone=>In Progress

    Back to the top


    Powered by Savane 3.1-cleanup1