<div dir="ltr"><div><div><div><div><div>I mean it's confusing and not well documented. <br></div>Might waste people's time in the following work of poppler, or potential bugs etc.<br><br></div>At least a warning comment there would be helpful.<br>


<br></div>But as you mentioned, shiftCTMAndClip is good enough to me.<br><br><br><br><br></div>regards,<br></div>- Lu</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Sat, Jan 26, 2013 at 8:56 PM, Albert Astals Cid <span dir="ltr"><<a href="mailto:aacid@kde.org" target="_blank">aacid@kde.org</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">El Dissabte, 26 de gener de 2013, a les 20:40:14, 王璐 va escriure:<br>
<div><div class="h5">> Hi all,<br>
><br>
>    In GfxState.h, this function is declared right after setCTM and<br>
> concatCTM, which looks like no big deal than the other 2, but actually also<br>
> modifies clip BBox of the State<br>
><br>
>    The only place this function is used is in the functions handling<br>
> transparency groups in SplashOutputDev.cc. I'm not familiar with that part,<br>
> but looks like an ad hoc hack.<br>
><br>
>    I found this while I want to track all functions that might update clip<br>
> BBox, this does not make sense to me. Although I've got no better solution,<br>
> at least it might be more reasonable if we add a new backdoor for clip BBox<br>
> and change shiftCTM to what it looks like. In this way SplashOutputDev<br>
> still takes the responsibility, but GfxState is relieved.<br>
><br>
>    What do you think ? I can create a patch.<br>
<br>
</div></div>Why do you think this is a problem?<br>
<br>
<br>
To be honest i don't see any need for a change, all these functions are<br>
internal and not supposed to be used outside the inner world of poppler<br>
itself, so the function is doing what it should do.<br>
<br>
At most we could rename the function, and name it something like<br>
shiftCTMAndClip or something, would that work for you?<br>
<br>
Cheers,<br>
  Albert<br>
<br>
><br>
>    regards,<br>
>    - Lu<br>
_______________________________________________<br>
poppler mailing list<br>
<a href="mailto:poppler@lists.freedesktop.org">poppler@lists.freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/poppler" target="_blank">http://lists.freedesktop.org/mailman/listinfo/poppler</a><br>
</blockquote></div><br></div>