Thread: [Mplayerplug-in-devel] some more glitches
Brought to you by:
kdekorte
From: John M. <tt...@te...> - 2004-09-10 14:42:22
|
Hey, I just got a second monitor, and now I am using xinerama. When I go in to full screen mode it now takes up both monitors... which sucks. In contrast totem just fills whatever monitor the totem window is in.. which is nice. Is this something I need to tweak within mplayer or is this in the plugin code? Also, on www.comedycentral.com after a Daily Show clip they have added a commercial that plays (it is a seperate stream now appended to the original clip). The problem is when I pause the clip and start it later, the commercial starts playing and what ever was left of the clip I had paused is cut off. John |
From: John M. <tt...@te...> - 2004-09-10 14:48:39
|
On Thu, 2004-09-09 at 11:14, John McCutchan wrote: > Hey, I just got a second monitor, and now I am using xinerama. > When I go in to full screen mode it now takes up both monitors... which > sucks. In contrast totem just fills whatever monitor the totem window is > in.. which is nice. Is this something I need to tweak within mplayer or > is this in the plugin code? > > Also, on www.comedycentral.com after a Daily Show clip they have added a > commercial that plays (it is a seperate stream now appended to the > original clip). The problem is when I pause the clip and start it later, > the commercial starts playing and what ever was left of the clip I had > paused is cut off. this was a bug on comedycentral.com's site. it has been fixed. John |
From: Kevin D. <kde...@ya...> - 2004-09-10 19:17:47
|
John, The plugin code is what causes the full screen to overlap. Since I don't have one or have access to a machine with dual screens this is almost impossible for me to debug. However, I can tell you where to tweak the code. And if you get something that works I will take a patch. At line 1914 in plugin.cpp there is this block of code #ifdef GTK_ENABLED XReparentWindow(GDK_WINDOW_XDISPLAY(gtkwidget->window), GDK_WINDOW_XWINDOW(gtkwidget->window), GDK_ROOT_WINDOW(), 0, 0); // need to resize here #ifdef GTK2_ENABLED xdisp = GDK_WINDOW_XDISPLAY(gtkwidget->window); gdisp = gdk_x11_lookup_xdisplay(xdisp); screen = gdk_display_get_default_screen(gdisp); x = gdk_screen_get_width(screen); y = gdk_screen_get_height(screen); #endif #ifdef GTK1_ENABLED x = gdk_screen_width(); y = gdk_screen_height(); #endif That is what controls the full screen resize. If you are using the gtk1 widget set it is not xinerama aware and there is no way to get it right. Kevin On Thursday 09 September 2004 09:14 am, John McCutchan wrote: > Hey, I just got a second monitor, and now I am using xinerama. > When I go in to full screen mode it now takes up both monitors... which > sucks. In contrast totem just fills whatever monitor the totem window is > in.. which is nice. Is this something I need to tweak within mplayer or > is this in the plugin code? > |
From: Kevin D. <kde...@ya...> - 2004-09-10 20:01:30
|
John, Well this question has come up before and I didn't have time or energy to look into it. But i did have the time and energy adn so since it was fresh on my mind. I did some research (looked at the totem code, which was helpful) and made a patch and it appears to work on my single head machine. Can you try CVS and see if it works on your dual head box. Kevin On Friday 10 September 2004 01:17 pm, Kevin DeKorte wrote: > John, > > The plugin code is what causes the full screen to overlap. Since I don't > have one or have access to a machine with dual screens this is almost > impossible for me to debug. However, I can tell you where to tweak the > code. And if you get something that works I will take a patch. > > At line 1914 in plugin.cpp there is this block of code > > #ifdef GTK_ENABLED > XReparentWindow(GDK_WINDOW_XDISPLAY(gtkwidget->window), > GDK_WINDOW_XWINDOW(gtkwidget->window), > GDK_ROOT_WINDOW(), 0, 0); > // need to resize here > #ifdef GTK2_ENABLED > xdisp = GDK_WINDOW_XDISPLAY(gtkwidget->window); > gdisp = gdk_x11_lookup_xdisplay(xdisp); > screen = gdk_display_get_default_screen(gdisp); > x = gdk_screen_get_width(screen); > y = gdk_screen_get_height(screen); > #endif > #ifdef GTK1_ENABLED > x = gdk_screen_width(); > y = gdk_screen_height(); > #endif > > > That is what controls the full screen resize. If you are using the gtk1 > widget set it is not xinerama aware and there is no way to get it right. > > Kevin > > On Thursday 09 September 2004 09:14 am, John McCutchan wrote: > > Hey, I just got a second monitor, and now I am using xinerama. > > When I go in to full screen mode it now takes up both monitors... which > > sucks. In contrast totem just fills whatever monitor the totem window is > > in.. which is nice. Is this something I need to tweak within mplayer or > > is this in the plugin code? > > ------------------------------------------------------- > This SF.Net email is sponsored by: YOU BE THE JUDGE. Be one of 170 > Project Admins to receive an Apple iPod Mini FREE for your judgement on > who ports your project to Linux PPC the best. Sponsored by IBM. > Deadline: Sept. 13. Go here: http://sf.net/ppc_contest.php > _______________________________________________ > Mplayerplug-in-devel mailing list > Mpl...@li... > https://lists.sourceforge.net/lists/listinfo/mplayerplug-in-devel |
From: John M. <tt...@te...> - 2004-09-14 13:03:37
|
One more thing, it seems to stay on all virtual desktops. In totem if I switch to a different workspace it stays in the workspace it was opened in. John On Fri, 2004-09-10 at 16:01, Kevin DeKorte wrote: > John, > > Well this question has come up before and I didn't have time or energy to look > into it. But i did have the time and energy adn so since it was fresh on my > mind. I did some research (looked at the totem code, which was helpful) and > made a patch and it appears to work on my single head machine. Can you try > CVS and see if it works on your dual head box. > > Kevin > > > On Friday 10 September 2004 01:17 pm, Kevin DeKorte wrote: > > John, > > > > The plugin code is what causes the full screen to overlap. Since I don't > > have one or have access to a machine with dual screens this is almost > > impossible for me to debug. However, I can tell you where to tweak the > > code. And if you get something that works I will take a patch. > > > > At line 1914 in plugin.cpp there is this block of code > > > > #ifdef GTK_ENABLED > > XReparentWindow(GDK_WINDOW_XDISPLAY(gtkwidget->window), > > GDK_WINDOW_XWINDOW(gtkwidget->window), > > GDK_ROOT_WINDOW(), 0, 0); > > // need to resize here > > #ifdef GTK2_ENABLED > > xdisp = GDK_WINDOW_XDISPLAY(gtkwidget->window); > > gdisp = gdk_x11_lookup_xdisplay(xdisp); > > screen = gdk_display_get_default_screen(gdisp); > > x = gdk_screen_get_width(screen); > > y = gdk_screen_get_height(screen); > > #endif > > #ifdef GTK1_ENABLED > > x = gdk_screen_width(); > > y = gdk_screen_height(); > > #endif > > > > > > That is what controls the full screen resize. If you are using the gtk1 > > widget set it is not xinerama aware and there is no way to get it right. > > > > Kevin > > > > On Thursday 09 September 2004 09:14 am, John McCutchan wrote: > > > Hey, I just got a second monitor, and now I am using xinerama. > > > When I go in to full screen mode it now takes up both monitors... which > > > sucks. In contrast totem just fills whatever monitor the totem window is > > > in.. which is nice. Is this something I need to tweak within mplayer or > > > is this in the plugin code? > > > > ------------------------------------------------------- > > This SF.Net email is sponsored by: YOU BE THE JUDGE. Be one of 170 > > Project Admins to receive an Apple iPod Mini FREE for your judgement on > > who ports your project to Linux PPC the best. Sponsored by IBM. > > Deadline: Sept. 13. Go here: http://sf.net/ppc_contest.php > > _______________________________________________ > > Mplayerplug-in-devel mailing list > > Mpl...@li... > > https://lists.sourceforge.net/lists/listinfo/mplayerplug-in-devel > > > ------------------------------------------------------- > This SF.Net email is sponsored by: YOU BE THE JUDGE. Be one of 170 > Project Admins to receive an Apple iPod Mini FREE for your judgement on > who ports your project to Linux PPC the best. Sponsored by IBM. > Deadline: Sept. 13. Go here: http://sf.net/ppc_contest.php > _______________________________________________ > Mplayerplug-in-devel mailing list > Mpl...@li... > https://lists.sourceforge.net/lists/listinfo/mplayerplug-in-devel > |
From: John M. <tt...@te...> - 2004-09-14 13:01:56
|
Hey, it works much better now, the only problem is that it always goes to monitor 1. If totem is on monitor 2 it will fullscreen there. Any progress on the crasher? John On Fri, 2004-09-10 at 16:01, Kevin DeKorte wrote: > John, > > Well this question has come up before and I didn't have time or energy to look > into it. But i did have the time and energy adn so since it was fresh on my > mind. I did some research (looked at the totem code, which was helpful) and > made a patch and it appears to work on my single head machine. Can you try > CVS and see if it works on your dual head box. > > Kevin > > > On Friday 10 September 2004 01:17 pm, Kevin DeKorte wrote: > > John, > > > > The plugin code is what causes the full screen to overlap. Since I don't > > have one or have access to a machine with dual screens this is almost > > impossible for me to debug. However, I can tell you where to tweak the > > code. And if you get something that works I will take a patch. > > > > At line 1914 in plugin.cpp there is this block of code > > > > #ifdef GTK_ENABLED > > XReparentWindow(GDK_WINDOW_XDISPLAY(gtkwidget->window), > > GDK_WINDOW_XWINDOW(gtkwidget->window), > > GDK_ROOT_WINDOW(), 0, 0); > > // need to resize here > > #ifdef GTK2_ENABLED > > xdisp = GDK_WINDOW_XDISPLAY(gtkwidget->window); > > gdisp = gdk_x11_lookup_xdisplay(xdisp); > > screen = gdk_display_get_default_screen(gdisp); > > x = gdk_screen_get_width(screen); > > y = gdk_screen_get_height(screen); > > #endif > > #ifdef GTK1_ENABLED > > x = gdk_screen_width(); > > y = gdk_screen_height(); > > #endif > > > > > > That is what controls the full screen resize. If you are using the gtk1 > > widget set it is not xinerama aware and there is no way to get it right. > > > > Kevin > > > > On Thursday 09 September 2004 09:14 am, John McCutchan wrote: > > > Hey, I just got a second monitor, and now I am using xinerama. > > > When I go in to full screen mode it now takes up both monitors... which > > > sucks. In contrast totem just fills whatever monitor the totem window is > > > in.. which is nice. Is this something I need to tweak within mplayer or > > > is this in the plugin code? > > > > ------------------------------------------------------- > > This SF.Net email is sponsored by: YOU BE THE JUDGE. Be one of 170 > > Project Admins to receive an Apple iPod Mini FREE for your judgement on > > who ports your project to Linux PPC the best. Sponsored by IBM. > > Deadline: Sept. 13. Go here: http://sf.net/ppc_contest.php > > _______________________________________________ > > Mplayerplug-in-devel mailing list > > Mpl...@li... > > https://lists.sourceforge.net/lists/listinfo/mplayerplug-in-devel > > > ------------------------------------------------------- > This SF.Net email is sponsored by: YOU BE THE JUDGE. Be one of 170 > Project Admins to receive an Apple iPod Mini FREE for your judgement on > who ports your project to Linux PPC the best. Sponsored by IBM. > Deadline: Sept. 13. Go here: http://sf.net/ppc_contest.php > _______________________________________________ > Mplayerplug-in-devel mailing list > Mpl...@li... > https://lists.sourceforge.net/lists/listinfo/mplayerplug-in-devel > |
From: Kevin D. <kde...@ya...> - 2004-09-14 15:55:04
|
John, Ok, I just made some more Xinerama fixes hopefully it will work now. Again I don't have Xinerama so testing this is kinda hit and miss. I'm not sure I got the virtual desktop issue fixed, but the monitor position should be fixed now. I changed a lot of the fullscreen creation and positioning code so hopefully this does it. CVS should have it by noon MST on 9/14. As for the crasher. I took another look at your stack trace and I'm not sure what it going on there.. The areas where it is crashing is in fork, pthread_cond_timedwait, poll and pthread_setcancelstate (all in libc). Now the only code that I can see in the BT that is in the plugin seems to be in the mypopen which calls fork and we really have not seen any issues with that in a long ( almost a year) time. Also, I watched probably 10 to 15 videos on comedycentral in row.. watching them to completion and interrupting them and even timing it while it was changing media and still could not get it to crash. I am using glibc 2.3.3-27 from Fedora Core. I noticed that your libc says it is 2.3.2. Perhaps there is something in there that needs to be upgraded. Again just a guess. Kevin On Tuesday 14 September 2004 07:17 am, John McCutchan wrote: > Hey, it works much better now, the only problem is that it always goes > to monitor 1. If totem is on monitor 2 it will fullscreen there. > > Any progress on the crasher? > > John > > On Fri, 2004-09-10 at 16:01, Kevin DeKorte wrote: > > John, > > > > Well this question has come up before and I didn't have time or energy to > > look into it. But i did have the time and energy adn so since it was > > fresh on my mind. I did some research (looked at the totem code, which > > was helpful) and made a patch and it appears to work on my single head > > machine. Can you try CVS and see if it works on your dual head box. > > > > Kevin > > > > On Friday 10 September 2004 01:17 pm, Kevin DeKorte wrote: > > > John, > > > > > > The plugin code is what causes the full screen to overlap. Since I > > > don't have one or have access to a machine with dual screens this is > > > almost impossible for me to debug. However, I can tell you where to > > > tweak the code. And if you get something that works I will take a > > > patch. > > > > > > At line 1914 in plugin.cpp there is this block of code > > > > > > #ifdef GTK_ENABLED > > > XReparentWindow(GDK_WINDOW_XDISPLAY(gtkwidget->window), > > > GDK_WINDOW_XWINDOW(gtkwidget->window), > > > GDK_ROOT_WINDOW(), 0, 0); > > > // need to resize here > > > #ifdef GTK2_ENABLED > > > xdisp = GDK_WINDOW_XDISPLAY(gtkwidget->window); > > > gdisp = gdk_x11_lookup_xdisplay(xdisp); > > > screen = gdk_display_get_default_screen(gdisp); > > > x = gdk_screen_get_width(screen); > > > y = gdk_screen_get_height(screen); > > > #endif > > > #ifdef GTK1_ENABLED > > > x = gdk_screen_width(); > > > y = gdk_screen_height(); > > > #endif > > > > > > > > > That is what controls the full screen resize. If you are using the gtk1 > > > widget set it is not xinerama aware and there is no way to get it > > > right. > > > > > > Kevin > > > > > > On Thursday 09 September 2004 09:14 am, John McCutchan wrote: > > > > Hey, I just got a second monitor, and now I am using xinerama. > > > > When I go in to full screen mode it now takes up both monitors... > > > > which sucks. In contrast totem just fills whatever monitor the totem > > > > window is in.. which is nice. Is this something I need to tweak > > > > within mplayer or is this in the plugin code? > > > > > > ------------------------------------------------------- > > > This SF.Net email is sponsored by: YOU BE THE JUDGE. Be one of 170 > > > Project Admins to receive an Apple iPod Mini FREE for your judgement on > > > who ports your project to Linux PPC the best. Sponsored by IBM. > > > Deadline: Sept. 13. Go here: http://sf.net/ppc_contest.php > > > _______________________________________________ > > > Mplayerplug-in-devel mailing list > > > Mpl...@li... > > > https://lists.sourceforge.net/lists/listinfo/mplayerplug-in-devel > > > > ------------------------------------------------------- > > This SF.Net email is sponsored by: YOU BE THE JUDGE. Be one of 170 > > Project Admins to receive an Apple iPod Mini FREE for your judgement on > > who ports your project to Linux PPC the best. Sponsored by IBM. > > Deadline: Sept. 13. Go here: http://sf.net/ppc_contest.php > > _______________________________________________ > > Mplayerplug-in-devel mailing list > > Mpl...@li... > > https://lists.sourceforge.net/lists/listinfo/mplayerplug-in-devel > > ------------------------------------------------------- > This SF.Net email is sponsored by: YOU BE THE JUDGE. Be one of 170 > Project Admins to receive an Apple iPod Mini FREE for your judgement on > who ports your project to Linux PPC the best. Sponsored by IBM. > Deadline: Sept. 13. Go here: http://sf.net/ppc_contest.php > _______________________________________________ > Mplayerplug-in-devel mailing list > Mpl...@li... > https://lists.sourceforge.net/lists/listinfo/mplayerplug-in-devel |
From: Kevin D. <kde...@ya...> - 2004-09-14 17:26:25
|
John, Ok, I have the virtual desktop switch fixed now... There is only one area that might be a problem and the only way to know is on a Xinerama desktop. When the gtk_window_fullscreen command is called I want to know if the newly create window covers both monitors or only one monitor. If it does both I need to know so that I can work something out. But I believe the final Xinerama issue has been fixed with the gtk2 widgets. Kevin On Tuesday 14 September 2004 09:53 am, Kevin DeKorte wrote: > John, > > Ok, I just made some more Xinerama fixes hopefully it will work now. Again > I don't have Xinerama so testing this is kinda hit and miss. I'm not sure I > got the virtual desktop issue fixed, but the monitor position should be > fixed now. I changed a lot of the fullscreen creation and positioning code > so hopefully this does it. CVS should have it by noon MST on 9/14. > > As for the crasher. I took another look at your stack trace and I'm not > sure what it going on there.. The areas where it is crashing is in fork, > pthread_cond_timedwait, poll and pthread_setcancelstate (all in libc). Now > the only code that I can see in the BT that is in the plugin seems to be in > the mypopen which calls fork and we really have not seen any issues with > that in a long ( almost a year) time. Also, I watched probably 10 to 15 > videos on comedycentral in row.. watching them to completion and > interrupting them and even timing it while it was changing media and still > could not get it to crash. I am using glibc 2.3.3-27 from Fedora Core. I > noticed that your libc says it is 2.3.2. Perhaps there is something in > there that needs to be upgraded. Again just a guess. > > Kevin > > On Tuesday 14 September 2004 07:17 am, John McCutchan wrote: > > Hey, it works much better now, the only problem is that it always goes > > to monitor 1. If totem is on monitor 2 it will fullscreen there. > > > > Any progress on the crasher? > > > > John > > > > On Fri, 2004-09-10 at 16:01, Kevin DeKorte wrote: > > > John, > > > > > > Well this question has come up before and I didn't have time or energy > > > to look into it. But i did have the time and energy adn so since it was > > > fresh on my mind. I did some research (looked at the totem code, which > > > was helpful) and made a patch and it appears to work on my single head > > > machine. Can you try CVS and see if it works on your dual head box. > > > > > > Kevin > > > > > > On Friday 10 September 2004 01:17 pm, Kevin DeKorte wrote: > > > > John, > > > > > > > > The plugin code is what causes the full screen to overlap. Since I > > > > don't have one or have access to a machine with dual screens this is > > > > almost impossible for me to debug. However, I can tell you where to > > > > tweak the code. And if you get something that works I will take a > > > > patch. > > > > > > > > At line 1914 in plugin.cpp there is this block of code > > > > > > > > #ifdef GTK_ENABLED > > > > XReparentWindow(GDK_WINDOW_XDISPLAY(gtkwidget->window), > > > > GDK_WINDOW_XWINDOW(gtkwidget->window), > > > > GDK_ROOT_WINDOW(), 0, 0); > > > > // need to resize here > > > > #ifdef GTK2_ENABLED > > > > xdisp = GDK_WINDOW_XDISPLAY(gtkwidget->window); > > > > gdisp = gdk_x11_lookup_xdisplay(xdisp); > > > > screen = gdk_display_get_default_screen(gdisp); > > > > x = gdk_screen_get_width(screen); > > > > y = gdk_screen_get_height(screen); > > > > #endif > > > > #ifdef GTK1_ENABLED > > > > x = gdk_screen_width(); > > > > y = gdk_screen_height(); > > > > #endif > > > > > > > > > > > > That is what controls the full screen resize. If you are using the > > > > gtk1 widget set it is not xinerama aware and there is no way to get > > > > it right. > > > > > > > > Kevin > > > > > > > > On Thursday 09 September 2004 09:14 am, John McCutchan wrote: > > > > > Hey, I just got a second monitor, and now I am using xinerama. > > > > > When I go in to full screen mode it now takes up both monitors... > > > > > which sucks. In contrast totem just fills whatever monitor the > > > > > totem window is in.. which is nice. Is this something I need to > > > > > tweak within mplayer or is this in the plugin code? > > > > > > > > ------------------------------------------------------- > > > > This SF.Net email is sponsored by: YOU BE THE JUDGE. Be one of 170 > > > > Project Admins to receive an Apple iPod Mini FREE for your judgement > > > > on who ports your project to Linux PPC the best. Sponsored by IBM. > > > > Deadline: Sept. 13. Go here: http://sf.net/ppc_contest.php > > > > _______________________________________________ > > > > Mplayerplug-in-devel mailing list > > > > Mpl...@li... > > > > https://lists.sourceforge.net/lists/listinfo/mplayerplug-in-devel > > > > > > ------------------------------------------------------- > > > This SF.Net email is sponsored by: YOU BE THE JUDGE. Be one of 170 > > > Project Admins to receive an Apple iPod Mini FREE for your judgement on > > > who ports your project to Linux PPC the best. Sponsored by IBM. > > > Deadline: Sept. 13. Go here: http://sf.net/ppc_contest.php > > > _______________________________________________ > > > Mplayerplug-in-devel mailing list > > > Mpl...@li... > > > https://lists.sourceforge.net/lists/listinfo/mplayerplug-in-devel > > > > ------------------------------------------------------- > > This SF.Net email is sponsored by: YOU BE THE JUDGE. Be one of 170 > > Project Admins to receive an Apple iPod Mini FREE for your judgement on > > who ports your project to Linux PPC the best. Sponsored by IBM. > > Deadline: Sept. 13. Go here: http://sf.net/ppc_contest.php > > _______________________________________________ > > Mplayerplug-in-devel mailing list > > Mpl...@li... > > https://lists.sourceforge.net/lists/listinfo/mplayerplug-in-devel > > ------------------------------------------------------- > This SF.Net email is sponsored by: thawte's Crypto Challenge Vl > Crack the code and win a Sony DCRHC40 MiniDV Digital Handycam > Camcorder. More prizes in the weekly Lunch Hour Challenge. > Sign up NOW http://ad.doubleclick.net/clk;10740251;10262165;m > _______________________________________________ > Mplayerplug-in-devel mailing list > Mpl...@li... > https://lists.sourceforge.net/lists/listinfo/mplayerplug-in-devel |