Oware: Fix variable names and types

Authored by dmadaan on Jun 12 2017, 1:07 PM.

Description

Oware: Fix variable names and types

Details

Committed
dmadaanJun 12 2017, 1:07 PM
Parents
R2:134f6fd3b4c9: Oware: Complete sowing of seeds
Branches
Unknown
Tags
Unknown
Restricted Application added a subscriber: jjazeix. ยท View Herald TranscriptJun 12 2017, 8:02 AM
jjazeix added inline comments.Jun 12 2017, 11:34 AM
/src/activities/oware/Oware.qml
90

real?

/src/activities/oware/oware.js
58

it may b good to have at least a local variable corresponding to this 4

92

i < 12 && j < 6 or, only one of both (I don't know which one is checked there but there is only one, not both, the comma operator is skipping one)

102

no more needed as you have put a good variable name

145

be careful of the indentation (not only there)