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

 
 
3 страниц V   1 2 3 >  
Reply to this topicStart new topic
> Ошибка при компиляции GNU (студия), Нужна подсказка
skyled
сообщение Jun 30 2010, 14:41
Сообщение #1


Местный
***

Группа: Участник
Сообщений: 217
Регистрация: 11-06-10
Пользователь №: 57 868



Код
buf_rx[15]=Crc8(rx,15);
Вот на эту строчку ругается.
Код
116: warning: passing argument 1 of 'Crc8' discards qualifiers from pointer target type
Не могу разобраться что это за ошибка. Подскажите, кто знает? Спасибо.
Go to the top of the page
 
+Quote Post
Палыч
сообщение Jun 30 2010, 15:04
Сообщение #2


Гуру
******

Группа: Свой
Сообщений: 2 399
Регистрация: 10-05-06
Из: г. Новочеркасск
Пользователь №: 16 954



Переменная rx - это у Вас что? Указатель должен быть первым аргументом процедуры
Go to the top of the page
 
+Quote Post
vadimuzzz
сообщение Jul 1 2010, 02:09
Сообщение #3


Гуру
******

Группа: Свой
Сообщений: 2 291
Регистрация: 21-07-05
Пользователь №: 6 988



Цитата(skyled @ Jun 30 2010, 21:41) *
Код
buf_rx[15]=Crc8(rx,15);
Вот на эту строчку ругается.
Код
116: warning: passing argument 1 of 'Crc8' discards qualifiers from pointer target type
Не могу разобраться что это за ошибка. Подскажите, кто знает? Спасибо.

сравните объявление rx и список параметров Crc8(). предупреждение гласит, что отброшен квалификатор типа у переменной rx (например, rx объявлен как volatile char[], а параметр у Crc8 имеет тип char*)
Go to the top of the page
 
+Quote Post
skyled
сообщение Jul 1 2010, 07:34
Сообщение #4


Местный
***

Группа: Участник
Сообщений: 217
Регистрация: 11-06-10
Пользователь №: 57 868



Вот так объявлено:
Код
volatile unsigned char buf_rx[16];
volatile unsigned char *rx=&buf_rx[16];

Вот функция:
Код
const unsigned char Crc8Table[256] = {
    0x00, 0x31, 0x62, 0x53, 0xC4, 0xF5, 0xA6, 0x97,
    0xB9, 0x88, 0xDB, 0xEA, 0x7D, 0x4C, 0x1F, 0x2E,
    0x43, 0x72, 0x21, 0x10, 0x87, 0xB6, 0xE5, 0xD4,
    0xFA, 0xCB, 0x98, 0xA9, 0x3E, 0x0F, 0x5C, 0x6D,
    0x86, 0xB7, 0xE4, 0xD5, 0x42, 0x73, 0x20, 0x11,
    0x3F, 0x0E, 0x5D, 0x6C, 0xFB, 0xCA, 0x99, 0xA8,
    0xC5, 0xF4, 0xA7, 0x96, 0x01, 0x30, 0x63, 0x52,
    0x7C, 0x4D, 0x1E, 0x2F, 0xB8, 0x89, 0xDA, 0xEB,
    0x3D, 0x0C, 0x5F, 0x6E, 0xF9, 0xC8, 0x9B, 0xAA,
    0x84, 0xB5, 0xE6, 0xD7, 0x40, 0x71, 0x22, 0x13,
    0x7E, 0x4F, 0x1C, 0x2D, 0xBA, 0x8B, 0xD8, 0xE9,
    0xC7, 0xF6, 0xA5, 0x94, 0x03, 0x32, 0x61, 0x50,
    0xBB, 0x8A, 0xD9, 0xE8, 0x7F, 0x4E, 0x1D, 0x2C,
    0x02, 0x33, 0x60, 0x51, 0xC6, 0xF7, 0xA4, 0x95,
    0xF8, 0xC9, 0x9A, 0xAB, 0x3C, 0x0D, 0x5E, 0x6F,
    0x41, 0x70, 0x23, 0x12, 0x85, 0xB4, 0xE7, 0xD6,
    0x7A, 0x4B, 0x18, 0x29, 0xBE, 0x8F, 0xDC, 0xED,
    0xC3, 0xF2, 0xA1, 0x90, 0x07, 0x36, 0x65, 0x54,
    0x39, 0x08, 0x5B, 0x6A, 0xFD, 0xCC, 0x9F, 0xAE,
    0x80, 0xB1, 0xE2, 0xD3, 0x44, 0x75, 0x26, 0x17,
    0xFC, 0xCD, 0x9E, 0xAF, 0x38, 0x09, 0x5A, 0x6B,
    0x45, 0x74, 0x27, 0x16, 0x81, 0xB0, 0xE3, 0xD2,
    0xBF, 0x8E, 0xDD, 0xEC, 0x7B, 0x4A, 0x19, 0x28,
    0x06, 0x37, 0x64, 0x55, 0xC2, 0xF3, 0xA0, 0x91,
    0x47, 0x76, 0x25, 0x14, 0x83, 0xB2, 0xE1, 0xD0,
    0xFE, 0xCF, 0x9C, 0xAD, 0x3A, 0x0B, 0x58, 0x69,
    0x04, 0x35, 0x66, 0x57, 0xC0, 0xF1, 0xA2, 0x93,
    0xBD, 0x8C, 0xDF, 0xEE, 0x79, 0x48, 0x1B, 0x2A,
    0xC1, 0xF0, 0xA3, 0x92, 0x05, 0x34, 0x67, 0x56,
    0x78, 0x49, 0x1A, 0x2B, 0xBC, 0x8D, 0xDE, 0xEF,
    0x82, 0xB3, 0xE0, 0xD1, 0x46, 0x77, 0x24, 0x15,
    0x3B, 0x0A, 0x59, 0x68, 0xFF, 0xCE, 0x9D, 0xAC
};

unsigned char Crc8(unsigned char *pcBlock, unsigned char len)
{unsigned char crc = 0xFF;
while (len--)
crc = Crc8Table[crc ^ *pcBlock++];
return crc;}

Хочет указатель и получает указатель.

volatile убрал, предупреждение осталось.
Go to the top of the page
 
+Quote Post
MrYuran
сообщение Jul 1 2010, 07:38
Сообщение #5


Беспросветный оптимист
******

Группа: Свой
Сообщений: 4 640
Регистрация: 26-12-07
Из: Н.Новгород
Пользователь №: 33 646



Приведите принудительно аргумент к unsigned char*, варнинг исчезнет.
Я обычно просто не обращаю внимания.
Это не ошибка, просто ахтунг, чтобы обратили внимание.


--------------------
Программирование делится на системное и бессистемное. ©Моё :)
— а для кого-то БГ — это Bill Gilbert =)
Go to the top of the page
 
+Quote Post
andrew_b
сообщение Jul 1 2010, 08:06
Сообщение #6


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

Группа: Свой
Сообщений: 1 975
Регистрация: 30-12-04
Из: Воронеж
Пользователь №: 1 757



Цитата(skyled @ Jul 1 2010, 11:34) *
Вот так объявлено:
Код
volatile unsigned char buf_rx[16];
volatile unsigned char *rx=&buf_rx[16];
rx у вас указывает куда-то за границу массива. Вы точно этого хотите?
Go to the top of the page
 
+Quote Post
MrYuran
сообщение Jul 1 2010, 08:08
Сообщение #7


Беспросветный оптимист
******

Группа: Свой
Сообщений: 4 640
Регистрация: 26-12-07
Из: Н.Новгород
Пользователь №: 33 646



Цитата(andrew_b @ Jul 1 2010, 12:06) *
rx у вас указывает куда-то за границу массива.

Точно подмечено, я и не заметил...
Вот для этого, наверно, ахтунг и выставляется.


--------------------
Программирование делится на системное и бессистемное. ©Моё :)
— а для кого-то БГ — это Bill Gilbert =)
Go to the top of the page
 
+Quote Post
skyled
сообщение Jul 1 2010, 08:18
Сообщение #8


Местный
***

Группа: Участник
Сообщений: 217
Регистрация: 11-06-10
Пользователь №: 57 868



Цитата
rx у вас указывает куда-то за границу массива. Вы точно этого хотите?
Нет, я этого не хочу. Как правильно все сделать? Дело в том, что эта часть кода зашита в МК, а со стороны ПК точно такая же функция подсчитывает Crc8. МК формирует пакет и отправляет его, а комп принимает и проверяет целостность. При этом значения получаются разные. Так и не понятно пока почему. Монитор порта показавает, что в обе стороны все передается нормально.
Go to the top of the page
 
+Quote Post
Палыч
сообщение Jul 1 2010, 08:25
Сообщение #9


Гуру
******

Группа: Свой
Сообщений: 2 399
Регистрация: 10-05-06
Из: г. Новочеркасск
Пользователь №: 16 954



Зачем Вы завели переменную rx? Чем вас не устраивает
Код
buf_rx[15]= Crc8( buf_rx, 15 );
или
Код
buf_rx[15]= Crc8( &buf_rx[0], 15 );



Цитата(skyled @ Jul 1 2010, 12:18) *
При этом значения получаются разные.
Это - потому, что контрольная сумма у Вас считается не пятнадцати передаваемых в buf_rx байтов, а следующих за массвиом...
Go to the top of the page
 
+Quote Post
skyled
сообщение Jul 1 2010, 08:27
Сообщение #10


Местный
***

Группа: Участник
Сообщений: 217
Регистрация: 11-06-10
Пользователь №: 57 868



Код
Зачем Вы завели переменную rx? Чем вас не устраивает
Да вобщем то меня устраивает все, что правильно работает. Хотела функция указатель - получила его. Может действительно переделать ее. Я про эту сторону вопроса и не подумал. Но всетаки как правильно тут с указателем разобраться?
Go to the top of the page
 
+Quote Post
Палыч
сообщение Jul 1 2010, 08:31
Сообщение #11


Гуру
******

Группа: Свой
Сообщений: 2 399
Регистрация: 10-05-06
Из: г. Новочеркасск
Пользователь №: 16 954



Выбросите переменную rx, если она не используется в других местах. Исправьте вызов процедуры, как Вам советовал выше.

P.S. Нужно переделать только вызов процедуры, саму процедуру переделывать не нужно!
Go to the top of the page
 
+Quote Post
MrYuran
сообщение Jul 1 2010, 08:35
Сообщение #12


Беспросветный оптимист
******

Группа: Свой
Сообщений: 4 640
Регистрация: 26-12-07
Из: Н.Новгород
Пользователь №: 33 646



Цитата(skyled @ Jul 1 2010, 12:27) *
Но всетаки как правильно тут с указателем разобраться?

Имя массива - это и есть указатель на начало массива.
Если вы указываете явно номер элемента, то вставляете в функцию адрес не начала массива, а указанного элемента.
Со всеми вытекающими.
В вашем случае вы вообще указываете на адрес за последним элементом массива.

Цитата(Палыч @ Jul 1 2010, 12:31) *
Выбросите переменную rx, если она не используется в других местах.

Указатель rx имеет смысл только в том случае, если буферов несколько и надо между ними переключаться.
Если буфер один - смысла нет, только лишний расход памяти и лишние телодвижения при обращении.


--------------------
Программирование делится на системное и бессистемное. ©Моё :)
— а для кого-то БГ — это Bill Gilbert =)
Go to the top of the page
 
+Quote Post
vadimuzzz
сообщение Jul 1 2010, 08:45
Сообщение #13


Гуру
******

Группа: Свой
Сообщений: 2 291
Регистрация: 21-07-05
Пользователь №: 6 988



Цитата(skyled @ Jul 1 2010, 14:34) *
volatile убрал, предупреждение осталось.

нету никаких ворнингов:
Код
...
unsigned char Crc8(unsigned char *pcBlock, unsigned char len)
{unsigned char crc = 0xFF;
while (len--)
crc = Crc8Table[crc ^ *pcBlock++];
return crc;}

int main()
{
    unsigned char buf_rx[16];
    unsigned char *rx=&buf_rx[0];
    return Crc8(rx,16);
}


или так
Код
unsigned char buf_rx[16];
return Crc8(&buf_rx[0],16);
Go to the top of the page
 
+Quote Post
MrYuran
сообщение Jul 1 2010, 08:48
Сообщение #14


Беспросветный оптимист
******

Группа: Свой
Сообщений: 4 640
Регистрация: 26-12-07
Из: Н.Новгород
Пользователь №: 33 646



Цитата(vadimuzzz @ Jul 1 2010, 12:45) *
нету никаких ворнингов:

Возможно, они у вас просто отключены.


--------------------
Программирование делится на системное и бессистемное. ©Моё :)
— а для кого-то БГ — это Bill Gilbert =)
Go to the top of the page
 
+Quote Post
vadimuzzz
сообщение Jul 1 2010, 08:55
Сообщение #15


Гуру
******

Группа: Свой
Сообщений: 2 291
Регистрация: 21-07-05
Пользователь №: 6 988



Цитата(MrYuran @ Jul 1 2010, 15:48) *
Возможно, они у вас просто отключены.

нет, если добавить volatile, они появятся. собственно вот опции: -O0 -g3 -Wall -c -fmessage-length=0. просто топикстартеру надо иметь в виду, что volatile там не просто так стоит.
Go to the top of the page
 
+Quote Post

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

 


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


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