On 05/08/2013 11:16 PM, David Herrmann wrote: > On Wed, May 8, 2013 at 5:34 AM, Chen Gang <gang.chen@xxxxxxxxxxx> wrote: >> > >> > For NUL terminated string, need always let it ended by zero. >> > >> > Since have already called memcpy() to initialize 'ci', so need not >> > redundent initializations. >> > >> > Better use ''if(session->hid) {} else if(session->input) {}'' instead >> > of ''if(session->hid) {}; if(session->input) {};'' > Yep, looks good now. > > Reviewed-by: David Herrmann <dh.herrmann@xxxxxxxxx> Thanks. -- Chen Gang Asianux Corporation -- To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html