I checked in the fix, but I'm still not able to figure out how to get
the copy filter to engage.  How are you doing that?  None of the
TightVNC family of VNC servers even generates that subencoding on the
server end.  It's not in TightVNC, even as far back as 1.0.


On 11/9/11 3:23 AM, matthieu.lochegn...@altissemiconductor.com wrote:
> 
> DRC <dcomman...@users.sourceforge.net> wrote on 08/11/2011 19:37:33:
> 
>> Odd that valgrind didn't trap it.  Perhaps the low-level datasets aren't
>> ever exercising the Tight Copy filter.
> 
> Hi,
> 
> are you confident with the few lines for the Tight Copy filter, below
> Pierre's fix?
> 
> Those were changed this way by commit 4763 :
> 
>          while (h > 0) {
> -          input->readBytes(ptr, rowSize);
> +          memcpy(ptr, srcPtr, rowSize * sizeof(PIXEL_T));
>           ptr += stride;
> +          srcPtr += w * sizeof(PIXEL_T);
>           h--;
>         }
> 
> I guess the new version does not copy the same number of bytes except in
> 8bpp.
> 
> Maybe this could be written :
>          while (h > 0) {
>            memcpy(ptr, srcPtr, w * sizeof(PIXEL_T));
>            ptr += stride;
>            srcPtr += w * sizeof(PIXEL_T);
>            h--;
>          }
> 
> or :
>          while (h > 0) {
>            memcpy(ptr, srcPtr, rowSize);
>            ptr += stride;
>            srcPtr += rowSize;
>            h--;
>          }
> 
> 
> -- 
> Matthieu Lochegnies
> Altis Semiconductor - http://www.altissemiconductor.com/
> 
> 
> ------------------------------------------------------------------------------
> RSA(R) Conference 2012
> Save $700 by Nov 18
> Register now
> http://p.sf.net/sfu/rsa-sfdev2dev1
> 
> 
> 
> _______________________________________________
> Tigervnc-devel mailing list
> Tigervnc-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/tigervnc-devel

------------------------------------------------------------------------------
RSA(R) Conference 2012
Save $700 by Nov 18
Register now
http://p.sf.net/sfu/rsa-sfdev2dev1
_______________________________________________
Tigervnc-devel mailing list
Tigervnc-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/tigervnc-devel

Reply via email to