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