Jak pozbyć się przestarzałej konwersji ostrzeżeń ze stałej ciągu na ostrzeżenia „char *” w GCC?

Pracuję więc nad bardzo dużą bazą kodu, a ostatnio zaktualizowałem do gcc 4.3, która teraz wyzwala to ostrzeżenie:

ostrzeżenie: przestarzała konwersja stałej ciągu na „char *”

Oczywiście poprawnym sposobem jest znalezienie każdej takiej deklaracji

char *s = "constant string";

lub wywołanie funkcji, takie jak:

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

i uczyń je const charwskaźnikami. Oznaczałoby to jednak dotknięcie minimum 564 plików, co nie jest zadaniem, które chcę wykonać w tym momencie. Problem w tej chwili polega na tym, że biegam -werror, więc potrzebuję sposobu, aby stłumić te ostrzeżenia. Jak mogę to zrobić?

Wierzę, że przejście -Wno-write-stringsdo gcc zniesie to ostrzeżenie.

Wszelkie funkcje, do których przekazujesz literały łańcuchowe, "I am a string literal"powinny być używane char const *jako typ zamiast char*.

Jeśli chcesz coś naprawić, napraw to dobrze.

Wyjaśnienie:

Nie można używać literałów łańcuchowych do inicjowania łańcuchów, które zostaną zmodyfikowane, ponieważ są one typu const char*. Casting dala constness aby później je modyfikować jest niezdefiniowane zachowanie , więc trzeba skopiować const char*ciągi charprzez charsię dynamicznie alokowanych char*ciągów w celu ich modyfikacji.

Przykład:

#include <iostream>

void print(char* ch);

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

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

Miałem podobny problem, rozwiązałem go w następujący sposób:

#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);
}

Czy to odpowiedni sposób na rozwiązanie tego? Nie mam dostępu do foodostosowania go do akceptacji const char*, chociaż byłoby to lepsze rozwiązanie (ponieważ foosię nie zmienia m).

Sprawdź obsługę gag Pragma diagnostyczna gcc oraz listę opcji ostrzeżeń -W (zmieniono: nowy link do opcji ostrzeżeń ).

W przypadku gcc możesz użyć #pragma warningdyrektyw, jak wyjaśniono tutaj .

Jeśli jest to aktywna baza kodu, nadal możesz chcieć zaktualizować bazę kodu. Oczywiście ręczne wykonanie zmian nie jest możliwe, ale uważam, że ten problem można rozwiązać raz na zawsze za pomocą jednego sedpolecenia. Jednak tego nie próbowałem, więc weź to z odrobiną soli.

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

Może to nie znaleźć wszystkich miejsc (nawet nie biorąc pod uwagę wywołań funkcji), ale złagodziłoby problem i umożliwiło ręczne wykonanie kilku pozostałych zmian.

Nie mogę użyć przełącznika kompilatora. Więc zmieniłem to:

char *setf = tigetstr("setf");

do tego:

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

Oto, jak to zrobić w pliku, abyś nie musiał modyfikować pliku Makefile.

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

Możesz później ...

#pragma GCC diagnostic pop

Zastąpić

char *str = "hello";

z

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

lub jeśli wywołujesz funkcję:

foo("hello");

zamień to na

foo((char*) "hello");

Zamiast:

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

To działa:

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

W C ++ użyj const_castponiższych instrukcji

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

Test stringjest const string. Możesz więc rozwiązać w ten sposób:

char str[] = "Test string";

lub:

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

Dlaczego nie użyć po prostu rzutowania?

(char*) "test"

Wykonuj rzutowanie od stałego łańcucha do wskaźnika char, tj

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

W C ++ zamień:

char *str = "hello";

z:

std::string str ("hello");

A jeśli chcesz to porównać:

str.compare("HALLO");

Nie rozumiem, jak zastosować twoje rozwiązanie :( - kalmanIsAGameChanger

Pracując z Arduino Sketch, miałem funkcję powodującą moje ostrzeżenia.

Oryginalna funkcja: char StrContains (char * str, char * sfind)

Aby zatrzymać ostrzeżenia, dodałem const przed char * str i char * sfind.

Zmodyfikowano: char StrContains (const char * str, const char * sfind).

Wszystkie ostrzeżenia zniknęły.

zobacz tę sytuację:

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
};

obserwuj pole nazwy, w gcc kompiluje się bez ostrzeżenia, ale w g ++ zrobi to, nie wiem dlaczego.

Można również utworzyć zapisywalny ciąg ze stałej ciągu, wywołując strdup().

Na przykład ten kod generuje ostrzeżenie:

putenv("DEBUG=1");

Jednak następujący kod nie robi tego (tworzy kopię ciągu na stercie przed przekazaniem go do putenv):

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

W tym przypadku (i być może w większości innych) wyłączenie ostrzeżenia jest złym pomysłem - istnieje z jakiegoś powodu. Inna alternatywa (domyślnie domyślna możliwość zapisywania wszystkich ciągów) jest potencjalnie nieefektywna.

Posłuchaj, co mówi ci kompilator!

po prostu użyj opcji -w dla g ++

przykład:

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

Pamiętaj, że nie pozwala to uniknąć wycofania, a raczej zapobiega wyświetlaniu ostrzeżenia na terminalu.

Teraz, jeśli naprawdę chcesz uniknąć utraty wartości, użyj słowa kluczowego const w następujący sposób:

const char* s="constant string";  

Dlaczego nie skorzystasz z -Wno-deprecatedopcji, aby zignorować przestarzałe komunikaty ostrzegawcze?

Problem polega na tym, że korzystam z opcji -Werror

To twój prawdziwy problem, IMO. Możesz wypróbować kilka zautomatyzowanych sposobów przejścia z (char *) na (const char *), ale nałożyłbym na nie pieniądze, nie tylko działając. Będziesz musiał zaangażować człowieka do przynajmniej części pracy. Krótko mówiąc, po prostu zignoruj ​​to ostrzeżenie (ale IMO zostaw je włączone, bo nigdy nie zostanie naprawione) i po prostu usuń -Werror.

Dziękuję wszystkim za pomoc. Wybór z tu i tam pochodzi z tego rozwiązania. To się kompiluje. Nie przetestowałem jeszcze kodu. Może jutro...

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 }

Wiem, że w tablicy timeServer jest tylko 1 element. Ale może być więcej. Reszta została na razie skomentowana, aby zaoszczędzić pamięć.

PyTypeObject PyDict_Type=
{ ...

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

obserwuj pole nazwy, w gcc kompiluje się bez ostrzeżenia, ale w g ++ zrobi to, nie wiem dlaczego.

w gcc (Compiling C) , -Wno-write-string jest domyślnie aktywne.

w g++ (Compiling C++) -Wwrite-string jest domyślnie aktywne

Dlatego zachowuje się inaczej. Dla nas używanie makr Boost_pythongeneruje takie ostrzeżenia. Więc używamy -Wno-write-stringsprzy kompilacji C ++, ponieważ zawsze używamy-Werror

Deklaracja ciągu jako constrozwiąże problem:

char const*s = "constant string";