Re: Another patch with a massive rendering difference.

Igor,

Once again, before the previous regressions have been examined, you
have committed a patch which introduces (expected) massive regression
differences. During the support call, Jack said that he would examine
the output from the previous set of regressions and report on any
that did not seem trivial or improved. Your committment of this patch
on top of the pending regressions greatly complicates his effort.

In this case it does not make sense that Courier vs. NimbusMonL-Regu
in the PDF file would cause *ANY* regression difference since the
same font (NimbusMonL-Regu) is used to render the PDF and the PS,
thus the raster image should not change. ONLY those tests that
display the font name on the page (as some Genoa tests do) can
reasonably be expected to change the raster image.

To what do you attribute the massive expected regressions for this
patch?

I think that it is reasonable for part of your bug fix task to be
examination of the changed files to insure that the differences
do not introduce an actual regression. Jack is only 1/2 time and
also does not have your insight into the nature of expected
differences in order to verify that differences are either trivial
or are improvements.

Regards,
Ray
__________________________________________________________________

Igor V. Melichev wrote:

> The patches 
> http://www.ghostscript.com/pipermail/gs-cvs/2003-October/003734.html
> http://www.ghostscript.com/pipermail/gs-cvs/2003-October/003735.html
> force pdfwrite to recognize /NimbusMonL-Regu as an alias for /Courier,
> and so on. Therefore those fonts are not linger embedded.
> It caused a rendering difference of the generated PDFs for the files
> 
> 0.pdf 
> 000040cf.000_60.pdf 
> 001-01.ps 
> 001-13.ps 
> 001-25.ps 
> 002-09.ps 
> 002-21.ps 
> 002-33.ps 
> 003-01.ps 
> 012-01.ps 
> 012-05.ps 
> 012-09.ps 
> 012-13.ps 
> 013-01.ps 
> 013-05.ps 
> 013-09.ps 
> 013-13.ps 
> 014-01.ps 
> 014-05.ps 
> 014-09.ps 
> 014-13.ps 
> 015-01.ps 
> 015-05.ps 
> 015-09.ps 
> 016-01.ps 
> 017-01.ps 
> 018-01.ps 
> 020-01.ps 
> 021-01.ps 
> 023-01.ps 
> 027-01.ps 
> 027-05.ps 
> 027-09.ps 
> 027-13.ps 
> 028-01.ps 
> 031-01.ps 
> 031-05.ps 
> 032-01.ps 
> 032-07.ps 
> 033-01.ps 
> 034-01.ps 
> 034-10.ps 
> 035-01.ps 
> 035-07.ps 
> 036-01.ps 
> 038-01.ps 
> 039-01.ps 
> 040-01.ps 
> 045-01.ps 
> 046-01.ps 
> 050-01.ps 
> 055-01.ps 
> 061-01.ps 
> 070-01.ps 
> 076-01.ps 
> 083-01.ps 
> 083-05.ps 
> 083-09.ps 
> 083-13.ps 
> 084-01.ps 
> 091-01.ps 
> 093-01.ps 
> 094-01.ps 
> 096-01.ps 
> 102-01.ps 
> 103-01.ps 
> 104-01.ps 
> 109-01.ps 
> 110-01.ps 
> 113-01.ps 
> 118-01.ps 
> 119-01.ps 
> 119-10.ps 
> 119-16.ps 
> 119-23.ps 
> 119-35.ps 
> 119-41.ps 
> 119-47.ps 
> 120-01.ps 
> 123-01.ps 
> 123-05.ps 
> 123-09.ps 
> 124-01.ps 
> 129-01.ps 
> 136-01.ps 
> 141-01.ps 
> 148-01.ps 
> 148-05.ps 
> 148-11.ps 
> 148-16.ps 
> 149-01.ps 
> 149-05.ps 
> 150-01.ps 
> 154-01.ps 
> 158-01.ps 
> 159.pdf 
> 162-01.ps 
> 165-01.ps 
> 166-01.ps 
> 169-01.ps 
> 169-05.ps 
> 169-09.ps 
> 169-13.ps 
> 170-01.ps 
> 175-01.ps 
> 176-01.ps 
> 177-01.ps 
> 181-01.ps 
> 184-01.ps 
> 192-01.ps 
> 194-01.ps 
> 1_pct_50_bang.pdf 
> 200-01.ps 
> 205-01.ps 
> 205-05.ps 
> 205-09.ps 
> 205-13.ps 
> 206-01.ps 
> 212-01.ps 
> 213-01.ps 
> 214-01.ps 
> 215-01.ps 
> 219-01.ps 
> 220-01.ps 
> 220-07.ps 
> 222-01.ps 
> 222-05.ps 
> 222-09.ps 
> 222-13.ps 
> 223-01.ps 
> 226-01.ps 
> 227-01.ps 
> 231-01.ps 
> 233-01.ps 
> 237-01.ps 
> 238-01.ps 
> 244-01.ps 
> 245-01.ps 
> 245-07.ps 
> 245-13.ps 
> 245-17.ps 
> 246-01.ps 
> 250-01.ps 
> 251-01.ps 
> 255-01.ps 
> 257-01.ps 
> 258-01.ps 
> 260-01.ps 
> 263-01.ps 
> 264-01.ps 
> 265-01.ps 
> 268-02.ps 
> 268-03.ps 
> 268-04.ps 
> 268-05.ps 
> 268-06.ps 
> 268-10.ps 
> 269-01.ps 
> 270-01.ps 
> 272-01.ps 
> 276-01.ps 
> 281-01.ps 
> 282-01.ps 
> 289-01.ps 
> 296-01.ps 
> 297-01.ps 
> 298-01.ps 
> 298-05.ps 
> 298-09.ps 
> 299-01.ps 
> 303-01.ps 
> 304-01.ps 
> 307-01.ps 
> 307-07.ps 
> 307-13.ps 
> 308-04.ps 
> 310-04.ps 
> 311-03.ps 
> 313-01.ps 
> 316-07.ps 
> 320-01.ps 
> 321-01.ps 
> 321-05.ps 
> 321-09.ps 
> 321-13.ps 
> 322-01.ps 
> 325-01.ps 
> 327-01.ps 
> 328-01.ps 
> 330-01.ps 
> 334-01.ps 
> 335-01.ps 
> 336-01.ps 
> 401-01.ps 
> 405-01.ps 
> 409-01.ps 
> 415-01.ps 
> 421-01.ps 
> 430-01.ps 
> 438-01.ps 
> 442-01.ps 
> 446-01.ps 
> 450-01.ps 
> 460-01.ps 
> 463-01.ps 
> 464-01.ps 
> 468-01.ps 
> 473-01.ps 
> 476-01.ps 
> 477-01.ps 
> 477-03.ps 
> 477-04.ps 
> 477-05.ps 
> 477-06.ps 
> 478-01.ps 
> 483-01.ps 
> 483-05.ps 
> 541.pdf 
> 541_623.pdf 
> 86554321.pdf 
> aaon97_p7.pdf 
> acrobat.pdf 
> adesso1.pdf 
> adesso2.pdf 
> adesso3.pdf 
> adesso5.pdf 
> adesso6.pdf 
> adesso7.pdf 
> adesso8.pdf 
> ADOBE1-4.pdf 
> AdobeLic.pdf 
> ats-3-5-99.pdf 
> bc_02a01_cal.pdf 
> brochurep1.pdf 
> bug-gstest.pdf 
> ca.pdf 
> ccc.pdf 
> chess.ps 
> colorcir.ps 
> Dave_Barry.pdf 
> DisplayedGood.pdf 
> DisplayedWrong.pdf 
> Encoding.ps 
> foo.pdf 
> frosch.pdf 
> GS_Stroke_error.pdf 
> H00216q.pdf 
> Jahr2000.pdf 
> js.pdf 
> kazmir.pdf 
> LD.pdf 
> messenger.pdf 
> messenger16.pdf 
> mspro.pdf 
> NECPNTD.pdf 
> new_rect.pdf 
> new_rect_nr.pdf 
> ngnews.pdf 
> ngnews1.pdf 
> p2b-100.pdf 
> pdftops.pdf 
> prfmm.pdf 
> pstopdf.pdf 
> pstopdfO3.pdf 
> QA_Inv.pdf 
> RealCities.pdf 
> rf1025.pdf 
> smdf.90441.102.pdf 
> Svd.pdf 
> test3.pdf 
> TextRize.pdf 
> textsavematrix.pdf 
> two_encodings.ps 
> Vu989qfj.pdf 
> waterfal.ps 
> xgfddg.pdf 
> xngnews.pdf 
> 
> Actually I'm not clear why they render differently when embedding,
> but the new rasters look better and this definitely is a progression.
> 
> Igor.
> 
> 
> 
> _______________________________________________
> gs-code-review mailing list
> [email protected]
> http://www.ghostscript.com/mailman/listinfo/gs-code-review
> 
>