【发布时间】:2011-10-23 23:11:26
【问题描述】:
我有一个带有 uint8_t * 参数的函数:
uint8_t* ihex_decode(uint8_t *in, size_t len, uint8_t *out)
{
uint8_t i, hn, ln;
for (i = 0; i < len; i+=2) {
hn = in[i] > '9' ? (in[i]|32) - 'a' + 10 : in[i] - '0';
ln = in[i+1] > '9' ? (in[i+1]|32) - 'a' + 10 : in[i+1] - '0';
out[i/2] = (hn << 4 ) | ln;
}
return out;
}
我使用这个函数:
uint8_t data[SPM_PAGESIZE]; // SPM_PAGESIZE = 256 bytes
uint8_t sysex_data[SPM_PAGESIZE/2];
ihex_decode(data, strlen(data), sysex_data);
但在这种情况下,我的编译器 (avr-gcc) 返回一个警告:
main.c|89|警告:传递“strlen”参数 1 的指针目标符号不同 /usr/include/string.h|399|注意:预期为 'const char *' 但参数类型为 'uint8_t *'
所以,我通过类型转换数据 var 找到了解决方案:
ihex_decode(data, strlen((const char *)data), sysex_data);
警告消失了,但我想知道这个解决方案是否安全。
有没有更好的办法?
谢谢
【问题讨论】:
-
你为什么使用
uint8_t来表示显然是char的类型? -
因为我的程序是在微控制器上运行的。所以我不能有负值。
-
这毫无意义。输入数据似乎是 ASCII 十六进制,所以它自然是 char。输出数据当然会保留为 uint8_t。
-
所以你建议我把
uint8_t *in改成char *in? -
这里的问题不在于使用 uint8_t,问题在于人们认为 char 是某种神秘的魔法类型。如果你仔细检查,你会发现 ASCII 表中没有负数。因此,将 uint8_t 用于 ASCII 字符非常好,实际上它是最适合使用的类型。由于历史/不合逻辑的原因,C 标准碰巧使用 char 类型,这就是您收到警告的原因。只需对此类警告进行类型转换,uint8_t 更正确。
标签: c casting constants avr-gcc