On 05/09/2013 04:42 PM, Jiri Kosina wrote: > On Wed, 8 May 2013, Chen Gang 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) {};'' >> > >> > Signed-off-by: Chen Gang <gang.chen@xxxxxxxxxxx> > Makes sense. > > Acked-by: Jiri Kosina <jkosina@xxxxxxx> > > Gustavo, going to take this, please? > > -- Jiri Kosina SUSE Labs > Thanks, and also help to fix the spelling: redundent -> redundant. (if need let me send patch v3, please reply to tell me, 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