[gs-commits] ghostpdl branch, pdfi, updated. jbig2dec-0.14-2191-g32a11ee

[email protected] (Nancy Durgin)
Newsgroups gmane.comp.printing.ghostscript.cvs
Message-ID <[email protected]>
The ghostpdl branch, pdfi has been updated
       via  32a11ee798e3c0bb7c33c9a8286dada7be9e1232 (commit)
      from  ee49a9b7cb87478083c9f35b86fb03dcf5914a1b (commit)

----------------------------------------------------------------------
commit 32a11ee798e3c0bb7c33c9a8286dada7be9e1232
Author: Nancy Durgin <[email protected]>
Date:   Wed Aug 28 09:11:51 2019 -0700

    Disable "GrayBackground" in SMask for now (set to 0)
    
    This value is supposed to be "currentgray" which is apparently
    a lot more complicated to calculate than one might suppose.
    Default it back to 0, which gets rid of extra gray bars on some samples,
    but breaks a few other ones.  Need to fix this later.
    
    See pdf_draw.ps/gssmask, zbegintransparencymaskgroup(), and zcurrentgray().
    
    Test sample: ~/work/samples/tests_private/comparefiles/Bug694293.pdf

diff --git a/pdf/pdf_trans.c b/pdf/pdf_trans.c
index 05cdc00..324ef8a 100644
--- a/pdf/pdf_trans.c
+++ b/pdf/pdf_trans.c
@@ -160,10 +160,15 @@ static int pdfi_trans_set_mask(pdf_context *ctx, pdfi_int_gstate *igs, int color
                 if (code < 0)
                     break;
                 params.Background[i] = (float)num;
-                if (i == 0)
-                    params.GrayBackground = (float)num;
             }
             params.Background_components = pdfi_array_size(BC);
+
+            /* TODO: Not sure how to handle this...  recheck PS code (pdf_draw.ps/gssmask) */
+            /* This should be "currentgray" for the color that we put in params.ColorSpace,
+             * It looks super-convoluted to actually get this value.  Really?
+             * (see zcurrentgray())
+             */
+            params.GrayBackground = 0; /* TODO */
         }
 
         code = gs_begin_transparency_mask(ctx->pgs, &params, &bbox, true);


Summary of changes:
 pdf/pdf_trans.c | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.