← Back to team overview

kicad-developers team mailing list archive

Re: Python: time_t needs mapping; tentative patch

 

Alright, here is a reworked patch that includes the ideas discussed here

  - Where we used to use time_t in the functions that deal with
timestamps, this is now done with a new typedef timestamp_t that makes
sure the c++ side and swigged Python side agree on the type.
  - this typedef is now in common.h, close to the declaration of
GetNewTimeStamp()
  - The type of this type is 'long' as this makes sure that it is at
least 32 bits. Trying to use something like int32_t did not work well
with swig that seems to be more comfortable by default with the
standard types (but some local Swig expert might know how to do this)

Cheers,
  Henner.

Attachment: timestamp-type.patch.gz
Description: GNU Zip compressed data


Follow ups

References