patchCERTI - Patches: patch #7881, RTI1516 enableTimeRegulation...

 
 

You are not allowed to post comments on this tracker with your current authentication level.

patch #7881: RTI1516 enableTimeRegulation endianity bug fix

Submitted by:  Eugene Chemeritskiy <tyz>
Submitted on:  Thu 01 Nov 2012 12:42:00 PM UTC  
 
Category:  CERTI Priority:  5 - Normal
Status:  Need Info Privacy:  Public
Assigned to:  Eric NOULARD <erk> Open/Closed:  Open

Tue 24 Sep 2013 02:26:01 PM UTC, comment #3: 

Yes I did just that (match the ducth comment)
but currently at this place the code contains reasonable things:

                //JvY: TODO Controleren of dit blijft werken met andere tijdsimplementaties
                union ud {
                        double   dv;
                        uint64_t uv;
                } value;
#ifdef HOST_IS_BIG_ENDIAN
                memcpy(&(value.uv), theLookahead.encode().data(), sizeof(double));
#else
                value.uv = CERTI_DECODE_DOUBLE_FROM_UINT64BE(theLookahead.encode().data());
#endif
                double lookAheadTime = value.dv;
                req.setLookahead(lookAheadTime);

Eric NOULARD <erk>
Project AdministratorIn charge of this item.
Tue 24 Sep 2013 02:06:06 PM UTC, comment #2: 

Hi Eric,

I cannot be sure about the version. I guess it is 3.4.2.

A patch file itself might be the problem.
It seems, that I used a patched version of CERTI as the original.

However, you can easily find the truly location of the reported bug in the sources by this Dutch (I guess) comment:

//JvY: TODO Controleren of dit blijft werken met andere tijdsimplementaties

Hope it helps.

Eugene Chemeritskiy <tyz>
Project Member
Tue 24 Sep 2013 01:39:21 PM UTC, comment #1: 

Hi Eugene,

I'm not able to merge this patch since
I cannot find the correspond section of code.

Could you tell me against which version of CERTI it has been
made the current CVS HEAD code looks good to me?

Eric NOULARD <erk>
Project AdministratorIn charge of this item.
Thu 01 Nov 2012 12:42:00 PM UTC, original submission:  

There is an error while getting double lookahead value from RTI1516fedTimeInterval: internal value is encoded to BE and decoded into the host native representation. This is not correct on LE hosts.

Eugene Chemeritskiy <tyz>
Project Member

 

Attached Files
file #26864:  RTI1516ambassador.cpp.patch added by tyz (679B - application/octet-stream)

 

Depends on the following items: None found

Items that depend on this one: None found

 

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

    Do you think this task is very important?
    If so, you can add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

     

     

    Follow 3 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2013-09-24 erk StatusNone => Need Info
        Assigned toNone => erk
    2012-11-01 tyz Attached File- => Added RTI1516ambassador.cpp.patch, #26864

    Back to the top


    Powered by Savane 3.5