<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.0//EN" "http://www.w3.org/TR/REC-html40/strict.dtd">
<html><head><meta name="qrichtext" content="1" /><style type="text/css">
p, li { white-space: pre-wrap; }
</style></head><body style=" font-family:'Sans Serif'; font-size:10pt; font-weight:400; font-style:normal;">
<p style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;">Andoni, thank you very much for detailed information!</p>
<p style="-qt-paragraph-type:empty; margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; "> </p>
<p style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;">So, I'll keep RUN_NS_APP_THREAD intact and remove 'have-ns-view' processing in 1.0 branch.</p>
<p style="-qt-paragraph-type:empty; margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; "> </p>
<p style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;">Here're bug reports I created for the problems I address:</p>
<p style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;">https://bugzilla.gnome.org/show_bug.cgi?id=703730</p>
<p style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;">https://bugzilla.gnome.org/show_bug.cgi?id=703729</p>
<p style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;">https://bugzilla.gnome.org/show_bug.cgi?id=691419</p>
<p style="-qt-paragraph-type:empty; margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; "> </p>
<p style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;">And I also plan to sort out this issue:</p>
<p style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;">https://bugzilla.gnome.org/show_bug.cgi?id=689730</p>
<p style="-qt-paragraph-type:empty; margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; "> </p>
<p style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;">В письме от 06 июл 2013 14:44:18 Вы написали:</p>
<p style=" margin-top:0px; margin-bottom:0px; margin-left:40px; margin-right:40px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;">GStreamer apps can run without a glib main loop running. We need the NS run loop for Cocoa when the sink needs to create its own window and there isn't any NS run loop running, like for instance with gst-launch.<br />This conditional code is used for OS X <= 10.5 (<a href="http://cgit.freedesktop.org/gstreamer-sdk/gst-plugins-good/tree/sys/osxvideo/osxvideosink.h#n43"><span style=" text-decoration: underline; color:#0057ae;">http://cgit.freedesktop.org/gstreamer-sdk/gst-plugins-good/tree/sys/osxvideo/osxvideosink.h#n43</span></a>) </p>
<p style="-qt-paragraph-type:empty; margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; "> </p>
<p style=" margin-top:0px; margin-bottom:0px; margin-left:40px; margin-right:40px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;">Thanks for clarification<br /></p>
<p style=" margin-top:0px; margin-bottom:0px; margin-left:44px; margin-right:40px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><br />2. When ontaining a window to display video<br />(gst_osx_video_sink_osxwindow_create() function in osxvideosink.m) it first<br />posts 'have-ns-view' message in hope that someone would receive it and take<br />inner GLView (actually responsible for displaying video) to subviews. 'Embed'<br />property also serves this purpose. Is it really needed functionality given<br />that we have GstVideoOverlay interface with methods exactly for this case? The<br />problem is that it obfuscates the code of<br />gst_osx_video_sink_osxwindow_create() considerably as we have to wait for<br />answer both on 'have-ns-view' message and<br />gst_video_overlay_prepare_window_handle() there.<br /></p>
<p style=" margin-top:0px; margin-bottom:0px; margin-left:40px; margin-right:40px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><br /></p>
<p style=" margin-top:0px; margin-bottom:0px; margin-left:40px; margin-right:40px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;">This was kept in 0.10 for backwards compatibility in the early stage of this sink and it should be removed in 1.0 <br /></p>
<p style=" margin-top:0px; margin-bottom:0px; margin-left:44px; margin-right:40px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><br />3. Would anyone mind if we split osxvideosink.h(.m) to two files with one have<br />C code of helper functions and another one have Objective C code of<br />GstOSXVideoSinkObject and some other objects? I think, it could make the whole<br />code simpler.<br /></p>
<p style=" margin-top:0px; margin-bottom:0px; margin-left:40px; margin-right:40px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><br /></p>
<p style=" margin-top:0px; margin-bottom:0px; margin-left:40px; margin-right:40px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;">Sure, go ahead :)<br /></p>
<p style=" margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><br /><br /></p></body></html>