The patch titled parser: remove unnecessary strlen() has been removed from the -mm tree. Its filename was parser-remove-unnecessary-strlen.patch This patch was dropped because it was merged into mainline or a subsystem tree The current -mm tree may be found at http://userweb.kernel.org/~akpm/mmotm/ ------------------------------------------------------ Subject: parser: remove unnecessary strlen() From: André Goddard Rosa <andre.goddard@xxxxxxxxx> No functional change. Cache strlen() result to avoid recalculating it up to 3 times on the worst case. Reduces code size a little by 32 bytes: text data bss dec hex filename 1385 0 0 1385 569 lib/parser.o-BEFORE 1353 0 0 1353 549 lib/parser.o-AFTER Signed-off-by: André Goddard Rosa <andre.goddard@xxxxxxxxx> Cc: Randy Dunlap <randy.dunlap@xxxxxxxxxx> Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx> --- lib/parser.c | 11 +++++++---- 1 file changed, 7 insertions(+), 4 deletions(-) diff -puN lib/parser.c~parser-remove-unnecessary-strlen lib/parser.c --- a/lib/parser.c~parser-remove-unnecessary-strlen +++ a/lib/parser.c @@ -56,13 +56,16 @@ static int match_one(char *s, const char args[argc].from = s; switch (*p++) { - case 's': - if (strlen(s) == 0) + case 's': { + size_t str_len = strlen(s); + + if (str_len == 0) return 0; - else if (len == -1 || len > strlen(s)) - len = strlen(s); + if (len == -1 || len > str_len) + len = str_len; args[argc].to = s + len; break; + } case 'd': simple_strtol(s, &args[argc].to, 0); goto num; _ Patches currently in -mm which might be from andre.goddard@xxxxxxxxx are origin.patch pid-tighten-pidmap-spinlock-critical-section-by-removing-kfree.patch pid-reduce-code-size-by-using-a-pointer-to-iterate-over-array.patch -- To unsubscribe from this list: send the line "unsubscribe mm-commits" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html