[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