Hi, (no need to CC me, I'm subscribed to the list already.)
In data domenica 9 ottobre 2016 15:30:44 CEST, Thomas Freitag ha scritto:
> That's not correct, Pino
No, my change is correct.
> If you take a deeper look at this error message, the %d is not a
> placeholder for a number, the percent sign should be printed. And to asure
> this it's escaped by a second percent sign.
error() uses GooString::format(), which uses its own syntax for
placeholders, described also in the comment above the
GooString::format() declaration in GooString.h. % is not used at all
by this function, and thus it was printed twice.
In this specific example:
> - error(errSyntaxError, -1, "'{0:s}' must contain '%%d' if more than one
> page should be extracted", destFileName);
> + error(errSyntaxError, -1, "'{0:s}' must contain '%d' if more than one
> page should be extracted", destFileName);
the placeholder for "destFileName" is the "{0:s}" part in the error
string.
--
Pino Toscano
signature.asc
Description: This is a digitally signed message part.
_______________________________________________ poppler mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/poppler
