On Thu, Nov 29, 2012 at 12:07 PM, YOSHIFUJI Hideaki <yoshfuji@xxxxxxxxxxxxxx> wrote: > Hi, Michael. > > Michael Kerrisk (man-pages) wrote: >> On Fri, Nov 16, 2012 at 6:28 PM, YOSHIFUJI Hideaki >> <yoshfuji@xxxxxxxxxxxxxx> wrote: >>> Please review. >> >> Thanks Hideaki. >> >> I've merged this into a branch, made a number of fixes, and added an >> example program. Could you check my changes in the diff below. (The >> complete page is attached.) > >> + for (i = if_ni; ! (i\->if_index == 0 && i\->if_name == NULL); i++) >> + printf("%d: %s\\n", i\->if_index, i\->if_name); > > Since type of if_index is unsigned int, please use %u here. > Thanks. Thanks. Fixed! Cheers, Michael -- Michael Kerrisk Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/ Author of "The Linux Programming Interface"; http://man7.org/tlpi/ -- To unsubscribe from this list: send the line "unsubscribe linux-man" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html