Le dimanche 23 octobre 2011 13:01:38, Leon Romanovsky a écrit : > On Sun, Oct 23, 2011 at 04:13, Pierre-Hugues Husson <[email protected]> wrote: > > + { > > + //Resume on home key press > > + char cmd[]={ 0x05, 0x03, 0x01, 0x02}; > > + cmd[2]=!!wakeup_on_home_key; > > + nvec_write_async(nvec, cmd, sizeof(cmd)); > > + } > > +} > > It is better to use the same coding style as was already in nvec > nvec_write_async(nvec, cmd, sizeof(cmd)); ===> nvec_write_async(nvec, > "\x05\x03\x01\x02", 4); So the code would look like: if(wakeup_on_home_key) nvec_write_async(nvec, "\x05\x03\x01\x02", 4); else nvec_write_async(nvec, "\x05\x03\x00\x02", 4); ? I find it less understandable (as in which field is of which use). But well, all nvec code is mostly about magic code, so why not.
_______________________________________________ Mailing list: https://launchpad.net/~ac100 Post to : [email protected] Unsubscribe : https://launchpad.net/~ac100 More help : https://help.launchpad.net/ListHelp
