GCCで「非推奨の文字列定数から「char *」への変換」警告を取り除く方法は？

だから私は非常に大きなコードベースに取り組んでいて、最近この警告をトリガーするgcc 4.3にアップグレードしました：

警告：文字列定数から 'char *'への非推奨の変換

明らかに、これを修正する正しい方法は、次のようなすべての宣言を見つけることです

char *s = "constant string";

または次のような関数呼び出し：

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

そしてそれらをconst charポインタにします。ただし、これは少なくとも564個のファイルを操作することを意味し、現時点で実行したい作業ではありません。現在の問題は、私がで実行している-werrorため、これらの警告を抑制する方法が必要です。どうやってやるの？

-Wno-write-stringsgccに渡すと、この警告が抑制されると思います。

文字列リテラルを渡す関数はすべて、の代わりにタイプとして"I am a string literal"使用する必要があります。char const *char*

何かを修正する場合は、正しく修正してください。

説明：

文字列リテラルはタイプなので、文字列リテラルを使用して変更される文字列を初期化することはできませんconst char*。後でそれらを修正するためにconst性を離れてキャストすると、ある未定義の動作、あなたのコピーする必要がありますので、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説明するようなディレクティブを使用できます。

それがアクティブなコードベースである場合でも、コードベースをアップグレードしたい場合があります。もちろん、変更を手動で実行することは現実的ではありませんが、この問題は1つの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 stringconst文字列です。したがって、次のように解決できます。

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）

警告を止めるために、char * strとchar * sfindの前にconstを追加しました。

変更：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"));

この場合（おそらく他のほとんどの場合）、警告をオフにするのは悪い考えです-理由があるためです。他の代替方法（デフォルトですべての文字列を書き込み可能にする）は、潜在的に非効率的です。

コンパイラがあなたに言っていることを聞いてください！

g ++には-wオプションを使用するだけです

例：

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

これは非推奨を回避するのではなく、端末に警告メッセージを表示しないようにすることを覚えておいてください。

廃止を本当に避けたい場合は、次のようにconstキーワードを使用します。

const char* s="constant string";  

-Wno-deprecated非推奨の警告メッセージを無視するオプションを使用しないのはなぜですか？

現在の問題は、私が-Werrorで実行していることです

これがあなたの本当の問題です、IMO。（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-stringsがデフォルトでアクティブになっています。

でg++ (Compiling C++)-Wwrite、文字列、デフォルトでアクティブになって

これが、異なる振る舞いがある理由です。のマクロを使用すると、Boost_pythonこのような警告が生成されます。したがって-Wno-write-strings、常に使用するため、C ++のコンパイル時に使用します。-Werror

文字列をaとして宣言constすると問題が解決します：

char const*s = "constant string";