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

Submitter:  Eugene Chemeritskiy <tyz>
Submitted:  Thu 01 Nov 2012 12:42:00 PM UTC
   
 
Category:  CERTI Priority:  5 - Normal
Status:  Need Info Privacy:  Public
Assigned to:  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>
Group administrator
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>
Group 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>
Group administrator
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>
Group Member

 

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

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)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

     

    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.13-f8d8.
    Corresponding source code