2
« on: December 12, 2012, 06:18:01 PM »
Hi,
I've recently been playing with uTasker on STM32F107 on IAR workbench 6.5 and it compiles with the following warnings:
(line numbers may be a little out as I've been playing with the code)
STM32.c
Warning[Pe111]: statement is unreachable C:\Google Drive\Projects\ST100\uTasker\Hardware\STM32\STM32.c 322
#endif
return 0; // we never return....
}
#endif
Expected error - no action required.
Warning[Pe009]: nested comment is not allowed C:\Google Drive\Projects\ST100\uTasker\Hardware\STM32\STM32.c 2001
/*********************************************/
Action - remove leading "/"
Warning[Pa082]: undefined behavior: the order of volatile accesses is undefined in this statement C:\Google Drive\Projects\ST100\uTasker\Hardware\STM32\STM32.c 633
__interrupt void EMAC_Interrupt(void)
{
while ((ETH_DMAIER & ETH_DMASR) & (ETH_DMAIER_TIE | ETH_DMAIER_TPSIE | ETH_DMAIER_TBUIE | ETH_DMAIER_TJTIE |
ETH_DMAIER_ROTIE | ETH_DMAIER_TUIE | ETH_DMAIER_RIE | ETH_DMAIER_RBUIE |
ETH_DMAIER_RPSIE | ETH_DMAIER_RWTIE | ETH_DMAIER_ETIE | ETH_DMAIER_FBEIE |
ETH_DMAIER_ERIE | ETH_DMAIER_AISE | ETH_DMAIER_NISE)) { // while enabled EMAC interrupt pending
Action: Not sure if this is an issue or not.
http.c
Warning[Pe546]: transfer of control bypasses initialization of: C:\Google Drive\Projects\ST100\uTasker\stack\http.c 994
variable "Len" (declared at line 1111 of "C:\Google Drive\Projects\ST100\uTasker\stack\http.c")
variable "ucPush" (declared at line 1112 of "C:\Google Drive\Projects\ST100\uTasker\stack\http.c")
for (iSessionNumber = 0; iSessionNumber < NO_OF_HTTP_SESSIONS; iSessionNumber++) {
if (http_session->OwnerTCPSocket == Socket) { // search for the session to handle this event
switch (ucEvent) {
case TCP_EVENT_CONREQ: // session requested
if (http_session->ucState == HTTP_STATE_FREE) {
...
if (http_session->FileLength > http_session->FilePoint) {
MAX_FILE_LENGTH Len = (http_session->FileLength - http_session->FilePoint);
unsigned char ucPush = 0;
Action: Not sure if this is a problem or not.
I hope this was useful.
Colin