On Fri, Jul 8, 2011 at 12:35 PM, Øyvind Harboe <[email protected]> wrote: > On Fri, Jul 8, 2011 at 12:31 PM, Drasko DRASKOVIC > <[email protected]> wrote: >> On Fri, Jul 8, 2011 at 12:14 PM, Øyvind Harboe <[email protected]> >> wrote: >>>> There is no particular need to cast this into uint8_t* and this can be >>>> kept as a void*. Would that suppress the warnings ? >>> >>> It does look like this code is using uint8_t * in lieu of void *... >> >> Why ? It is just an address of 1-byte placeholder to which you would >> like to copy byte read from the memory. > > It's not about what you do, it is about telling the compiler as much > as possible about the types so that it can check for problems. > > I don't know the code too well to comment about which types > should be used. Especially with big/small endian...
I know this code, and I can change it. I just do not see any casting to unaligned access by visual inspection, and I can not reproduce the problem. So I do not know where to start changing - everything looks OK to me here. BR, Drasko _______________________________________________ Openocd-development mailing list [email protected] https://lists.berlios.de/mailman/listinfo/openocd-development
