Honestly I read and re-read a lot of post on this site regarding to the struct theme. But I need your help.

I have C-style structures

   struct Time
   {
      uint16_t  year; //    year with four digits like 2016
      uint8_t   month; // 1 .. 12
      uint8_t   day; // 1 .. 31
      uint8_t   hour; // 0 .. 23, 24 hour representation
      uint8_t   minute; // 0 .. 59
      uint8_t   second; // 0 .. 59
   };

and class with member function, which implementation is in DLL.

class DeviceInterface {
virtual uint32_t getTime(Time&  time) = 0;
};

where uint32_t value is a status code.

And here is auto generated SWIG C code:

SWIGINTERN PyObject *_wrap_getTime(PyObject *SWIGUNUSEDPARM(self), PyObject *args) {
  PyObject *resultobj = 0;
  DeviceInterface *arg1 = (DeviceInterface *) 0 ;
  Time *arg2 = 0 ;
  void *argp1 = 0 ;
  int res1 = 0 ;
  void *argp2 = 0 ;
  int res2 = 0 ;
  PyObject *swig_obj[2] ;
  uint32_t result;
  
  if (!SWIG_Python_UnpackTuple(args, "getTime", 2, 2, swig_obj)) SWIG_fail;
  res1 = SWIG_ConvertPtr(swig_obj[0], &argp1,SWIGTYPE_p_DeviceInterface, 0 |  0 );
  if (!SWIG_IsOK(res1)) {
    SWIG_exception_fail(SWIG_ArgError(res1), "in method '" "getTime" "', argument " "1"" of type '" "DeviceInterface *""'"); 
  }
  arg1 = reinterpret_cast< DeviceInterface * >(argp1);
  res2 = SWIG_ConvertPtr(swig_obj[1], &argp2, SWIGTYPE_p_Time,  0 );
  if (!SWIG_IsOK(res2)) {
    SWIG_exception_fail(SWIG_ArgError(res2), "in method '" "getTime" "', argument " "2"" of type '" "Time &""'"); 
  }
  if (!argp2) {
    SWIG_exception_fail(SWIG_ValueError, "invalid null reference " "in method '" "getTime" "', argument " "2"" of type '" "Time &""'"); 
  }
  arg2 = reinterpret_cast< Time * >(argp2);
  result = (uint32_t)(arg1)->getTime(*arg2);
  resultobj = SWIG_From_unsigned_SS_int(static_cast< unsigned int >(result));
  return resultobj;
fail:
  return NULL;
}

From code above I don't see what time value, i.e arg2 variable, somehow has been returned. So what I need to write in SWIG interface file to get both status code and time?

CodePudding user response：

You can write a typemap to append the Time output argument. SWIG generates a proxy for the structure that can be generated as needed via SWIG_NewPointerObj(). Full example below:

DeviceInterface.h (minimal implementation)

#include <stdint.h>

struct Time
{
    uint16_t year;   // year with four digits like 2016
    uint8_t  month;  // 1 .. 12
    uint8_t  day;    // 1 .. 31
    uint8_t  hour;   // 0 .. 23, 24 hour representation
    uint8_t  minute; // 0 .. 59
    uint8_t  second; // 0 .. 59
};

class DeviceInterface {
public:
    virtual uint32_t getTime(Time& time) {
        time.year = 2021;
        time.month = 12;
        time.day = 30;
        time.hour = 9;
        time.minute = 47;
        time.second = 30;
        return 0;
    }
};

test.i

%module test

%{
#include "DeviceInterface.h"
%}

%include <stdint.i>

// Do not require an input parameter for Time.
// Instead, SWIG will allocate one.
%typemap(in,numinputs=0) Time& %{
    $1 = new Time;
%}

// After calling the function, process Time as an output.
// Convert the allocated pointer to a SWIG Python wrapper
// and pass ownership to Python.  Python will free the object
// when it goes out of scope.
%typemap(argout) Time& (PyObject* tmp) %{
    // Convert C pointer to SWIG Python wrapper
    tmp = SWIG_NewPointerObj($1, $1_descriptor, SWIG_POINTER_OWN);
    // Append to existing uint32_t return value
    $result = SWIG_Python_AppendOutput($result, tmp);
%}

%include "DeviceInterface.h"

Demo:

>>> import test
>>> d=test.DeviceInterface()
>>> r=d.getTime()
>>> r
[0, <test.Time; proxy of <Swig Object of type 'Time *' at 0x000001ED4F866090> >]
>>> r[1].year
2021
>>> r[1].month
12

If you want to customize the display of the SWIG wrapper for Time to be more human-readable, you can extend the Time object using the following:

%module test

%{
// Added to support the __repr__ implementation
#include <string>
#include <sstream>

#include "DeviceInterface.h"
%}

%include <stdint.i>
%include <std_string.i> // SWIG support for std::string.

%typemap(in,numinputs=0) Time& %{
    $1 = new Time;
%}

%typemap(argout) Time& (PyObject* tmp) %{
    tmp = SWIG_NewPointerObj($1, $1_descriptor, SWIG_POINTER_OWN);
    $result = SWIG_Python_AppendOutput($result, tmp);
%}

// Extend time to suport Python's __repr__.
// It must return a string representing how to display the object in Python.
%extend Time {
    std::string __repr__()
    {
        std::ostringstream ss;
        ss << "Time(year=" << $self->year << ", month=" << (unsigned)$self->month
           << ", day=" << (unsigned)$self->day << ", hour=" << (unsigned)$self->hour
           << ", minute=" << (unsigned)$self->minute << ", second=" << (unsigned)$self->second << ")";
        return ss.str();
    }
}

%include "DeviceInterface.h"

Demo:

>>> import test
>>> d=test.DeviceInterface()
>>> r=d.getTime()
>>> r
[0, Time(year=2021, month=12, day=30, hour=9, minute=47, second=30)]