[Sumover-dev] [svn commit] r4701 - in vic/branches/mpeg4: video

sumover-dev at cs.ucl.ac.uk sumover-dev at cs.ucl.ac.uk
Sun Mar 21 02:38:42 GMT 2010


Author: douglask
Date: Sun Mar 21 02:38:42 2010
New Revision: 4701

Modified:
   vic/branches/mpeg4/main.cpp
   vic/branches/mpeg4/video/grabber-macosx.cpp

Log:
Removed MacOS X 10.3 workaround code originally from Aqua AG vic which I initially thought was a Quicktime Pthread related issue and wrote so in the comments.

It was actually to do with the framework initialization not being called. Framework initialization is not required and shouldn't be used with MacOS X 10.3.9 and later.


Modified: vic/branches/mpeg4/main.cpp
==============================================================================
--- vic/branches/mpeg4/main.cpp	(original)
+++ vic/branches/mpeg4/main.cpp	Sun Mar 21 02:38:42 2010
@@ -41,12 +41,6 @@
 #include <string.h>
 #include <ctype.h>
 
-
-#if 0
-   // XXX - MacOS X 10.3 Ptheard workaround
-   extern "C" void find_macosx_capture_devices(void);
-#endif
-
 #ifdef WIN32
 //#include <winsock.h>
 #include <process.h>
@@ -529,10 +523,6 @@
 	opterr = 1;
 #endif
 
-#if 0
-       // XXX - MacOS X 10.3 Ptheard workaround
-       find_macosx_capture_devices();
-#endif
 	// Option list; If letter is followed by ':' then it takes an argument
 	const char* options = 
 		"A:B:C:c:D:d:f:F:HI:i:j:K:lL:M:m:N:n:o:Pq:QrsST:t:U:u:vV:w:x:X:y";

Modified: vic/branches/mpeg4/video/grabber-macosx.cpp
==============================================================================
--- vic/branches/mpeg4/video/grabber-macosx.cpp	(original)
+++ vic/branches/mpeg4/video/grabber-macosx.cpp	Sun Mar 21 02:38:42 2010
@@ -629,16 +629,7 @@
 };
 
 // Instantiate the video capture device.
-#if 1
-// this doesn't work with MacOS X 10.3 (QuickTime pthread problem),
-// but works fine with 10.4
 static MacOSXScanner find_capture_devices;
-#else
-// this C function is called from main as a workaround for MacOS X 10.3
-extern "C" void find_macosx_capture_devices(void) {
-    MacOSXScanner macOSXScanner;
-}
-#endif
 
 MacOSXScanner::MacOSXScanner()
 {



More information about the Sumover-dev mailing list