bugCERTI - Bugs: bug #24420, getRegionToken() and getRegion()...

 
 

bug #24420: getRegionToken() and getRegion() not implemented

Submitter:  Petr Gotthard <gotthardp>
Submitted:  Wed 01 Oct 2008 07:51:02 AM UTC
   
 
Category:  CERTI Severity:  * 2 - Minor
Item Group:  HLA compliance error Status:  In Progress
Privacy:  Public Assigned to:  gotthardp
Open/Closed:  Closed Release:  * CVS
Reproducibility:  Every Time Fixed Release:  * 3.4.0
* Mandatory Fields

Add a New Comment Rich Markup
   

Jump to the original submission

Thu 11 Nov 2010 12:16:49 PM UTC, comment #6: 

From the source code I would bet
this is done.

Closing the bug.

Eric NOULARD <erk>
Group administrator
Fri 03 Oct 2008 07:48:43 AM UTC, comment #5: 

I found the description in the DoD's HLA RTI 1.3 Programmer's Guide version 3.2 (section A.7.27 and A.7.28).

Petr Gotthard <gotthardp>
Group Member
Thu 02 Oct 2008 08:24:22 PM UTC, comment #4: 

I did not touch or look at the DDM code so that
I may not be very responsive...
May be Pierre may be faster than me on this I add him as CC.

Concerning your patch I agree this one seems OK,
my opinion is you may apply it.
However where did you find the spec for the missing

getRegionToken()
getRegion()

Regarding DDM you'll find most of the code in
RTIA/DataDistribution
Concerning memory management you are right deleteRegion
should only be called once.
If ever a federate call it twice then
RTI::RTIambassador::deleteRegion
should throw "RegionNotKnown".

If you go further on this it would be nice to
add may be a "regionTest" to the HLA_TestsSuite which
may enhanced our test suite and validate the
newly implemented API.

However as usual with OSS it's a wish :=)

Eric NOULARD <erk>
Group administrator
Thu 02 Oct 2008 02:27:37 PM UTC, comment #3: 

Hello. I found that PyHLA needs these methods, so I assigned this task to myself. ;-) However, I would appreciate your guidance.

These methods are expected to convert the internal regionHandle to the Region* (and vice versa).

(1) The getRegionToken() is an easy task. Do you agree with the attached patch?

(2) The getRegion() would need a list of all Region*'s. Is this list somewehere available to the federate?
We should also consider memory management. I think that there should be only one Region* instance and thus the deleteRegion() should be called only once, regardless how many getRegion calls were before. Right?
[There will be a similar problems once the attributeValueSet.getregion() calls are implemented. I haven't found any note in HLA1.3 regarding who is supposed to delete that memory.]

(file #16600)

Petr Gotthard <gotthardp>
Group Member
Thu 02 Oct 2008 09:54:55 AM UTC, comment #2: 

Empty implementation is now stored in CVS. A DLC compliant application will compile with CERTI, but fail when one of these methods is invoked.

Petr Gotthard <gotthardp>
Group Member
Wed 01 Oct 2008 01:38:11 PM UTC, comment #1: 
Eric NOULARD <erk>
Group administrator
Wed 01 Oct 2008 07:51:02 AM UTC, original submission:  

The getRegionToken() and getRegion() method need to be implemnted for DLC API compliancy.

See also bug #23188.

Petr Gotthard <gotthardp>
Group Member

 

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

Attach Files:
   
   
Comment:
   

Attached Files
file #16600:  patch.diff added by gotthardp (937B - text/x-patch - getRegionToken() implementation)

 

Depends on the following items: None found

Digest:
   bug dependencies.

 

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

    Follow 7 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2010-11-11 erk Open/ClosedOpen Closed
        Fixed ReleaseUnknown 3.4.0
    2008-11-05 erk Dependencies- bugs #24753 is dependent
    2008-11-05 erk StatusNone In Progress
    2008-10-02 erk Carbon-Copy- Added siron
    2008-10-02 gotthardp Attached File- Added patch.diff, #16600
        Assigned toNone gotthardp

    Back to the top

    Powered by Savane 3.13-758e.
    Corresponding source code