swig, pass каретки без указания размеров - PullRequest
Новая
       8

swig, pass каретки без указания размеров

1 голос
/ 09 июня 2019

Это часть кода, который я хочу обернуть: 

typedef unsigned char UINT8;
...
void fn(UINT8 data[]);

В файл .i я включил: 

From 8.2.2 carrays.i:
%array_class(UINT8, UINT8Array)

Для поддержки операции записи в массив: 

%typemap(in) UINT8[ANY] ($1_basetype temp[$1_dim0]) {
    int i;
    if (!PySequence_Check($input)) {
        PyErr_SetString(PyExc_ValueError,"Expected a sequence as input");
        return NULL;
    }
    if (PySequence_Length($input) != $1_dim0) {
        PyErr_SetString(PyExc_ValueError,"Input sequence size incorrect, should have $1_dim0 ints");
        return NULL;
    }
    for (i = 0; i < $1_dim0; i++) {
        PyObject *o = PySequence_GetItem($input,i);
        if (PyNumber_Check(o)) {
            temp[i] = ($1_basetype)PyInt_AsLong(o);
            Py_DECREF(o);
        } else {
            Py_XDECREF(o);
            PyErr_SetString(PyExc_ValueError,"Input sequence elements must be numbers");      
            return NULL;
        }
    }
    $1 = temp;
}

затем при компиляции мой код завершается ошибкой: 

example_wrap.cxx:40:15: error: storage size of ‘temp2’ isn’t known
   UINT8 temp2[] ;

Обратите внимание, что упаковка работает, если моя функция выглядит так:

void fn (данные UINT8 [10]);

void fn (данные UINT8 *);

Есть ли способ, которым я могу сказать swig, когда есть UINT8 [], чтобы чертить его как UINT8 *. Примерно так: 

%typemap(in) UINT8[] ($1_basetype *temp) {

Спасибо, Пабло

1 Ответ

1 голос
/ 12 июня 2019

В комментариях ОП указано, что параметр функции имеет фиксированный размер.Давайте предположим, что заголовочный файл выглядит следующим образом: 

#ifdef _WIN32
#   define API __declspec(dllexport)
#else
#   define API
#endif

typedef unsigned char UINT8;

API void fn(UINT8 data[]); // data must be length 300

Карта типов OP работает, но не знает размер.Чтобы сообщить SWIG о наличии фиксированного размера и при этом использовать преимущества универсальной карты типов, можно использовать следующее: 

%module test

%{
#include "test.h"

// Implementation...this would normally be in a .cpp file and linked in.
// Note to see the result an argout typemap would be needed too...
API void fn(UINT8 data[]) {
    for(int i = 0; i < 300; ++i)
        data[i] += 1;
}
%}

// OP's original generalized array typemap...
%typemap(in) UINT8[ANY] ($1_basetype temp[$1_dim0]) {
    int i;
    if (!PySequence_Check($input)) {
        PyErr_SetString(PyExc_ValueError,"Expected a sequence as input");
        return NULL;
    }
    if (PySequence_Length($input) != $1_dim0) {
        PyErr_SetString(PyExc_ValueError,"Input sequence size incorrect, should have $1_dim0 ints");
        return NULL;
    }
    for (i = 0; i < $1_dim0; i++) {
        PyObject *o = PySequence_GetItem($input,i);
        if (PyNumber_Check(o)) {
            temp[i] = ($1_basetype)PyInt_AsLong(o);
            Py_DECREF(o);
        } else {
            Py_XDECREF(o);
            PyErr_SetString(PyExc_ValueError,"Input sequence elements must be numbers");
            return NULL;
        }
    }
    $1 = temp;
}

// Assume there are lots of other functions in test.h,
// but we don't want to wrap the function that requires
// a fixed-sized array and doesn't declare the size.
%ignore fn;

// Process test.h, but skip wrapping fn.
%include "test.h"

// This "un-ignores" the function and provides a replacement
// that declares the fixed size.
// See http://www.swig.org/Doc3.0/SWIGDocumentation.html#SWIG_rename_ignore
%rename("%s") fn;
void fn(UINT8 temp[300]);

Демо: 

>>> import test
>>> test.fn([1,2,3])
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
ValueError: Input sequence size incorrect, should have 300 ints
>>> test.fn([1]*300)  # This works
>>> test.fn('a'*300)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
ValueError: Input sequence elements must be numbers
...