On 10/31/14 19:27, ira.weiny@xxxxxxxxx wrote:
+ format_specifier = 0;
+ for (tmp = node_desc; tmp[0] != '\0'; tmp++) {
+ if (tmp[0] == '%' && tmp[1] == 'h') {
+ format_specifier = 1;
+ break;
+ }
+ }
Hello Ira,
Sorry that I hadn't noticed this before but I think the above code can
be simplified into "format_specifier = !!strstr(node_desc, "%h")". With
or without that change, if you want you can add:
Reviewed-by: Bart Van Assche <bvanassche@xxxxxxx>
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html