PyArg_ParseTupleAndKeywords выдает предупреждение: ISO C ++ запрещает преобразовывать строковую константу в 'char *' '[-Wwrite-strings] - PullRequest
0 голосов
/ 10 июня 2019

Поиск примеров использования PyArg_ParseTupleAndKeywords Я нашел следующие вопросы:

  1. Как можно использовать PyArg_ParseTupleAndKeywords для анализа кортежа с необязательными аргументами и ключевыми словами?
  2. Как работает PyArg_ParseTupleAndKeywords?

Они оба используют такие вещи, как static char* kwlist[] = {"a", "b", NULL}

static int PyClass_init(PyClass* self, PyObject* args, PyObject* kwargs) {
    char* path;
    char* regex;
    static char* kwlist[] = {"", "", NULL};

    if( !PyArg_ParseTupleAndKeywords( args, kwargs, "s|s", kwlist, &path, &regex ) ) {
        return -1;
    }

    // other code ...
    return 0;
}

Компиляция с помощью language = "c++" на setup.py и сборка с -std=c++11 выдает это предупреждение:

x86_64-linux-gnu-gcc -pthread -DNDEBUG -g -fwrapv -O0 -Wall -g -fstack-protector-strong -Wformat -Werror=format-security -Wdate-time -D_FORTIFY_SOURCE=2 -fPIC -Isource -I/usr/include/python3.6m -c source/test.cpp -o build/temp.linux-x86_64-3.6/source/test.o -O0 -g -ggdb -std=c++11 -fstack-protector-all
source/test.cpp: In function ‘int PyClass_init(PyClass*, PyObject*, PyObject*)’:
source/test.cpp:41:42: warning: ISO C++ forbids converting a string constant to ‘char*’ [-Wwrite-strings]
   static char* kwlist[] = {"a", "b", NULL};
                                          ^

В поисках этой ошибки я нашел этот вопрос Почему преобразование из строковой константы в 'char *' допустимо в C, но недопустимо в C ++ , требуя исправления, но применяя исправление, поскольку static char* const kwlist[] = {"a", "b", NULL}; сохраняет / обновляет предупреждение о появлении ошибки:

source/test.cpp:41:50: warning: ISO C++ forbids converting a string constant to ‘char*’ [-Wwrite-strings]
     static char* const kwlist[] = {"a", "b", NULL};
                                                  ^
source/test.cpp:41:50: warning: ISO C++ forbids converting a string constant to ‘char*’ [-Wwrite-strings]
source/test.cpp:43:89: error: invalid conversion from ‘char* const*’ to ‘char**’ [-fpermissive]
     if( !PyArg_ParseTupleAndKeywords( args, kwargs, "s|s", kwlist, &path, &regex ) ) {
                                                                                         ^
In file included from /usr/include/python3.6m/Python.h:117:0,
                 from source/test.cpp:3:
/usr/include/python3.6m/modsupport.h:17:41: note:   initializing argument 4 of ‘int _PyArg_ParseTupleAndKeywords_SizeT(PyObject*, PyObject*, const char*, char**, ...)’
 #define PyArg_ParseTupleAndKeywords     _PyArg_ParseTupleAndKeywords_SizeT
                                         ^
/usr/include/python3.6m/modsupport.h:17:41: note: in definition of macro ‘PyArg_ParseTupleAndKeywords’
 #define PyArg_ParseTupleAndKeywords     _PyArg_ParseTupleAndKeywords_SizeT
                                         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
error: command 'x86_64-linux-gnu-gcc' failed with exit status 1

Как я могу избавиться от предупреждения, используя эквивалентную конструкцию static char* kwlist[] = {"a", "b", NULL} с C++ 11 при совместимости с требованиями API Python C?


Обновление

После того, как предложил, я попытался static const char* kwlist[] = {"a", "b", NULL}, но PyArg_ParseTupleAndKeywords не принимает его:

source/test.cpp:43:89: error: invalid conversion from ‘const char**’ to ‘char**’ [-fpermissive]
     if( !PyArg_ParseTupleAndKeywords( args, kwargs, "s|s", kwlist, &filepath, &rawregex ) ) {
                                                                                         ^
In file included from /usr/include/python3.6m/Python.h:117:0,
                 from source/test.cpp:3:
/usr/include/python3.6m/modsupport.h:17:41: note:   initializing argument 4 of ‘int _PyArg_ParseTupleAndKeywords_SizeT(PyObject*, PyObject*, const char*, char**, ...)’
 #define PyArg_ParseTupleAndKeywords     _PyArg_ParseTupleAndKeywords_SizeT
                                         ^
/usr/include/python3.6m/modsupport.h:17:41: note: in definition of macro ‘PyArg_ParseTupleAndKeywords’
 #define PyArg_ParseTupleAndKeywords     _PyArg_ParseTupleAndKeywords_SizeT
                                         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
error: command 'x86_64-linux-gnu-gcc' failed with exit status 1

Ответы [ 2 ]

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

Использовать изменяемую пустую строку:

char empty[] = "";
static char* kwlist[] = {empty, empty, NULL};
1 голос
/ 10 июня 2019

Пример кода, из которого вы скопировали, вероятно, C, где предупреждение не так серьезно рассматривается (gcc не сообщает об ошибке в таком коде, где g++ выводит предупреждение даже без параметров предупреждения).

Исправление здесь заключается в добавлении const ключевого слова, например:

static const char* kwlist[] = {"a", "b", NULL};

Ключевое слово const, которое вы пробовали, применялось не к значениям, а к указателям.

Затем, чтобы заставить функцию принять char **, используйте const_cast:

const_cast<char **>(kwlist)

как это:

PyArg_ParseTupleAndKeywords( args, kwargs, "s|s", const_cast<char **>(kwlist), &path, &regex ) ) 

Функция не собирается изменять значения с помощью неявного «контракта» (поскольку это Python API, он не собирается вам лгать), поэтому в данном случае приведение к const в порядке)

PS: Альтернатива, предоставленная , этот другой ответ также работает. Обратите внимание, что использование пустых изменяемых строк не безопаснее, чем приведение к константе, так как если бы вызываемая функция решила записать в пустые строки, это также вызвало бы неопределенное поведение.

...