<div dir="ltr"><div class="gmail_default" style="font-family:arial,helvetica,sans-serif;font-size:small;color:#000000"><span style="font-family:Arial,Helvetica,sans-serif;color:rgb(34,34,34)">On Tue, Oct 16, 2018 at 4:59 PM Eliot Miranda <<a href="mailto:eliot.miranda@gmail.com" target="_blank">eliot.miranda@gmail.com</a>> wrote:</span><br></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"> <div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr">Hi All,<div><br></div><div>    I notice that primitiveDisplayCharacters calls copyBits for each character when doing blits that don't use copyLoopPixMap (as recorded in the quickBlt variable).  That means for certain usages we're invoking lockSurfaces & unlockSurfaces for each character.  It would be straight-forward to hoist the (un)lockSurfaces sends out of the loop.  Is there any reason not too do this?  i..e is there any reason why one would want (un)lockSurfaces invoked on each character in the quickBlt == false case?</div></div></div></div></div></blockquote><div><br></div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif;font-size:small;color:rgb(0,0,0)">You would have to know the affected area in advance to lock all at once. It that's easy to compute, sure.</div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif;font-size:small;color:rgb(0,0,0)"><br></div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif;font-size:small;color:rgb(0,0,0)">Also, I'm not sure we're even using that primitive with the current fonts?</div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif;font-size:small;color:rgb(0,0,0)"><br></div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif;font-size:small;color:rgb(0,0,0)">- Bert -</div></div></div>