реклама на сайте
подробности

 
 
 
Reply to this topicStart new topic
> Сложности с ANSI C в Keil, при установке этой опции проект не собирается
le-greem
сообщение Jun 24 2011, 10:00
Сообщение #1


Участник
*

Группа: Участник
Сообщений: 16
Регистрация: 22-04-11
Пользователь №: 64 545



Добрый день. Столкнулся с непонятной реакцией Keil на этапе настройки прерываний. Суть заключается в том, возникла необходимость использования одной из библиотек, написанных на ANSI С, о чем свидетельствует начало заголовочного файла. При выборе Target Options -> C/C++ -> Strict ANSI C в настройках начинает ругаться компилятор фразой "invalid type conversion" на следующее выражение:

install_irq(TIMER0_INT, (void *)TIMER0_Handler, 1 );

Данная функция определена в другом модуле как unsigned long install_irq( unsigned long IntNumber, void *HandlerAddr, unsigned long Priority ) {...},
причем TIMER0_INT определен в заголовке как

#define TIMER0_INT 4

TIMER0_Handler определена как "void TIMER0_Handler (void) __irq"

Примечательно, что при отключении данной опции проект собирается без проблем и прерывание срабатывает.


Не могли бы вы подсказать в каком направлении двигаться в этом случае?
Go to the top of the page
 
+Quote Post
scifi
сообщение Jun 24 2011, 10:07
Сообщение #2


Гуру
******

Группа: Свой
Сообщений: 3 020
Регистрация: 7-02-07
Пользователь №: 25 136



Цитата(le-greem @ Jun 24 2011, 14:00) *
Не могли бы вы подсказать в каком направлении двигаться в этом случае?

Он ругается на ключевое слово "__irq". Такого слова в строгом ANSI C нет.
Мне кажется, Вы неправильно понимаете опцию ANSI C. Она для того, чтобы отлавливать опечатки/ошибки на этапе написания кода (типа правил MISRA C). Если код готовый и рабочий, то собирать его можно и без опции ANSI C.
Go to the top of the page
 
+Quote Post
le-greem
сообщение Jun 24 2011, 10:16
Сообщение #3


Участник
*

Группа: Участник
Сообщений: 16
Регистрация: 22-04-11
Пользователь №: 64 545



В моем случае эта опция необходима, т.к. проект на стадии разработки. =) Есть ли возможность объявить эту функцию в указанном выше режиме?
Go to the top of the page
 
+Quote Post
sergeeff
сообщение Jun 24 2011, 10:34
Сообщение #4


Профессионал
*****

Группа: Свой
Сообщений: 1 481
Регистрация: 10-04-05
Пользователь №: 4 007



Цитата(le-greem @ Jun 24 2011, 13:16) *
В моем случае эта опция необходима, т.к. проект на стадии разработки. =) Есть ли возможность объявить эту функцию в указанном выше режиме?


Непонятно ваше стремление. Нет в ANSI C __irq да и наверное еще чего из мира embedded. Вам рабочая программа ведь нужна? Ну так ее и пишите.
Go to the top of the page
 
+Quote Post
le-greem
сообщение Jun 24 2011, 10:41
Сообщение #5


Участник
*

Группа: Участник
Сообщений: 16
Регистрация: 22-04-11
Пользователь №: 64 545



Спасибо за ответ. Очень вразумительно. sm.gif
Go to the top of the page
 
+Quote Post
Sanya_kv
сообщение Jun 24 2011, 10:46
Сообщение #6


Частый гость
**

Группа: Свой
Сообщений: 185
Регистрация: 25-02-09
Из: Россия
Пользователь №: 45 369



Цитата(le-greem @ Jun 24 2011, 14:00) *
TIMER0_Handler определена как "void TIMER0_Handler (void) __irq"

Объяви __irq void TIMER0_Handler (void)
Keil не как не определится куда её (__irq) лучше пристроить wink.gif
Go to the top of the page
 
+Quote Post
scifi
сообщение Jun 24 2011, 11:55
Сообщение #7


Гуру
******

Группа: Свой
Сообщений: 3 020
Регистрация: 7-02-07
Пользователь №: 25 136



Цитата(le-greem @ Jun 24 2011, 14:16) *
В моем случае эта опция необходима, т.к. проект на стадии разработки. =)

Не понимаю, как из "проект на стадии разработки" следует "опция необходима". Нет в ней необходимости.

Вот моё понимание того, почему опция Strict ANSI C существует: если возникло желание написать переносимый код (ANSI C без всяких добавок), то включаем эту опцию, чтобы компилятор сообщал о нарушениях. Кстати, имейте в виду, что для полной переносимости кода одного соблюдения ANSI C мало. Большинство реальных программ содержат и непереносимый код (к примеру, __irq). Такой код группируем в отдельные исходники и их компилируем без опции ANSI C. Вот и всё.
Go to the top of the page
 
+Quote Post
ViKo
сообщение Jun 24 2011, 18:08
Сообщение #8


Универсальный солдатик
******

Группа: Модераторы
Сообщений: 8 634
Регистрация: 1-11-05
Из: Минск
Пользователь №: 10 362



Цитата(le-greem @ Jun 24 2011, 13:16) *
В моем случае эта опция необходима, т.к. проект на стадии разработки.

Наоборот, включите поддержку стандарта C99 (по этим буквам в помощи найдете, как), и будете пользоваться всеми его улучшениями, по сравнению с ANSI C.
Go to the top of the page
 
+Quote Post
le-greem
сообщение Jun 27 2011, 04:48
Сообщение #9


Участник
*

Группа: Участник
Сообщений: 16
Регистрация: 22-04-11
Пользователь №: 64 545



Цитата(scifi @ Jun 24 2011, 15:55) *
Не понимаю, как из "проект на стадии разработки" следует "опция необходима". Нет в ней необходимости....

Необходимость как раз и состоит в "отлове" нарушений...

Цитата(scifi @ Jun 24 2011, 15:55) *
...Такой код группируем в отдельные исходники и их компилируем без опции ANSI C. Вот и всё.


Ну я примерно так и сделал, поместив код в блоки условной компиляции.

Цитата(ViKo @ Jun 24 2011, 22:08) *
Наоборот, включите поддержку стандарта C99 (по этим буквам в помощи найдете, как), и будете пользоваться всеми его улучшениями, по сравнению с ANSI C.


Эта опция уже использовалась.

Сообщение отредактировал le-greem - Jun 27 2011, 04:52
Go to the top of the page
 
+Quote Post
jorikdima
сообщение Jun 27 2011, 05:18
Сообщение #10


тут может быть ваша реклама
*****

Группа: Свой
Сообщений: 1 164
Регистрация: 15-03-06
Из: Санкт-Петербург/CA
Пользователь №: 15 280



Соглашусь, что незачем себя ограничивать ANSI C если нет требования переносимости или линкуемости с другими модулями на ANSI C. Автор, сосредоточьтесь лучше на написании программы sm.gif
Go to the top of the page
 
+Quote Post

Reply to this topicStart new topic
1 чел. читают эту тему (гостей: 1, скрытых пользователей: 0)
Пользователей: 0

 


RSS Текстовая версия Сейчас: 18th July 2025 - 07:19
Рейтинг@Mail.ru


Страница сгенерированна за 0.0164 секунд с 7
ELECTRONIX ©2004-2016