Ложное предупреждение «вычисленное значение не используется»

Question

Ложное предупреждение «вычисленное значение не используется»

0

Я поддерживаю некоторый уродливый код устаревшего со следующей функцией, и я получаю

warning: value computed is not used

для строк, отмеченных комментариями ниже:

void ReadKeyValuePipe(char* buffer, char* key, char* value) {
    char* pos;
    char key_str[1024];
    char* val = value;
    sprintf(key_str,"%s:",key);
    if((pos = strstr(buffer,key))) {
        pos += strlen(key_str);
        while (*pos && *pos != '|') {
            *val = *pos;
            *val++; // this is actually used
            *pos++; // so is this
        }
        *val = 0;
    }
}

Когда я удаляю эти строки, код прерывается. Это имеет смысл, поскольку они, как представляется, увеличивают маркеры.

Как заставить компилятор распознать, что эти вычисления фактически используются?

Chris Redford 28 окт. 2014, в 20:20

Источник

1

Они не используются - смотрите связанный дубликат.
BartoszKP 28 окт. 2014, в 18:28
0

Я был бы больше обеспокоен скрытым переполнением буфера при вызове sprintf . По крайней мере, assert(strlen(key) + 2 <= sizeof(key_str));
Casey 28 окт. 2014, в 21:21
0

@Casey Вся эта функция кажется мне неправильным способом сделать это, но я должен выбрать свои сражения.
Chris Redford 28 окт. 2014, в 21:22
0

WTF? key_str используется только неэффективный способ расчета 1 + strlen(key) ? Должно ли это быть strstr(buffer, key_str) вместо этого? Эта функция действительно ужасна.
Casey 28 окт. 2014, в 21:24
0

Шутки в сторону. Если бы я собирался переписать эту функцию, я просто преобразовал бы любой char* в string и сделал бы это с разделением строки .
Chris Redford 28 окт. 2014, в 21:26

Показать ещё 3 комментария

Теги:

c++

c

c++11

gcc-warning

gcc4.7

2 ответа

1

Вы можете использовать -Wno-unused-value компилятора -Wno-unused-value если вы уверены, что ваш код верен. Это не так, поскольку вы, очевидно, не знаете, что *val++; делает.

*val++; 
*pos++;

такой же как:

 val++;
 pos++;

Igor 28 окт. 2014, в 17:22

1

Проблема этого подхода заключается в том, что этот уродливый унаследованный код и мои новые дополнения иногда имеют реальные неиспользуемые значения, и я полагаюсь на предупреждения, чтобы распознать и устранить это. (Я не был человеком, который проголосовал за тебя).
Chris Redford 28 окт. 2014, в 18:25
0

Вы можете применить -Wno-unused-value только к нескольким строкам кода с проблемой, используя pragma GCC diagnostic ignored подход pragma GCC diagnostic ignored
Walter 28 окт. 2014, в 18:43

Ещё вопросы

Они не используются - смотрите связанный дубликат.
Я был бы больше обеспокоен скрытым переполнением буфера при вызове sprintf . По крайней мере, assert(strlen(key) + 2 <= sizeof(key_str));
@Casey Вся эта функция кажется мне неправильным способом сделать это, но я должен выбрать свои сражения.
WTF? key_str используется только неэффективный способ расчета 1 + strlen(key) ? Должно ли это быть strstr(buffer, key_str) вместо этого? Эта функция действительно ужасна.
Шутки в сторону. Если бы я собирался переписать эту функцию, я просто преобразовал бы любой char* в string и сделал бы это с разделением строки .
Проблема этого подхода заключается в том, что этот уродливый унаследованный код и мои новые дополнения иногда имеют реальные неиспользуемые значения, и я полагаюсь на предупреждения, чтобы распознать и устранить это. (Я не был человеком, который проголосовал за тебя).
Вы можете применить -Wno-unused-value только к нескольким строкам кода с проблемой, используя pragma GCC diagnostic ignored подход pragma GCC diagnostic ignored

Barry · Accepted Answer · 2014-10-28T16-40-00.000Z

Вы разыгрываете val и pos а затем увеличиваете их, но вы никогда не используете результат разыменования. Вы можете просто заменить эти строки:

*val = *pos;
++val;
++pos;

Или, возможно, что код первоначально был до того, как кто-то "зафиксировал" его:

*val++ = *pos++;