Bug#747181: xpdf: too many warning messages
Control: reassign -1 libpoppler102 20.09.0-3.1
Control: retitle -1 poppler: many incorrect messages "Syntax Warning: Bad
bounding box in Type 3 glyph"
Control: affects -1 xpdf
On 2021-09-18 10:14:03 +0900, Masanori Goto wrote:
> Thanks for the check. It sounds more poppler side. I just read this
> email, but are you OK to reassign it to poppler?
Doing it now.
> Also it might be better to annotate a prefix like "poppler" in the error in
> general - though it's a bit of a large change...
Yes, but if this can be done in the error() function, that would
be quite a small change.
> 2021年9月17日(金) 22:30 Vincent Lefevre :
>
> > On 2021-09-17 14:40:06 +0200, Vincent Lefevre wrote:
> > > The warning comes from xpdf/SplashOutputDev.cc
> > >
> > > if (xMin - xt < t3Font->glyphX ||
> > > yMin - yt < t3Font->glyphY ||
> > > xMax - xt > t3Font->glyphX + t3Font->glyphW ||
> > > yMax - yt > t3Font->glyphY + t3Font->glyphH) {
> > > if (t3Font->validBBox) {
> > > error(errSyntaxWarning, -1, "Bad bounding box in Type 3 glyph");
> > > }
> > > return;
> > > }
> >
> > Actually it doesn't (commenting out this line doesn't change
> > anything). Perhaps a bug in poppler, then, as it has similar
> > code in poppler/SplashOutputDev.cc:
> >
> > if (xMin - xt < t3Font->glyphX || yMin - yt < t3Font->glyphY || xMax -
> > xt > t3Font->glyphX + t3Font->glyphW || yMax - yt > t3Font->glyphY +
> > t3Font->glyphH) {
> > if (t3Font->validBBox) {
> > error(errSyntaxWarning, -1, "Bad bounding box in Type 3
> > glyph");
> > }
> > return;
> > }
--
Vincent Lefèvre - Web:
Bug#747181: xpdf: too many warning messages
Thanks for the check. It sounds more poppler side. I just read this
email, but are you OK to reassign it to poppler?
Also it might be better to annotate a prefix like "poppler" in the error in
general - though it's a bit of a large change...
2021年9月17日(金) 22:30 Vincent Lefevre :
> On 2021-09-17 14:40:06 +0200, Vincent Lefevre wrote:
> > The warning comes from xpdf/SplashOutputDev.cc
> >
> > if (xMin - xt < t3Font->glyphX ||
> > yMin - yt < t3Font->glyphY ||
> > xMax - xt > t3Font->glyphX + t3Font->glyphW ||
> > yMax - yt > t3Font->glyphY + t3Font->glyphH) {
> > if (t3Font->validBBox) {
> > error(errSyntaxWarning, -1, "Bad bounding box in Type 3 glyph");
> > }
> > return;
> > }
>
> Actually it doesn't (commenting out this line doesn't change
> anything). Perhaps a bug in poppler, then, as it has similar
> code in poppler/SplashOutputDev.cc:
>
> if (xMin - xt < t3Font->glyphX || yMin - yt < t3Font->glyphY || xMax -
> xt > t3Font->glyphX + t3Font->glyphW || yMax - yt > t3Font->glyphY +
> t3Font->glyphH) {
> if (t3Font->validBBox) {
> error(errSyntaxWarning, -1, "Bad bounding box in Type 3
> glyph");
> }
> return;
> }
>
> --
> Vincent Lefèvre - Web:
Bug#747181: xpdf: too many warning messages
On 2021-09-17 14:40:06 +0200, Vincent Lefevre wrote:
> The warning comes from xpdf/SplashOutputDev.cc
>
> if (xMin - xt < t3Font->glyphX ||
> yMin - yt < t3Font->glyphY ||
> xMax - xt > t3Font->glyphX + t3Font->glyphW ||
> yMax - yt > t3Font->glyphY + t3Font->glyphH) {
> if (t3Font->validBBox) {
> error(errSyntaxWarning, -1, "Bad bounding box in Type 3 glyph");
> }
> return;
> }
Actually it doesn't (commenting out this line doesn't change
anything). Perhaps a bug in poppler, then, as it has similar
code in poppler/SplashOutputDev.cc:
if (xMin - xt < t3Font->glyphX || yMin - yt < t3Font->glyphY || xMax - xt >
t3Font->glyphX + t3Font->glyphW || yMax - yt > t3Font->glyphY + t3Font->glyphH)
{
if (t3Font->validBBox) {
error(errSyntaxWarning, -1, "Bad bounding box in Type 3 glyph");
}
return;
}
--
Vincent Lefèvre - Web:
Bug#747181: xpdf: too many warning messages
Control: retitle -1 xpdf: too many incorrect warning messages
Control: severity -1 minor
Hi,
On 2018-08-31 08:10:52 +0900, Masanori Goto wrote:
> Yes, xpdf shows many warnings, and adding "-q" command line option
> should make xpdf silent. Could you try it out? Thanks!
The -q option silences the warnings, but it is described as
Don't print any messages or errors.
i.e. it will also silence the errors, which is not acceptable.
Moreover, there shouldn't be a need to silence anything, as
the PDF file (generated by ps2pdf) appears to be correct;
in particular, atril and xournal don't complain.
The warning comes from xpdf/SplashOutputDev.cc
if (xMin - xt < t3Font->glyphX ||
yMin - yt < t3Font->glyphY ||
xMax - xt > t3Font->glyphX + t3Font->glyphW ||
yMax - yt > t3Font->glyphY + t3Font->glyphH) {
if (t3Font->validBBox) {
error(errSyntaxWarning, -1, "Bad bounding box in Type 3 glyph");
}
return;
}
Note that the values are double's, so that the comparisons may be
subject to floating-point rounding errors. I don't know whether
this is the reason here (I could try to look at this more closely),
but I suppose that this code could issue warnings only because of
that.
If this warning comes from a bug in xpdf, such a bug should be fixed.
--
Vincent Lefèvre - Web:
Bug#747181: xpdf: too many warning messages
Hi, Yes, xpdf shows many warnings, and adding "-q" command line option should make xpdf silent. Could you try it out? Thanks!
Bug#747181: xpdf: too many warning messages
Package: xpdf Version: 3.03-17 Severity: wishlist On some PDF files, xpdf outputs many warning messages, such as: Syntax Warning: Bad bounding box in Type 3 glyph These warnings can fill up the terminal, distract the user, and make xpdf slow (I have an example where there are more that 1000 warnings per page). At most one warning per page should be output, possibly with the number of times it occurred. -- System Information: Debian Release: jessie/sid APT prefers unstable APT policy: (500, 'unstable'), (500, 'testing'), (500, 'stable'), (1, 'experimental') Architecture: amd64 (x86_64) Foreign Architectures: i386 Kernel: Linux 3.14-1-amd64 (SMP w/8 CPU cores) Locale: LANG=POSIX, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8) Shell: /bin/sh linked to /bin/dash Versions of packages xpdf depends on: ii libc6 2.18-5 ii libgcc1 1:4.9.0-2 ii libpoppler44 0.24.5-3 ii libstdc++64.9.0-2 ii libx11-6 2:1.6.2-1 ii libxm42.3.4-5 ii libxt61:1.1.4-1 Versions of packages xpdf recommends: ii cups-bsd 1.7.2-3 ii gsfonts-x110.22 ii poppler-data 0.4.6-5 ii poppler-utils 0.24.5-3 xpdf suggests no packages. -- no debconf information -- To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
