[poppler] GfxState::shiftCTM

[王璐]

Hi all,

   In GfxState.h, this function is declared right after setCTM and
concatCTM, which looks like no big deal than the other 2, but actually also
modifies clip BBox of the State

   The only place this function is used is in the functions handling
transparency groups in SplashOutputDev.cc. I'm not familiar with that part,
but looks like an ad hoc hack.

   I found this while I want to track all functions that might update clip
BBox, this does not make sense to me. Although I've got no better solution,
at least it might be more reasonable if we add a new backdoor for clip BBox
and change shiftCTM to what it looks like. In this way SplashOutputDev
still takes the responsibility, but GfxState is relieved.

   What do you think ? I can create a patch.

   regards,
   - Lu
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/poppler/attachments/20130126/aaf95593/attachment.html>