bugCERTI - Bugs: bug #25119, neither stdint.h nor cstdint...

 
 

bug #25119: neither stdint.h nor cstdint available on all platforms

Submitted by:  Petr Gotthard <gotthardp>
Submitted on:  Thu 18 Dec 2008 12:41:52 PM UTC  
 
Category: CERTISeverity: 3 - Normal
Item Group: Compilation errorStatus: Ready For Test
Privacy: PublicAssigned to: Petr Gotthard <gotthardp>
Open/Closed: ClosedRelease: CVS
Reproducibility: Every TimeFixed Release: 3.3.2

Add a New Comment(Rich Markup)
   

You are not logged in

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

 

(Jump to the original submission Jump to the original submission)

Thu 18 Dec 2008 08:24:49 PM UTC, comment #6:

Done; IRIX64 cdash success report sumbitted ;-)

latest CERTI compiles under
- Solaris (SunStudio)
- FreeBSD (gcc)
- IRIX64 (gcc)
- Linux (gcc)

Petr Gotthard <gotthardp>
Project MemberIn charge of this item.
Thu 18 Dec 2008 06:37:51 PM UTC, comment #5:

Use inttypes.h on all non-win platforms

note however that my manual says:
The <inttypes.h> header shall include the <stdint.h> header.

so if inttypes.h is available it should fits our needs.

Eric NOULARD <erk>
Project Administrator
Thu 18 Dec 2008 05:37:36 PM UTC, comment #4:

> Whatever the solution,
> I think we should concentrate (when possible)
> platform/compiler specific #ifdef


So:

a) use inttypes.h on all non-win platforms, or

b) use inttypes.h on irix only and keep stdint.h for remaining non-win platforms

Petr Gotthard <gotthardp>
Project MemberIn charge of this item.
Thu 18 Dec 2008 04:14:50 PM UTC, comment #3:

Whatever the solution,
I think we should concentrate (when possible)
platform/compiler specific #ifdef and so on in

certi.hh
or in a to be defined include
libhla_basictypes.hh which would be a public header
included in certi.hh AND in
HLAtypesIEEE1516.hh

most of the current certi.hh definitions
may be put into libhla_basictypes.hh
CERTI (and libCERTI) may depend on libHLA
but we should not introduce dependencies
on libHLA which MUST stay standalone.

Eric NOULARD <erk>
Project Administrator
Thu 18 Dec 2008 04:02:37 PM UTC, comment #2:

> This one was suggested
> http://www.azillionmonkeys.com/qed/pstdint.h


This one is pretty complex. May be a scaled-down solution would be better.

>> How about to use inttypes.h instead?
> is it available on WIN32 or should we use it just for Un*x case?


All stdint.h references in CERTI are Un*x only. Neither stdint.h, nor inttypes.h is available in Visual Studio (2005).

Petr Gotthard <gotthardp>
Project MemberIn charge of this item.
Thu 18 Dec 2008 03:52:17 PM UTC, comment #1:

May be we can ship a "portable stdint header".
With both certi and libHLA:

This one was suggested
http://www.azillionmonkeys.com/qed/pstdint.h
by a CERTI user some time ago:
https://savannah.nongnu.org/bugs/?func=detailitem&item_id=21611

it is under BSD license.
We would have to ship pstdint.h in with our public header.

> How about to use inttypes.h instead?


is it available on WIN32 or should we use it just
for Un*x case?

> I think it's good to avoid ifdef's whenever possible


that's right it make the code uglier.

Eric NOULARD <erk>
Project Administrator
Thu 18 Dec 2008 12:41:52 PM UTC, original submission:

How about to use inttypes.h instead? It's also a standard header and it's available on IRIX.

I think it's good to avoid ifdef's whenever possible, but alternatively we may use #ifdef HAS_INTTYPES_H. [I think it's bad in public .h files like in libHLA because it requires users to run cmake/autoconf.]

Petr Gotthard <gotthardp>
Project MemberIn charge of this item.

 

(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 erk (Posted a comment)
  • -unavailable- added by gotthardp (Submitted the item)
  •  

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

     

     

    Follow 3 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Wed 29 Apr 2009 09:05:39 AM UTCerkOpen/ClosedOpen=>Closed
      Fixed ReleaseNextRelease=>3.3.2
    Thu 18 Dec 2008 08:24:49 PM UTCgotthardpStatusIn Progress=>Ready For Test

    Back to the top


    Powered by Savane 3.1-cleanup1