New Ghostscript/GhostPDL compiler warnings - 2018-10-10-11:45:08 - 34cc326eb2c5695833361887fe0b32e8d987741c

[email protected] Wed, 10 Oct 2018 13:38:49 -0700 (PDT)
Newsgroups gmane.comp.printing.ghostscript.regression
Message-ID <20181010203850.592542040265@i7>
Previous Revision: 04a517f39cc3e25a7aec9766b45aeb2144718ee6
 Current Revision: 34cc326eb2c5695833361887fe0b32e8d987741c

commit 34cc326eb2c5695833361887fe0b32e8d987741c
Author:     Chris Liddell <[email protected]>
AuthorDate: Wed Oct 10 15:38:10 2018 +0100
CommitDate: Wed Oct 10 17:18:53 2018 +0100

    Bug 699927: don't include operator arrays in execstack output
    
    When we transfer the contents of the execution stack into the array, take the
    extra step of replacing any operator arrays on the stack with the operator
    that reference them.
    
    This prevents the contents of Postscript defined, internal only operators (those
    created with .makeoperator) being exposed via execstack (and thus, via error
    handling).
    
    This necessitates a change in the resource remapping 'resource', which contains
    a procedure which relies on the contents of the operators arrays being present.
    As we already had internal-only variants of countexecstack and execstack
    (.countexecstack and .execstack) - using those, and leaving thier operation
    including the operator arrays means the procedure continues to work correctly.
    
    Both .countexecstack and .execstack are undefined after initialization.
    
    Also, when we store the execstack (or part thereof) for an execstackoverflow
    error, make the same oparray/operator substitution as above for execstack.

Resource/Init/gs_init.ps
Resource/Init/gs_resmp.ps
psi/int.mak
psi/interp.c
psi/interp.h
psi/zcontrol.c



Ghostscript:

new cppcheck warnings (Cppcheck 1.72): 

psi/interp.c:1196: error: Expression 'iref_temp=(const struct ref_s*)iref_packed,iref_temp' depends on order of evaluation of side effects
psi/interp.c:1406: error: Expression 'iref_temp=(const struct ref_s*)iref_packed,iref_temp' depends on order of evaluation of side effects

http://miles.ghostscript.com:8080/artifex/34cc326eb2c5695833361887fe0b32e8d987741c/gs-cppcheck.txt