On Friday 23 October 2009, Ruediger Pluem wrote: > > - cp = strchr(path, '('); > > - if (cp) { > > + cp = strrchr(path, '('); > > + cp2 = path + strlen(path) - 1; > > + if (cp && (*cp2 == ')')) { > > *cp++ = '\0'; > > - > > - if (!(cp2 = strchr(cp, ')'))) { > > - return "Invalid argument: no closing parenthesis"; > > - } > > No error message any longer for missing parenthesis? > I doubt that filenames like /somewhere/somefile(something are > intended. >
The question is if such filenames should be allowed. But I guess it's enought to allow them in case the cache size is given, too. Fixed in r829362