Làm cách nào để loại bỏ chuyển đổi `không dùng nữa từ các chuỗi cảnh báo sang 'char *'` cảnh báo trong GCC?

Vì vậy, tôi đang làm việc trên một cơ sở mã cực lớn và gần đây đã nâng cấp lên gcc 4.3, hiện đang kích hoạt cảnh báo này:

cảnh báo: chuyển đổi không dùng từ hằng chuỗi sang 'char *'

Rõ ràng, cách chính xác để khắc phục điều này là tìm mọi tuyên bố như

char *s = "constant string";

hoặc gọi hàm như:

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

và làm cho chúng const charcon trỏ. Tuy nhiên, điều đó có nghĩa là chạm 564 tệp, tối thiểu, đây không phải là nhiệm vụ tôi muốn thực hiện tại thời điểm này. Vấn đề hiện tại là tôi đang chạy theo -werror, vì vậy tôi cần một số cách để dập tắt những cảnh báo này. Làm thế nào tôi có thể làm điều đó?

Tôi tin rằng việc chuyển qua -Wno-write-stringsgcc sẽ ngăn chặn cảnh báo này.

Bất kỳ chức năng nào mà bạn chuyển qua chuỗi ký tự "I am a string literal"nên sử dụng char const *làm loại thay vì char*.

Nếu bạn định sửa một cái gì đó, hãy sửa nó ngay.

Giải trình:

Bạn không thể sử dụng chuỗi ký tự để khởi tạo các chuỗi sẽ được sửa đổi, vì chúng thuộc loại const char*. Bỏ đi các hằng số để sau đó sửa đổi chúng là hành vi không xác định , do đó bạn phải sao chép các const char*chuỗi của mình charbằng charcác char*chuỗi được phân bổ động để sửa đổi chúng.

Thí dụ:

#include <iostream>

void print(char* ch);

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

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

Tôi đã có một vấn đề tương tự, tôi đã giải quyết nó như thế này:

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

Đây có phải là một cách thích hợp để giải quyết điều này? Tôi không có quyền truy cập để foođiều chỉnh nó để chấp nhận const char*, mặc dù đó sẽ là một giải pháp tốt hơn (vì fookhông thay đổi m).

Kiểm tra hỗ trợ Chẩn đoán thực dụng của gcc và danh sách các tùy chọn cảnh báo -W (đã thay đổi: liên kết mới đến các tùy chọn cảnh báo ).

Đối với gcc, bạn có thể sử dụng các #pragma warningchỉ thị như được giải thích ở đây .

Nếu đó là một cơ sở mã hoạt động, bạn vẫn có thể muốn nâng cấp cơ sở mã. Tất nhiên, thực hiện các thay đổi theo cách thủ công là không khả thi nhưng tôi tin rằng vấn đề này có thể được giải quyết một lần và mãi mãi bằng một sedlệnh duy nhất . Mặc dù vậy, tôi đã không thử nó, vì vậy hãy dùng những hạt muối sau đây.

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

Điều này có thể không tìm thấy tất cả các địa điểm (thậm chí không xem xét các cuộc gọi chức năng) nhưng nó sẽ làm giảm bớt vấn đề và làm cho nó có thể thực hiện một vài thay đổi còn lại bằng tay.

Tôi không thể sử dụng chuyển đổi trình biên dịch. Vì vậy, tôi đã biến điều này:

char *setf = tigetstr("setf");

đến đây:

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

Đây là cách thực hiện nội tuyến trong một tệp, vì vậy bạn không phải sửa đổi Makefile của mình.

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

Bạn có thể sau đó ...

#pragma GCC diagnostic pop

Thay thế

char *str = "hello";

với

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

hoặc nếu bạn đang gọi chức năng:

foo("hello");

thay thế nó bằng

foo((char*) "hello");

Thay vì:

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

Những công việc này:

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

Trong C ++, sử dụng const_castnhư dưới đây

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

Test stringlà chuỗi const. Vì vậy, bạn có thể giải quyết như thế này:

char str[] = "Test string";

hoặc là:

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

Tại sao không chỉ sử dụng kiểu đúc?

(char*) "test"

Thực hiện typecasting từ chuỗi không đổi đến con trỏ char tức là

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

Trong C ++, Thay thế:

char *str = "hello";

với:

std::string str ("hello");

Và nếu bạn muốn so sánh nó:

str.compare("HALLO");

Tôi không hiểu cách áp dụng giải pháp của bạn :( - kalmanIsAGameChanger

Làm việc với Arduino Sketch, tôi có một chức năng gây ra các cảnh báo của mình.

Hàm gốc: char StrContains (char * str, char * sfind)

Để ngăn chặn các cảnh báo, tôi đã thêm const ở phía trước char * str và char * sfind.

Đã sửa đổi: char StrContains (const char * str, const char * sfind).

Tất cả các cảnh báo đã biến mất.

xem tình huống này:

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

xem trường tên, trong gcc nó biên dịch mà không có cảnh báo, nhưng trong g ++, tôi sẽ không biết tại sao.

Bạn cũng có thể tạo một chuỗi có thể ghi từ hằng chuỗi bằng cách gọi strdup().

Chẳng hạn, mã này tạo cảnh báo:

putenv("DEBUG=1");

Tuy nhiên, đoạn mã sau không (nó tạo một bản sao của chuỗi trên heap trước khi chuyển nó tới putenv):

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

Trong trường hợp này (và có lẽ trong hầu hết những người khác) tắt cảnh báo là một ý tưởng tồi - đó là lý do. Sự thay thế khác (làm cho tất cả các chuỗi có thể ghi theo mặc định) có khả năng không hiệu quả.

Hãy lắng nghe những gì trình biên dịch đang nói với bạn!

chỉ sử dụng tùy chọn -w cho g ++

thí dụ:

g ++ -w -o đơn giản.o đơn giản.cpp -lpthread

Hãy nhớ điều này không tránh sự phản đối thay vì nó ngăn hiển thị thông báo cảnh báo trên thiết bị đầu cuối.

Bây giờ nếu bạn thực sự muốn tránh sự phản đối, hãy sử dụng từ khóa const như thế này:

const char* s="constant string";  

Tại sao bạn không sử dụng -Wno-deprecatedtùy chọn để bỏ qua các thông báo cảnh báo không dùng nữa?

Vấn đề bây giờ là tôi đang chạy với -Werror

Đây là vấn đề thực sự của bạn, IMO. Bạn có thể thử một số cách tự động để chuyển từ (char *) sang (const char *) nhưng tôi sẽ đặt tiền cho chúng không chỉ hoạt động. Bạn sẽ phải có một con người tham gia ít nhất một số công việc. Trong thời gian ngắn, chỉ cần bỏ qua cảnh báo (nhưng IMO vẫn tiếp tục hoặc sẽ không bao giờ được sửa) và chỉ cần xóa -Werror.

Cảm ơn tất cả vì sự giúp đỡ. Chọn từ đây và có giải pháp này. Điều này biên dịch sạch. Chưa kiểm tra mã. Ngày mai ... có lẽ ...

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 }

Tôi biết, chỉ có 1 mục trong mảng timeServer. Nhưng có thể có nhiều hơn. Phần còn lại đã được bình luận ra bây giờ để tiết kiệm bộ nhớ.

PyTypeObject PyDict_Type=
{ ...

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

xem trường tên, trong gcc nó biên dịch mà không có cảnh báo, nhưng trong g ++, tôi sẽ không biết tại sao.

trong gcc (Compiling C) , -Wno-write-String được kích hoạt theo mặc định.

trong g++ (Compiling C++) -Wwrite-String được kích hoạt theo mặc định

Đây là lý do tại sao có một hành vi khác nhau. Đối với chúng tôi bằng cách sử dụng macro Boost_pythontạo ra các cảnh báo như vậy. Vì vậy, chúng tôi sử dụng -Wno-write-stringskhi biên dịch C ++ vì chúng tôi luôn sử dụng-Werror

Khai báo chuỗi như constsẽ giải quyết vấn đề:

char const*s = "constant string";