[poppler] Remove some GlobalParams dependencies from PSOutputDev.
Albert Astals Cid
aacid at kde.org
Thu Jul 28 06:57:50 EST 2005
Having a deeped look at your commit i think it will not work.
You are passing 0, 0, 0, 0 for int imgLLXA, int imgLLYA, int imgURXA, int=20
imgURYA
that triggers=20
if (imgLLX =3D=3D 0 && imgURX =3D=3D 0 && imgLLY =3D=3D 0 && imgURY =3D=3D =
0) {
globalParams->getPSImageableArea(&imgLLX, &imgLLY, &imgURX, &imgURY);
}
that asks for variables that are only modified on=20
GlobalParams::parsePSFileSize, GlobalParams::setPSFileSize, etc.
I'd suggest removing your changes and make the glib binding call
void GlobalParams::setPSPaperWidth(int width);
void GlobalParams::setPSPaperHeight(int height);
that is what we are doing it in kpdf and that sets the said variables=20
accordingly.
Sorry for the late review.
Albert
A Dimecres 27 Juliol 2005 21:27, Martin Kretzschmar va escriure:
> Kristian H=C3=B8gsberg schrieb:
> > Yep, that looks good. As Albert says, there's probably not much reason
> > to keep the xpdfrc code in there, but that's up to you, we can always
> > remove it later.
>
> Okay. I simplified the patch to just add the new parameters to the
> existing constructors (and didn't add new constructor overloads). But I
> didn't do the GlobalParams cleanup.
>
> Regards,
>
> Martin
>
> _______________________________________________
> poppler mailing list
> poppler at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/poppler
______________________________________________
Renovamos el Correo Yahoo!
Nuevos servicios, más seguridad
http://correo.yahoo.es
More information about the poppler
mailing list