Timeline for Countdown Code: 'League of Legends'
Current License: CC BY-SA 3.0
5 events
| when toggle format | what | by | license | comment | |
|---|---|---|---|---|---|
| Jun 25, 2014 at 8:55 | history | edited | Vogel612 | CC BY-SA 3.0 |
fixed if-else bug. else if branch was previously inaccessible
|
| Apr 25, 2014 at 3:32 | comment | added | 200_success |
A perfect example of why you should always write explicit braces. Also, the repetitive if statements should be chained using else if. Better yet, use a switch block. (I see @JerryCoffin has already addressed the issue.)
|
|
| Apr 24, 2014 at 19:38 | comment | added | Pharap | @Vogel612 A decent compiler would make that optimisation for you. | |
| Apr 24, 2014 at 14:42 | comment | added | Vogel612 |
I'd also precalculate endTime - startTime to improve performance. If done correctly, you can even shorten the parameter list then...
|
|
| Apr 24, 2014 at 14:39 | history | answered | DFord | CC BY-SA 3.0 |