??? 07/07/06 15:13 Read: times |
#119818 - Formatted, but not the same code! Responding to: ???'s previous message |
Yes - now you've got the formatting!
But it's not the same code that you originally posted, is it? Originally, you said:
void externalinterrupt(void) interrupt 0 { if(!flag){ TR0 = 1; flag = !flag; } if(flag){ TR0 = 0; IEN0 = 0x00; flag = !flag; timercounts = (TH0*256) + TL0; time = timercounts/921583; rpm = (unsigned int)(30/time); } With formatting, you said:
void external interrupt(void) interrupt 0{ if(!flag){ TR0 = 1; //enable timer0 flag = 1; } else if(flag){ TR0 = 0; //disable timer0 IEN0 =0x00; //disable all interrupts flag =0; timercounts = (TH0*256) + TL0; time = timercounts/921583; rpm = 30/time; } So the if(!flag) ...; if(flag){has become if(!flag) ... else if(flag){but the second 'if' is still pointless! See: http://www.8052.com/forum/read.phtml?id=119814 |
Topic | Author | Date |
frequency of a digital pulse | 01/01/70 00:00 | |
Variable definitions ? | 01/01/70 00:00 | |
Please post formatted code | 01/01/70 00:00 | |
Some points: | 01/01/70 00:00 | |
misconceptions cleared | 01/01/70 00:00 | |
inappropiate response | 01/01/70 00:00 | |
the entire problem again | 01/01/70 00:00 | |
flag=!flag ? | 01/01/70 00:00 | |
The value of Flag is known! | 01/01/70 00:00 | |
I see, but I don't C | 01/01/70 00:00 | |
It's illogical, Captain... | 01/01/70 00:00 | |
It is worse than that![]() | 01/01/70 00:00 | |
nope | 01/01/70 00:00 | |
Please post formatted code!! | 01/01/70 00:00 | |
he did follow the instructions | 01/01/70 00:00 | |
Oh no he didn't! | 01/01/70 00:00 | |
Argument | 01/01/70 00:00 | |
well, I saw the indentions and - shame | 01/01/70 00:00 | |
formatted code................. | 01/01/70 00:00 | |
thus therefore undefined | 01/01/70 00:00 | |
Formatted, but not the same code! | 01/01/70 00:00 | |
2 problems off the top | 01/01/70 00:00 | |
a blatant understatement | 01/01/70 00:00 | |
Implicit casting mess. | 01/01/70 00:00 | |
commentless code is worthless | 01/01/70 00:00 |