Re: [Vnc2swf-users] memory error using localhost

[Yusuke S. <yu...@cs...>]

Bill, sorry for the late reply.

Bill Seddon <bil...@ly...> wrote:
> I've been able to record a VNC session from remote servers running on
> both Windows 2000 and Windows 2003.  As a result, I have three questions
> I'd welcome your comments on:
> 
> 1) Memory error

Well, I've heard some people still get MemoryError even after they
turned on the local connection. I know that MemoryError occurs
when it tries to read negative length of data from sockets. So
still something gets wrong here...  But so far I couldn't replay
this error on my environment.  As you said, I don't think the
actual memory size is a problem.  I'm still trying to figure this
out, so if you find it works on some other machines and discover
some difference, please let me know.

> 2) Record clipping not yet supported?
> 
> I've tried using the clipping region option on the stream from one of
> the remote servers and it appears to be ignored when recording using the
> VNC recording mode.  Is that right?

Oh, sorry, I just noticed there's a silly bug. Problem 2) and 3)
should be fixed with the following patch. I'll release the fixed
version as soon as possible. Thanks for reporting.

Yusuke

Index: movie.py
===================================================================
RCS file: /cvsroot/vnc2swf/pyvnc2swf/movie.py,v
retrieving revision 1.8
diff -u -r1.8 movie.py
--- movie.py    3 Oct 2005 20:00:11 -0000       1.8
+++ movie.py    16 Oct 2005 16:53:47 -0000
@@ -64,12 +64,14 @@
     # THIS MUST BE CALLED BEFORE MovieOutputStream.open()
     if not self.clipping:
       self.clipping = (0,0,w0,h0)
+    else:
+      (w0,h0) = (self.clipping[2], self.clipping[3])
     if self.scaling:
       (w0,h0) = (int(w0*self.scaling), int(h0*self.scaling))
     if self.width != None and (self.width != w0 or self.height != h0):
       print >>stderr, 'Warning: movie size already set: %dx%d' % (self.width, self.height)
     elif self.width == None:
-      (self.width, self.height) = (w0, h0)
+      (self.width, self.height) = (w0,h0)
       print >>stderr, 'Output movie size: %dx%d' % (self.width, self.height)
     if not self.framerate:
       self.framerate = 12.0