Как избавиться от `устаревшего преобразования из строковой константы в 'char *'` предупреждений в GCC?

Поэтому я работаю над чрезвычайно большой кодовой базой и недавно обновлена ​​до gcc 4.3, который теперь вызывает это предупреждение:

предупреждение: устаревшее преобразование из строковой константы в 'char *'

Очевидно, что правильный способ исправить это - найти каждое объявление как

char *s = "constant string";

или вызов функции как:

void foo(char *s);
foo("constant string");

и сделать их const charуказатели. Однако это означало бы касание как минимум 564 файлов, что не является задачей, которую я хотел бы выполнить в данный момент. Проблема сейчас в том, что я бегу -werror, поэтому мне нужен какой-то способ подавить эти предупреждения. Как я могу это сделать?

Я верю, что переход -Wno-write-stringsк gcc подавит это предупреждение.

Любые функции, в которые вы передаете строковые литералы, "I am a string literal"должны использовать char const *вместо типа char*.

Если вы собираетесь что-то исправить, исправьте это правильно.

Объяснение:

Вы не можете использовать строковые литералы для инициализации строк, которые будут изменены, потому что они имеют тип const char*. Отбрасывая константность , чтобы позже изменить их в неопределенное поведение , так что вы должны скопировать const char*строки charпутем charв динамически выделенных char*строк, чтобы изменить их.

Пример:

#include <iostream>

void print(char* ch);

void print(const char* ch) {
    std::cout<<ch;
}

int main() {
    print("Hello");
    return 0;
}

У меня была похожая проблема, я решил ее так:

#include <string.h>

extern void foo(char* m);

int main() {
    // warning: deprecated conversion from string constant to ‘char*’
    //foo("Hello");

    // no more warning
    char msg[] = "Hello";
    foo(msg);
}

Это подходящий способ решения этой проблемы? У меня нет доступа, чтобы fooадаптировать его для принятия const char*, хотя это было бы лучшим решением (потому fooчто не меняется m).

Проверьте поддержку gcc Diagnostic Pragma и список параметров предупреждения -W (изменено: новая ссылка на параметры предупреждений ).

Для gcc вы можете использовать #pragma warningдирективы, как описано здесь .

Если это активная кодовая база, вы все равно можете обновить кодовую базу. Конечно, внесение изменений вручную невозможно, но я считаю, что эта проблема может быть решена раз и навсегда одной единственной sedкомандой. Я не пробовал, однако, так что возьмите следующее с солью.

find . -exec sed -E -i .backup -n \
    -e 's/char\s*\*\s*(\w+)\s*= "/char const* \1 = "/g' {} \;

Это может не найти все места (даже не принимая во внимание вызовы функций), но это уменьшит проблему и позволит выполнить несколько оставшихся изменений вручную.

Я не могу использовать переключатель компилятора. Итак, я превратил это:

char *setf = tigetstr("setf");

к этому:

char *setf = tigetstr((char *)"setf");

Вот как это сделать в файле, поэтому вам не нужно изменять ваш Makefile.

// gets rid of annoying "deprecated conversion from string constant blah blah" warning
#pragma GCC diagnostic ignored "-Wwrite-strings"

Вы можете потом позже ...

#pragma GCC diagnostic pop

замещать

char *str = "hello";

с

char *str = (char*)"hello";

или если вы вызываете функцию:

foo("hello");

замени это

foo((char*) "hello");

Вместо:

void foo(char *s);
foo("constant string");

Это работает:

void foo(const char s[]);
foo("constant string");

В C ++ используйте const_castкак показано ниже

char* str = const_cast<char*>("Test string");

Test stringконстантная строка Таким образом, вы можете решить так:

char str[] = "Test string";

или:

const char* str = "Test string";
printf(str);

Почему бы просто не использовать приведение типов?

(char*) "test"

Делать типизацию от константной строки до указателя на символ

char *s = (char *) "constant string";

В C ++ замените:

char *str = "hello";

с:

std::string str ("hello");

И если вы хотите сравнить это:

str.compare("HALLO");

Я не понимаю, как применить ваше решение :( - kalmanIsAGameChanger

Работая с Arduino Sketch, у меня была функция, вызывающая мои предупреждения.

Оригинальная функция: char StrContains (char * str, char * sfind)

Чтобы прекратить предупреждения, я добавил const перед char * str и char * sfind.

Изменено: char StrContains (const char * str, const char * sfind).

Все предупреждения ушли.

увидеть эту ситуацию:

typedef struct tagPyTypeObject
{
    PyObject_HEAD;
    char *name;
    PrintFun print;
    AddFun add;
    HashFun hash;
} PyTypeObject;

PyTypeObject PyDict_Type=
{
    PyObject_HEAD_INIT(&PyType_Type),
    "dict",
    dict_print,
    0,
    0
};

смотреть поле имени, в gcc он компилируется без предупреждения, но в g ++ это будет, я не знаю почему.

Вы также можете создать доступную для записи строку из строковой константы, вызвав ее strdup().

Например, этот код генерирует предупреждение:

putenv("DEBUG=1");

Однако следующий код этого не делает (он делает копию строки в куче, прежде чем передать ее putenv):

putenv(strdup("DEBUG=1"));

В этом случае (и, возможно, в большинстве других) отключение предупреждения является плохой идеей - оно есть по причине. Другой вариант (по умолчанию все строки доступны для записи) потенциально неэффективен.

Послушайте, что говорит вам компилятор!

просто используйте опцию -w для g ++

пример:

g ++ -w -o simple.o simple.cpp -lpthread

Помните, что это не предотвращает устаревание, а предотвращает показ предупреждающего сообщения на терминале.

Теперь, если вы действительно хотите избежать устаревания, используйте ключевое слово const:

const char* s="constant string";  

Почему вы не используете -Wno-deprecatedопцию, чтобы игнорировать устаревшие предупреждающие сообщения?

Проблема сейчас в том, что я работаю с -Werror

Это твоя настоящая проблема, ИМО. Вы можете попробовать некоторые автоматизированные способы перехода от (char *) к (const char *), но я бы положил деньги на них, а не просто на работу. Вам придется привлекать человека, по крайней мере, для выполнения некоторых работ. В краткосрочной перспективе просто проигнорируйте предупреждение (но IMO оставьте его включенным, иначе оно никогда не будет исправлено) и просто удалите -Werror.

Спасибо всем за помощь. Выбор здесь и там приходит это решение. Это компилируется чисто. Еще не проверяли код. Может быть, завтра...

const char * timeServer[] = { "pool.ntp.org" }; // 0 - Worldwide 
#define WHICH_NTP            0 // Which NTP server name to use.
...
sendNTPpacket(const_cast<char*>(timeServer[WHICH_NTP])); // send an NTP packet to a server
...
void sendNTPpacket(char* address) { code }

Я знаю, есть только 1 элемент в массиве timeServer. Но могло быть и больше. Остальные были закомментированы, чтобы сохранить память.

PyTypeObject PyDict_Type=
{ ...

PyTypeObject PyDict_Type=
{
  PyObject_HEAD_INIT(&PyType_Type),
                     "dict",
                     dict_print,
                     0,
                     0
}; 

смотреть поле имени, в gcc он компилируется без предупреждения, но в g ++ это будет, я не знаю почему.

в gcc (Compiling C) -Wno-write-strings активен по умолчанию.

в g++ (Compiling C++) -Wwrite-strings активен по умолчанию

Вот почему существует другое поведение. Для нас использование макросов Boost_pythonгенерирует такие предупреждения. Поэтому мы используем -Wno-write-stringsпри компиляции C ++, так как мы всегда используем-Werror

Объявление строки как constрешит проблему:

char const*s = "constant string";