Bagaimana cara menyingkirkan `konversi yang ditinggalkan dari string konstan ke peringatan 'char *'` di GCC?

Jadi saya sedang mengerjakan basis kode yang sangat besar, dan baru-baru ini ditingkatkan menjadi gcc 4.3, yang sekarang memicu peringatan ini:

peringatan: konversi yang tidak digunakan lagi dari string konstan ke 'char *'

Jelas, cara yang benar untuk memperbaikinya adalah dengan menemukan setiap deklarasi suka

char *s = "constant string";

atau panggilan fungsi seperti:

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

dan membuat mereka const charpetunjuk. Namun, itu berarti menyentuh 564 file, minimum, yang bukan tugas yang ingin saya lakukan pada saat ini. Masalahnya sekarang adalah saya sedang berlari -werror, jadi saya perlu cara untuk meredam peringatan ini. Bagaimana saya bisa melakukan itu?

Saya percaya melewatinya -Wno-write-stringske gcc akan menekan peringatan ini.

Fungsi apa pun yang Anda "I am a string literal"gunakan string literal harus digunakan char const *sebagai jenis alih-alih char*.

Jika Anda akan memperbaiki sesuatu, perbaiki dengan benar.

Penjelasan:

Anda tidak dapat menggunakan string literal untuk menginisialisasi string yang akan dimodifikasi, karena mereka bertipe const char*. Casting pergi constness untuk kemudian memodifikasi mereka adalah perilaku undefined , sehingga Anda harus menyalin Anda const char*string chardengan charmenjadi dinamis dialokasikan char*string dalam rangka untuk mengubah mereka.

Contoh:

#include <iostream>

void print(char* ch);

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

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

Saya memiliki masalah yang sama, saya menyelesaikannya seperti ini:

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

Apakah ini cara yang tepat untuk menyelesaikan ini? Saya tidak memiliki akses untuk foomengadaptasikannya untuk menerima const char*, meskipun itu akan menjadi solusi yang lebih baik (karena footidak berubah m).

Lihat dukungan Pragma Diagnostik gcc , dan daftar opsi peringatan -W (diubah: tautan baru ke opsi peringatan ).

Untuk gcc, Anda dapat menggunakan #pragma warningarahan seperti yang dijelaskan di sini .

Jika basis kode aktif, Anda mungkin masih ingin meningkatkan basis kode. Tentu saja, melakukan perubahan secara manual tidak layak tetapi saya percaya bahwa masalah ini dapat diselesaikan sekali dan untuk semua dengan satu sedperintah tunggal . Saya belum mencobanya, jadi ambil yang berikut dengan sebutir garam.

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

Ini mungkin tidak menemukan semua tempat (bahkan tidak mempertimbangkan panggilan fungsi) tetapi itu akan mengurangi masalah dan memungkinkan untuk melakukan beberapa perubahan yang tersisa secara manual.

Saya tidak bisa menggunakan saklar kompiler. Jadi saya telah mengubah ini:

char *setf = tigetstr("setf");

untuk ini:

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

Berikut ini cara melakukannya dalam file, jadi Anda tidak perlu memodifikasi Makefile Anda.

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

Anda kemudian dapat ...

#pragma GCC diagnostic pop

Menggantikan

char *str = "hello";

dengan

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

atau jika Anda memanggil fungsi:

foo("hello");

ganti ini dengan

foo((char*) "hello");

Dari pada:

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

Ini bekerja:

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

Di C ++, gunakan const_castseperti di bawah ini

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

Test stringadalah string const. Jadi Anda bisa menyelesaikannya seperti ini:

char str[] = "Test string";

atau:

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

Mengapa tidak menggunakan tipe casting saja?

(char*) "test"

Apakah typecasting dari string konstan ke pointer char yaitu

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

Di C ++, Ganti:

char *str = "hello";

dengan:

std::string str ("hello");

Dan jika Anda ingin membandingkannya:

str.compare("HALLO");

Saya tidak mengerti bagaimana cara menerapkan solusi Anda :( - kalmanIsAGameChanger

Bekerja dengan Arduino Sketch, saya memiliki fungsi yang menyebabkan peringatan saya.

Fungsi asli: char StrContains (char * str, char * sfind)

Untuk menghentikan peringatan, saya menambahkan const di depan karakter * dan karakter *.

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

Semua peringatan hilang.

lihat situasi ini:

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

perhatikan bidang nama, di gcc kompilasi tanpa peringatan, tetapi di g ++ itu akan, saya tidak tahu mengapa.

Anda juga dapat membuat string yang dapat ditulis dari konstanta string dengan memanggil strdup().

Misalnya, kode ini menghasilkan peringatan:

putenv("DEBUG=1");

Namun, kode berikut ini tidak (itu membuat salinan string di heap sebelum meneruskannya ke putenv):

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

Dalam hal ini (dan mungkin di sebagian besar lainnya) mematikan peringatan adalah ide yang buruk - itu ada karena suatu alasan. Alternatif lain (membuat semua string dapat ditulis secara default) berpotensi tidak efisien.

Dengarkan apa yang dikatakan kompiler!

cukup gunakan opsi -w untuk g ++

contoh:

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

Ingat ini tidak menghindari penghentian melainkan mencegah menampilkan pesan peringatan di terminal.

Sekarang jika Anda benar-benar ingin menghindari penghentian penggunaan kata kunci const seperti ini:

const char* s="constant string";  

Mengapa Anda tidak menggunakan -Wno-deprecatedopsi ini untuk mengabaikan pesan peringatan yang sudah usang?

Masalahnya sekarang adalah saya menjalankan dengan -Werror

Ini adalah masalah Anda yang sebenarnya, IMO. Anda dapat mencoba beberapa cara otomatis untuk berpindah dari (char *) ke (const char *) tetapi saya akan menggunakan uang untuk itu tidak hanya bekerja. Anda harus melibatkan manusia setidaknya untuk beberapa pekerjaan. Untuk jangka pendek, abaikan saja peringatan (tapi IMO biarkan, atau tidak akan pernah diperbaiki) dan cukup hapus -Werror.

Terima kasih atas bantuannya. Memilih dari sana-sini solusi ini. Kompilasi ini bersih. Belum menguji kodenya. Besok mungkin...

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 }

Saya tahu, hanya ada 1 item dalam array timeServer. Tetapi mungkin ada lebih banyak. Sisanya dikomentari untuk saat ini untuk menghemat memori.

PyTypeObject PyDict_Type=
{ ...

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

perhatikan bidang nama, di gcc kompilasi tanpa peringatan, tetapi di g ++ itu akan, saya tidak tahu mengapa.

di gcc (Compiling C), -Wno-write-string aktif secara default.

di g++ (Compiling C++)-Wwrite-string aktif secara default

Inilah sebabnya mengapa ada perilaku yang berbeda. Bagi kami menggunakan makro Boost_pythonmenghasilkan peringatan seperti itu. Jadi kami menggunakan -Wno-write-stringsketika mengkompilasi C ++ karena kami selalu menggunakan-Werror

Nyatakan string sebagai constkehendak memecahkan masalah:

char const*s = "constant string";