Hi! Please CC to [email protected] now that we've finally got a SLOF mailing list :-) !
On Wed, 8 Jul 2015 18:47:44 +0200 Dinar valeev <[email protected]> wrote: ... > /** > + * Checks if keypos is a latin key > + * @param keypos > + * @return - > + */ > +void check_latin(uint8_t keypos) > + if (keypos > KEYP_LATIN_A || keypos < KEYP_LATIN_Z) { > + return true; > + } else { > + return false; > + } > + That does not look like valid C to me ... "void" return type and returning true or false ... and what about the outermost curly braces? Anyway, you could even write this much more simple without if-statement instead: bool check_latin(uint8_t keypos) { return keypos > KEYP_LATIN_A || keypos < KEYP_LATIN_Z; } Thomas _______________________________________________ Linuxppc-dev mailing list [email protected] https://lists.ozlabs.org/listinfo/linuxppc-dev
