bugCERTI - Bugs: bug #21399, Some CERTI (obsolete) builtin...

 
 

bug #21399: Some CERTI (obsolete) builtin buffer size may needs adjustment

Submitted by:  Eric NOULARD <erk>
Submitted on:  Tue 23 Oct 2007 01:37:50 PM UTC  
 
Category: CERTISeverity: 3 - Normal
Item Group: Software errorStatus: Fixed
Privacy: PublicAssigned to: ROUSSELOT <rousse>
Open/Closed: ClosedRelease: 3.2.4
Reproducibility: NoneFixed Release: 3.3.0

Add a New Comment(Rich Markup)
   

You are not logged in

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

 

Sun 09 Dec 2007 05:06:04 PM UTC, comment #2:

This issue is currently addressed by Jean-Yves.

Eric NOULARD <erk>
Project Administrator
Thu 25 Oct 2007 09:01:01 PM UTC, comment #1:

I think we may welle
get rid of most (if not ALL) the constant
first 5 or 6 of them are used to size
string represented by a char array.
We may replace those by an std::string
and use

std::string::c_str() when needed
of course this implies dynamic allocation
but those string are used sparingly and as far as saw
not in time critical tight simulation loop.

I will try this.
I linked this bug to other related bug.

Eric NOULARD <erk>
Project Administrator
Tue 23 Oct 2007 01:37:50 PM UTC, original submission:

Current CERTI version
still use deprecated statically sized object
whose size is driven by the following constants
(see certi/include/certi.hh) see hereafter.

Users may need to change the default value
depending on the size of the object exchanged
by his federation.
If any those values are changed, one should
recompile CERTI.

The most important values are:
#define MAX_BYTES_PER_VALUE 150
#define MAX_USER_TAG_LENGTH 32
#define BUFFER_SIZE_DEFAULT 236

The RTI should throw RTIinternalError
if any operation would need more than
those size.

We should implement dynamic sizing of buffer.

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

 

Digest:
   bug dependencies.

Digest:
   bug dependencies.

 

Carbon-Copy List
  • -unavailable- added by erk (Submitted the item)
  • -unavailable- added by erk (Encountered the same bug)
  •  

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

     

     

    Follow 9 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Wed 02 Jul 2008 08:52:08 AM UTCerkStatusNone=>Fixed
      Open/ClosedOpen=>Closed
      Fixed ReleaseUnknown=>3.3.0
    Sat 24 May 2008 10:21:55 AM UTCerkDependencies-=>bugs #23329 is dependent
    Sun 09 Dec 2007 05:06:04 PM UTCerkAssigned toerk=>rousse
    Thu 25 Oct 2007 09:01:01 PM UTCerkAssigned toNone=>erk
      Dependencies-=>Depends on bugs #21423
      Dependencies-=>Depends on bugs #19685
    Tue 23 Oct 2007 01:37:50 PM UTCerkCarbon-Copy-=>Added patricecarle

    Back to the top


    Powered by Savane 3.1-cleanup1