RCU Forums - View Single Post - CDI gr8flyer55
View Single Post
Old 09-22-2012 | 10:06 AM
  #361  
jakestew
Senior Member
 
Joined: Aug 2011
Posts: 144
Likes: 0
Received 0 Likes on 0 Posts
From: Moscow, ID
Default RE: CDI gr8flyer55

This is fine. Those are only warnings, not errors.
The first three are because I have declared variables, but not actually used them in the code.

The fourth warning tells me to check the code with their validation tool, and I don't know where it is or if I have it. Because I used a pointer to switch between the table it thinks I should use their tool to make sure that it can only get a value that will work. If my pointer took a wrong value it would read a wrong and unexpected memory location, which would give some random value in memory instead of the expected table value.

Thanks for reminding me... I actually think there will be a problem with that pointer. When 255 is rounded up to 256 it will actually not roll over to 0 anymore. Then the pointer will be pointing at the 16-bits after the end of the table. Essentially, it will be loading a unknown value at 916 RPM.

You can probably test the code as is, but I'll work on fixing that.


-Jake