Як позбутися від "застарілої конверсії з постійних рядків у попередження" 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 і списком опцій попередження -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є рядком const. Тож ви можете вирішити так:

char str[] = "Test string";

або:

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

Чому б не просто використовувати кастинг типів?

(char*) "test"

Робіть набір тексту від постійної рядки до покажчика char, тобто

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

В C ++ замініть:

char *str = "hello";

з:

std::string str ("hello");

І якщо ви хочете порівняти це:

str.compare("HALLO");

Я не розумію, як застосувати ваше рішення :( - kalmanIsAGameChanger

Працюючи з ескізом Arduino, у мене була функція, яка викликала мої застереження.

Оригінальна функція: 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 }

Я знаю, у масиві timeServer є лише 1 елемент. Але може бути і більше. Решта зараз прокоментували, щоб зберегти пам’ять.

PyTypeObject PyDict_Type=
{ ...

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

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

в gcc (Compiling C) , -Wno-write-рядки активовано за замовчуванням.

в g++ (Compiling C++) -Wwrite-string є активним за замовчуванням

Ось чому є інша поведінка. Для нас, що використовують макроси Boost_pythonгенерує такі попередження. Таким чином, ми використовуємо -Wno-write-stringsпри компілюванні C ++, оскільки завжди використовуємо-Werror

Оголосити рядок як засіб constвирішить проблему:

char const*s = "constant string";