Warning: #962-D: use of a type with no linkage to declare a function, О чем предупреждает Кейл? Что ему не нравится? Опции

[Professor Chaos]

Простенький пример проекта из одного main.cpp файла

Код

// Файл main.cpp
// Тип состояния флага события
typedef volatile enum {
  FLAG_STATE_OFF         = 0,    // Флаг события сброшен
  FLAG_STATE_ON            = 1    // Флаг события взведён
} Flag_State_t;

// Получить значение флага события по указателю на него
Flag_State_t GetFlagState (const Flag_State_t *Flag_ptr) {return *Flag_ptr;}

// Установить заданное значение флага события по указателю на него
void SetFlagState (Flag_State_t *Flag_ptr,Flag_State_t NewFlsgState) {
  *Flag_ptr=NewFlsgState;
};

// Глобальные объекты - флаги событий
Flag_State_t MyFlag1, MyFlag2;                        



int main () {    
    
  SetFlagState (&MyFlag1,FLAG_STATE_ON);  // MyFlag1=FLAG_STATE_ON;
  MyFlag2=GetFlagState (&MyFlag1);             // MyFlag2=MyFlag1;
  while(1);
}

Компиляция выдает 2 предупреждения на функции GetFlagState и SetFlagState :

warning: #962-D: use of a type with no linkage to declare a function

Если в объявлении типа Flag_State_t убрать volatile - предупреждения исчезнут.
Вопрос: о чём предупреждает Кейл? Чем ему не нравится volatile в определении типа Flag_State_t ?

Более того, если вызовы функций заменить на непосредственное присваивание значений (строки, что закомментированы), то предупреждений тоже не будет. Т.е. наличие volatile играет роль только при наличии функций, аргументы или возвращаемые значения которых имеют тип, определённый с данным спецификатором.

По логике работы программы volatile необходим, т.к. флаги событий взводятся в функциях обработки прерываний, а сбрасываются и проверяются в основном цикле программы. Без него нельзя, иначе компилятор может некорректно оптимизировать код и программа не будет работать, как ей положено.

[VladislavS]

Добавлю лишь, что в C++ для объявления типа перечисления не нужен typedef. Вот так будет достаточно.

Код

enum Flag_State_t {
  FLAG_STATE_OFF  = 0,    // Флаг события сброшен
  FLAG_STATE_ON    = 1    // Флаг события взведён
};

[Arlleex]

Добавлю лишь, что в C++ для объявления типа перечисления не нужен typedef. Вот так будет достаточно.

Код

enum Flag_State_t {
  FLAG_STATE_OFF  = 0,    // Флаг события сброшен
  FLAG_STATE_ON    = 1    // Флаг события взведён
};

А может и нужен. Не знаю как в C++, а в Си typedef может быть полезен так же, как и в остальных случаях - чтобы не писать везде enum (type) при каждом объявлении переменной перечислимого типа.
Но зачем вообще что-то возвращать по указателю как выше и установку тоже по указателю, ума не приложу...

Сообщение отредактировал Arlleex - Aug 6 2018, 04:57

[jcxz]

А может и нужен. Не знаю как в C++, а в Си typedef может быть полезен так же, как и в остальных случаях - чтобы не писать везде enum (type) при каждом объявлении переменной перечислимого типа.

В си++ как и в случае со структурами/классами, так и с enum - писать слово "enum" при объявлении переменной не обязательно. Вместе с именем объявленного типа компилятор сохраняет и информацию о том что это: структура-ли, enum-ли, ....
Вообще не понимаю: си++ уже седой весь и мхом порос с этими правилами, ещё несколько десятилетий назад, когда я его изучал, всё это уже было в нём, но почему-то народ упорно продолжает лепить где ни попадя typedef.... Какой смысл? Или работодатель за кол-во знаков в исходнике платит??? typedef в си++ мало где необходим. Но упорство просто какое-то фанатическое....

Но зачем вообще что-то возвращать по указателю как выше и установку тоже по указателю, ума не приложу...

Там вообще не исходник, а какое-то недоразумение. Нет смысла его обсуждать. Самое разумное - автору посоветовать учебник по си.