annotate src/audacious/playback.c @ 4109:2d6fd3f630f0

Adds playback_free() function to fix memory leak in playback_stop(). (Bugzilla #42)
author Jussi Judin <jjudin+audacious@iki.fi>
date Tue, 25 Dec 2007 00:53:33 -0600
parents 05e6d9db003c
children 2d4b4f13d10d
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
rev   line source
2313
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
1 /* Audacious - Cross-platform multimedia player
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
2 * Copyright (C) 2005-2007 Audacious development team
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
3 *
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
4 * Based on BMP:
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
5 * Copyright (C) 2003-2004 BMP development team.
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
6 *
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
7 * Based on XMMS:
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
8 * Copyright (C) 1998-2003 XMMS development team.
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
9 *
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
10 * This program is free software; you can redistribute it and/or modify
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
11 * it under the terms of the GNU General Public License as published by
3121
3b6d316f8b09 GPL3 relicensing.
William Pitcock <nenolod@atheme-project.org>
parents: 3093
diff changeset
12 * the Free Software Foundation; under version 3 of the License.
2313
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
13 *
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
14 * This program is distributed in the hope that it will be useful,
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
15 * but WITHOUT ANY WARRANTY; without even the implied warranty of
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
16 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
17 * GNU General Public License for more details.
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
18 *
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
19 * You should have received a copy of the GNU General Public License
3121
3b6d316f8b09 GPL3 relicensing.
William Pitcock <nenolod@atheme-project.org>
parents: 3093
diff changeset
20 * along with this program. If not, see <http://www.gnu.org/licenses>.
3123
f1c756f39e6c Invoke "Plugins are not derived work" clause provided by GPL3.
William Pitcock <nenolod@atheme-project.org>
parents: 3121
diff changeset
21 *
f1c756f39e6c Invoke "Plugins are not derived work" clause provided by GPL3.
William Pitcock <nenolod@atheme-project.org>
parents: 3121
diff changeset
22 * The Audacious team does not consider modular code linking to
f1c756f39e6c Invoke "Plugins are not derived work" clause provided by GPL3.
William Pitcock <nenolod@atheme-project.org>
parents: 3121
diff changeset
23 * Audacious or using our public API to be a derived work.
2313
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
24 */
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
25
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
26 #ifdef HAVE_CONFIG_H
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
27 # include "config.h"
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
28 #endif
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
29
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
30 #include <glib.h>
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
31 #include <glib/gi18n.h>
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
32 #include <glib/gprintf.h>
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
33 #include <stdlib.h>
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
34 #include <string.h>
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
35 #include <time.h>
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
36
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
37 #include <unistd.h>
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
38 #include <sys/types.h>
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
39 #include <sys/stat.h>
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
40 #include <dirent.h>
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
41
2717
6884a2144a01 [svn] - make audacious build again
nenolod
parents: 2663
diff changeset
42 #include "configdb.h"
2313
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
43
2420
b23f52fe132f [svn] Removed util.* from libaudacious
mf0102
parents: 2416
diff changeset
44 #include "hook.h"
3152
c9471f83944c Add playback_eof(). Plugins should use this instead of playback->eof = 1.
William Pitcock <nenolod@atheme-project.org>
parents: 3128
diff changeset
45 #include "eventqueue.h"
2313
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
46 #include "input.h"
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
47 #include "main.h"
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
48 #include "ui_equalizer.h"
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
49 #include "output.h"
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
50 #include "playlist.h"
2420
b23f52fe132f [svn] Removed util.* from libaudacious
mf0102
parents: 2416
diff changeset
51 #include "ui_main.h"
2313
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
52 #include "ui_playlist.h"
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
53 #include "ui_skinselector.h"
3072
84de3244aeaa replace Playstatus with UiSkinnedPlaystatus
Tomasz Mon <desowin@gmail.com>
parents: 3054
diff changeset
54 #include "ui_skinned_playstatus.h"
2313
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
55 #include "util.h"
3082
91df04ec5ef7 From: Daniel Drake <dsd@gentoo.org>
William Pitcock <nenolod@atheme-project.org>
parents: 3042
diff changeset
56 #include "visualization.h"
3251
f02623377013 goodbye widgets directory
Tomasz Mon <desowin@gmail.com>
parents: 3182
diff changeset
57 #include "skin.h"
2313
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
58
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
59 #include "playback.h"
3152
c9471f83944c Add playback_eof(). Plugins should use this instead of playback->eof = 1.
William Pitcock <nenolod@atheme-project.org>
parents: 3128
diff changeset
60 #include "playback_evlisteners.h"
2313
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
61
3677
30ef1287da19 Seperate the 10hz loop dependent code from the rest of the remaining polling loop.
William Pitcock <nenolod@atheme.org>
parents: 3514
diff changeset
62 static gint song_info_timeout_source = 0;
30ef1287da19 Seperate the 10hz loop dependent code from the rest of the remaining polling loop.
William Pitcock <nenolod@atheme.org>
parents: 3514
diff changeset
63 static gint update_vis_timeout_source = 0;
2313
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
64
3677
30ef1287da19 Seperate the 10hz loop dependent code from the rest of the remaining polling loop.
William Pitcock <nenolod@atheme.org>
parents: 3514
diff changeset
65 /* XXX: there has to be a better way than polling here! */
30ef1287da19 Seperate the 10hz loop dependent code from the rest of the remaining polling loop.
William Pitcock <nenolod@atheme.org>
parents: 3514
diff changeset
66 static gboolean
30ef1287da19 Seperate the 10hz loop dependent code from the rest of the remaining polling loop.
William Pitcock <nenolod@atheme.org>
parents: 3514
diff changeset
67 playback_update_vis_func(gpointer unused)
30ef1287da19 Seperate the 10hz loop dependent code from the rest of the remaining polling loop.
William Pitcock <nenolod@atheme.org>
parents: 3514
diff changeset
68 {
30ef1287da19 Seperate the 10hz loop dependent code from the rest of the remaining polling loop.
William Pitcock <nenolod@atheme.org>
parents: 3514
diff changeset
69 if (!playback_get_playing())
30ef1287da19 Seperate the 10hz loop dependent code from the rest of the remaining polling loop.
William Pitcock <nenolod@atheme.org>
parents: 3514
diff changeset
70 return FALSE;
30ef1287da19 Seperate the 10hz loop dependent code from the rest of the remaining polling loop.
William Pitcock <nenolod@atheme.org>
parents: 3514
diff changeset
71
30ef1287da19 Seperate the 10hz loop dependent code from the rest of the remaining polling loop.
William Pitcock <nenolod@atheme.org>
parents: 3514
diff changeset
72 input_update_vis(playback_get_time());
30ef1287da19 Seperate the 10hz loop dependent code from the rest of the remaining polling loop.
William Pitcock <nenolod@atheme.org>
parents: 3514
diff changeset
73
30ef1287da19 Seperate the 10hz loop dependent code from the rest of the remaining polling loop.
William Pitcock <nenolod@atheme.org>
parents: 3514
diff changeset
74 return TRUE;
30ef1287da19 Seperate the 10hz loop dependent code from the rest of the remaining polling loop.
William Pitcock <nenolod@atheme.org>
parents: 3514
diff changeset
75 }
3319
d4f6507cded3 added a system that allows input plugins to signal to the core (actually, they MUST do it) that they're ready for playback and it's safe to call their stop function; this fixes a nasty race condition that triggered on fast track-switching
Giacomo Lozito <james@develia.org>
parents: 3259
diff changeset
76
d4f6507cded3 added a system that allows input plugins to signal to the core (actually, they MUST do it) that they're ready for playback and it's safe to call their stop function; this fixes a nasty race condition that triggered on fast track-switching
Giacomo Lozito <james@develia.org>
parents: 3259
diff changeset
77 static gint
d4f6507cded3 added a system that allows input plugins to signal to the core (actually, they MUST do it) that they're ready for playback and it's safe to call their stop function; this fixes a nasty race condition that triggered on fast track-switching
Giacomo Lozito <james@develia.org>
parents: 3259
diff changeset
78 playback_set_pb_ready(InputPlayback *playback)
d4f6507cded3 added a system that allows input plugins to signal to the core (actually, they MUST do it) that they're ready for playback and it's safe to call their stop function; this fixes a nasty race condition that triggered on fast track-switching
Giacomo Lozito <james@develia.org>
parents: 3259
diff changeset
79 {
d4f6507cded3 added a system that allows input plugins to signal to the core (actually, they MUST do it) that they're ready for playback and it's safe to call their stop function; this fixes a nasty race condition that triggered on fast track-switching
Giacomo Lozito <james@develia.org>
parents: 3259
diff changeset
80 g_mutex_lock(playback->pb_ready_mutex);
d4f6507cded3 added a system that allows input plugins to signal to the core (actually, they MUST do it) that they're ready for playback and it's safe to call their stop function; this fixes a nasty race condition that triggered on fast track-switching
Giacomo Lozito <james@develia.org>
parents: 3259
diff changeset
81 playback->pb_ready_val = 1;
d4f6507cded3 added a system that allows input plugins to signal to the core (actually, they MUST do it) that they're ready for playback and it's safe to call their stop function; this fixes a nasty race condition that triggered on fast track-switching
Giacomo Lozito <james@develia.org>
parents: 3259
diff changeset
82 g_cond_signal(playback->pb_ready_cond);
d4f6507cded3 added a system that allows input plugins to signal to the core (actually, they MUST do it) that they're ready for playback and it's safe to call their stop function; this fixes a nasty race condition that triggered on fast track-switching
Giacomo Lozito <james@develia.org>
parents: 3259
diff changeset
83 g_mutex_unlock(playback->pb_ready_mutex);
d4f6507cded3 added a system that allows input plugins to signal to the core (actually, they MUST do it) that they're ready for playback and it's safe to call their stop function; this fixes a nasty race condition that triggered on fast track-switching
Giacomo Lozito <james@develia.org>
parents: 3259
diff changeset
84 return 0;
d4f6507cded3 added a system that allows input plugins to signal to the core (actually, they MUST do it) that they're ready for playback and it's safe to call their stop function; this fixes a nasty race condition that triggered on fast track-switching
Giacomo Lozito <james@develia.org>
parents: 3259
diff changeset
85 }
d4f6507cded3 added a system that allows input plugins to signal to the core (actually, they MUST do it) that they're ready for playback and it's safe to call their stop function; this fixes a nasty race condition that triggered on fast track-switching
Giacomo Lozito <james@develia.org>
parents: 3259
diff changeset
86
d4f6507cded3 added a system that allows input plugins to signal to the core (actually, they MUST do it) that they're ready for playback and it's safe to call their stop function; this fixes a nasty race condition that triggered on fast track-switching
Giacomo Lozito <james@develia.org>
parents: 3259
diff changeset
87 static gint
d4f6507cded3 added a system that allows input plugins to signal to the core (actually, they MUST do it) that they're ready for playback and it's safe to call their stop function; this fixes a nasty race condition that triggered on fast track-switching
Giacomo Lozito <james@develia.org>
parents: 3259
diff changeset
88 playback_wait_for_pb_ready(InputPlayback *playback)
d4f6507cded3 added a system that allows input plugins to signal to the core (actually, they MUST do it) that they're ready for playback and it's safe to call their stop function; this fixes a nasty race condition that triggered on fast track-switching
Giacomo Lozito <james@develia.org>
parents: 3259
diff changeset
89 {
d4f6507cded3 added a system that allows input plugins to signal to the core (actually, they MUST do it) that they're ready for playback and it's safe to call their stop function; this fixes a nasty race condition that triggered on fast track-switching
Giacomo Lozito <james@develia.org>
parents: 3259
diff changeset
90 g_mutex_lock(playback->pb_ready_mutex);
d4f6507cded3 added a system that allows input plugins to signal to the core (actually, they MUST do it) that they're ready for playback and it's safe to call their stop function; this fixes a nasty race condition that triggered on fast track-switching
Giacomo Lozito <james@develia.org>
parents: 3259
diff changeset
91 while (playback->pb_ready_val != 1)
d4f6507cded3 added a system that allows input plugins to signal to the core (actually, they MUST do it) that they're ready for playback and it's safe to call their stop function; this fixes a nasty race condition that triggered on fast track-switching
Giacomo Lozito <james@develia.org>
parents: 3259
diff changeset
92 g_cond_wait(playback->pb_ready_cond, playback->pb_ready_mutex);
d4f6507cded3 added a system that allows input plugins to signal to the core (actually, they MUST do it) that they're ready for playback and it's safe to call their stop function; this fixes a nasty race condition that triggered on fast track-switching
Giacomo Lozito <james@develia.org>
parents: 3259
diff changeset
93 g_mutex_unlock(playback->pb_ready_mutex);
d4f6507cded3 added a system that allows input plugins to signal to the core (actually, they MUST do it) that they're ready for playback and it's safe to call their stop function; this fixes a nasty race condition that triggered on fast track-switching
Giacomo Lozito <james@develia.org>
parents: 3259
diff changeset
94 return 0;
d4f6507cded3 added a system that allows input plugins to signal to the core (actually, they MUST do it) that they're ready for playback and it's safe to call their stop function; this fixes a nasty race condition that triggered on fast track-switching
Giacomo Lozito <james@develia.org>
parents: 3259
diff changeset
95 }
d4f6507cded3 added a system that allows input plugins to signal to the core (actually, they MUST do it) that they're ready for playback and it's safe to call their stop function; this fixes a nasty race condition that triggered on fast track-switching
Giacomo Lozito <james@develia.org>
parents: 3259
diff changeset
96
3696
418ac922ce87 Use a mutex/condvar/timed wait to allow tickless operation in the output loop.
William Pitcock <nenolod@atheme.org>
parents: 3677
diff changeset
97 static void
418ac922ce87 Use a mutex/condvar/timed wait to allow tickless operation in the output loop.
William Pitcock <nenolod@atheme.org>
parents: 3677
diff changeset
98 playback_set_pb_change(InputPlayback *playback)
418ac922ce87 Use a mutex/condvar/timed wait to allow tickless operation in the output loop.
William Pitcock <nenolod@atheme.org>
parents: 3677
diff changeset
99 {
418ac922ce87 Use a mutex/condvar/timed wait to allow tickless operation in the output loop.
William Pitcock <nenolod@atheme.org>
parents: 3677
diff changeset
100 g_mutex_lock(playback->pb_change_mutex);
418ac922ce87 Use a mutex/condvar/timed wait to allow tickless operation in the output loop.
William Pitcock <nenolod@atheme.org>
parents: 3677
diff changeset
101 g_cond_signal(playback->pb_change_cond);
418ac922ce87 Use a mutex/condvar/timed wait to allow tickless operation in the output loop.
William Pitcock <nenolod@atheme.org>
parents: 3677
diff changeset
102 g_mutex_unlock(playback->pb_change_mutex);
418ac922ce87 Use a mutex/condvar/timed wait to allow tickless operation in the output loop.
William Pitcock <nenolod@atheme.org>
parents: 3677
diff changeset
103 }
418ac922ce87 Use a mutex/condvar/timed wait to allow tickless operation in the output loop.
William Pitcock <nenolod@atheme.org>
parents: 3677
diff changeset
104
3705
38259e9394a2 add InputPlayback::set_params() and InputPlayback::set_title() which will
William Pitcock <nenolod@atheme.org>
parents: 3696
diff changeset
105 static void
38259e9394a2 add InputPlayback::set_params() and InputPlayback::set_title() which will
William Pitcock <nenolod@atheme.org>
parents: 3696
diff changeset
106 playback_set_pb_params(InputPlayback *playback, gchar *title,
38259e9394a2 add InputPlayback::set_params() and InputPlayback::set_title() which will
William Pitcock <nenolod@atheme.org>
parents: 3696
diff changeset
107 gint length, gint rate, gint freq, gint nch)
38259e9394a2 add InputPlayback::set_params() and InputPlayback::set_title() which will
William Pitcock <nenolod@atheme.org>
parents: 3696
diff changeset
108 {
38259e9394a2 add InputPlayback::set_params() and InputPlayback::set_title() which will
William Pitcock <nenolod@atheme.org>
parents: 3696
diff changeset
109 playback->title = g_strdup(title);
38259e9394a2 add InputPlayback::set_params() and InputPlayback::set_title() which will
William Pitcock <nenolod@atheme.org>
parents: 3696
diff changeset
110 playback->length = length;
38259e9394a2 add InputPlayback::set_params() and InputPlayback::set_title() which will
William Pitcock <nenolod@atheme.org>
parents: 3696
diff changeset
111 playback->rate = rate;
38259e9394a2 add InputPlayback::set_params() and InputPlayback::set_title() which will
William Pitcock <nenolod@atheme.org>
parents: 3696
diff changeset
112 playback->freq = freq;
38259e9394a2 add InputPlayback::set_params() and InputPlayback::set_title() which will
William Pitcock <nenolod@atheme.org>
parents: 3696
diff changeset
113 playback->nch = nch;
38259e9394a2 add InputPlayback::set_params() and InputPlayback::set_title() which will
William Pitcock <nenolod@atheme.org>
parents: 3696
diff changeset
114
38259e9394a2 add InputPlayback::set_params() and InputPlayback::set_title() which will
William Pitcock <nenolod@atheme.org>
parents: 3696
diff changeset
115 /* XXX: this can be removed/merged here someday */
38259e9394a2 add InputPlayback::set_params() and InputPlayback::set_title() which will
William Pitcock <nenolod@atheme.org>
parents: 3696
diff changeset
116 playback->plugin->set_info(title, length, rate, freq, nch);
38259e9394a2 add InputPlayback::set_params() and InputPlayback::set_title() which will
William Pitcock <nenolod@atheme.org>
parents: 3696
diff changeset
117 }
38259e9394a2 add InputPlayback::set_params() and InputPlayback::set_title() which will
William Pitcock <nenolod@atheme.org>
parents: 3696
diff changeset
118
38259e9394a2 add InputPlayback::set_params() and InputPlayback::set_title() which will
William Pitcock <nenolod@atheme.org>
parents: 3696
diff changeset
119 static void
38259e9394a2 add InputPlayback::set_params() and InputPlayback::set_title() which will
William Pitcock <nenolod@atheme.org>
parents: 3696
diff changeset
120 playback_set_pb_title(InputPlayback *playback, gchar *title)
38259e9394a2 add InputPlayback::set_params() and InputPlayback::set_title() which will
William Pitcock <nenolod@atheme.org>
parents: 3696
diff changeset
121 {
38259e9394a2 add InputPlayback::set_params() and InputPlayback::set_title() which will
William Pitcock <nenolod@atheme.org>
parents: 3696
diff changeset
122 playback->title = g_strdup(title);
38259e9394a2 add InputPlayback::set_params() and InputPlayback::set_title() which will
William Pitcock <nenolod@atheme.org>
parents: 3696
diff changeset
123
38259e9394a2 add InputPlayback::set_params() and InputPlayback::set_title() which will
William Pitcock <nenolod@atheme.org>
parents: 3696
diff changeset
124 playback->plugin->set_info_text(title);
38259e9394a2 add InputPlayback::set_params() and InputPlayback::set_title() which will
William Pitcock <nenolod@atheme.org>
parents: 3696
diff changeset
125 }
38259e9394a2 add InputPlayback::set_params() and InputPlayback::set_title() which will
William Pitcock <nenolod@atheme.org>
parents: 3696
diff changeset
126
3152
c9471f83944c Add playback_eof(). Plugins should use this instead of playback->eof = 1.
William Pitcock <nenolod@atheme-project.org>
parents: 3128
diff changeset
127 void
c9471f83944c Add playback_eof(). Plugins should use this instead of playback->eof = 1.
William Pitcock <nenolod@atheme-project.org>
parents: 3128
diff changeset
128 playback_eof(void)
c9471f83944c Add playback_eof(). Plugins should use this instead of playback->eof = 1.
William Pitcock <nenolod@atheme-project.org>
parents: 3128
diff changeset
129 {
c9471f83944c Add playback_eof(). Plugins should use this instead of playback->eof = 1.
William Pitcock <nenolod@atheme-project.org>
parents: 3128
diff changeset
130 event_queue("playback eof", playlist_get_active());
c9471f83944c Add playback_eof(). Plugins should use this instead of playback->eof = 1.
William Pitcock <nenolod@atheme-project.org>
parents: 3128
diff changeset
131 }
c9471f83944c Add playback_eof(). Plugins should use this instead of playback->eof = 1.
William Pitcock <nenolod@atheme-project.org>
parents: 3128
diff changeset
132
3158
92717dcb09f6 Send a message from the monitor thread to the main thread for audio errors, like we do with eof condition.
William Pitcock <nenolod@atheme-project.org>
parents: 3156
diff changeset
133 void
92717dcb09f6 Send a message from the monitor thread to the main thread for audio errors, like we do with eof condition.
William Pitcock <nenolod@atheme-project.org>
parents: 3156
diff changeset
134 playback_error(void)
92717dcb09f6 Send a message from the monitor thread to the main thread for audio errors, like we do with eof condition.
William Pitcock <nenolod@atheme-project.org>
parents: 3156
diff changeset
135 {
92717dcb09f6 Send a message from the monitor thread to the main thread for audio errors, like we do with eof condition.
William Pitcock <nenolod@atheme-project.org>
parents: 3156
diff changeset
136 event_queue("playback audio error", NULL);
92717dcb09f6 Send a message from the monitor thread to the main thread for audio errors, like we do with eof condition.
William Pitcock <nenolod@atheme-project.org>
parents: 3156
diff changeset
137 }
92717dcb09f6 Send a message from the monitor thread to the main thread for audio errors, like we do with eof condition.
William Pitcock <nenolod@atheme-project.org>
parents: 3156
diff changeset
138
2313
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
139 gint
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
140 playback_get_time(void)
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
141 {
2438
8750a62abed8 [svn] Provide flags in InputPlayback for common plugin flag needs, and provide a
iabervon
parents: 2436
diff changeset
142 InputPlayback *playback;
2420
b23f52fe132f [svn] Removed util.* from libaudacious
mf0102
parents: 2416
diff changeset
143 g_return_val_if_fail(playback_get_playing(), -1);
2438
8750a62abed8 [svn] Provide flags in InputPlayback for common plugin flag needs, and provide a
iabervon
parents: 2436
diff changeset
144 playback = get_current_input_playback();
2313
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
145
3180
384cb84c968f Gracefully handle playback being NULL even if playback_get_playing() is TRUE during playback init.
Christian Birchinger <joker@netswarm.net>
parents: 3179
diff changeset
146 if (!playback) /* playback can be NULL during init even if playing is TRUE */
384cb84c968f Gracefully handle playback being NULL even if playback_get_playing() is TRUE during playback init.
Christian Birchinger <joker@netswarm.net>
parents: 3179
diff changeset
147 return -1;
2438
8750a62abed8 [svn] Provide flags in InputPlayback for common plugin flag needs, and provide a
iabervon
parents: 2436
diff changeset
148 if (playback->plugin->get_time)
8750a62abed8 [svn] Provide flags in InputPlayback for common plugin flag needs, and provide a
iabervon
parents: 2436
diff changeset
149 return playback->plugin->get_time(playback);
8750a62abed8 [svn] Provide flags in InputPlayback for common plugin flag needs, and provide a
iabervon
parents: 2436
diff changeset
150 if (playback->error)
8750a62abed8 [svn] Provide flags in InputPlayback for common plugin flag needs, and provide a
iabervon
parents: 2436
diff changeset
151 return -2;
8750a62abed8 [svn] Provide flags in InputPlayback for common plugin flag needs, and provide a
iabervon
parents: 2436
diff changeset
152 if (!playback->playing ||
3346
71d8d93f1bad Implemented TrackChange, StatusChange signals, with a stub for CapsChange.
Ben Tucker <ben.tucker@gmail.com>
parents: 3326
diff changeset
153 (playback->eof && !playback->output->buffer_playing()))
2438
8750a62abed8 [svn] Provide flags in InputPlayback for common plugin flag needs, and provide a
iabervon
parents: 2436
diff changeset
154 return -1;
8750a62abed8 [svn] Provide flags in InputPlayback for common plugin flag needs, and provide a
iabervon
parents: 2436
diff changeset
155 return playback->output->output_time();
2313
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
156 }
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
157
3716
9489aae0d872 introduce playback_get_length()
Tomasz Mon <desowin@gmail.com>
parents: 3711
diff changeset
158 gint
9489aae0d872 introduce playback_get_length()
Tomasz Mon <desowin@gmail.com>
parents: 3711
diff changeset
159 playback_get_length(void)
9489aae0d872 introduce playback_get_length()
Tomasz Mon <desowin@gmail.com>
parents: 3711
diff changeset
160 {
9489aae0d872 introduce playback_get_length()
Tomasz Mon <desowin@gmail.com>
parents: 3711
diff changeset
161 InputPlayback *playback;
9489aae0d872 introduce playback_get_length()
Tomasz Mon <desowin@gmail.com>
parents: 3711
diff changeset
162 g_return_val_if_fail(playback_get_playing(), -1);
9489aae0d872 introduce playback_get_length()
Tomasz Mon <desowin@gmail.com>
parents: 3711
diff changeset
163 playback = get_current_input_playback();
9489aae0d872 introduce playback_get_length()
Tomasz Mon <desowin@gmail.com>
parents: 3711
diff changeset
164
3717
8ec5bb8aac58 playback_get_length(): Support InputPlayback::set_params().
William Pitcock <nenolod@atheme.org>
parents: 3716
diff changeset
165 if (playback->length)
8ec5bb8aac58 playback_get_length(): Support InputPlayback::set_params().
William Pitcock <nenolod@atheme.org>
parents: 3716
diff changeset
166 return playback->length;
8ec5bb8aac58 playback_get_length(): Support InputPlayback::set_params().
William Pitcock <nenolod@atheme.org>
parents: 3716
diff changeset
167
3716
9489aae0d872 introduce playback_get_length()
Tomasz Mon <desowin@gmail.com>
parents: 3711
diff changeset
168 if (playback && playback->plugin->get_song_tuple) {
9489aae0d872 introduce playback_get_length()
Tomasz Mon <desowin@gmail.com>
parents: 3711
diff changeset
169 Tuple *tuple = playback->plugin->get_song_tuple(playback->filename);
9489aae0d872 introduce playback_get_length()
Tomasz Mon <desowin@gmail.com>
parents: 3711
diff changeset
170 if (tuple_get_value_type(tuple, FIELD_LENGTH, NULL) == TUPLE_INT)
3926
05e6d9db003c playback_get_length() had always returned 1 whenever it uses tuple.
Yoshiki Yazawa <yaz@cc.rim.or.jp>
parents: 3722
diff changeset
171 return tuple_get_int(tuple, FIELD_LENGTH, NULL);
3716
9489aae0d872 introduce playback_get_length()
Tomasz Mon <desowin@gmail.com>
parents: 3711
diff changeset
172 }
3717
8ec5bb8aac58 playback_get_length(): Support InputPlayback::set_params().
William Pitcock <nenolod@atheme.org>
parents: 3716
diff changeset
173
3716
9489aae0d872 introduce playback_get_length()
Tomasz Mon <desowin@gmail.com>
parents: 3711
diff changeset
174 return -1;
9489aae0d872 introduce playback_get_length()
Tomasz Mon <desowin@gmail.com>
parents: 3711
diff changeset
175 }
9489aae0d872 introduce playback_get_length()
Tomasz Mon <desowin@gmail.com>
parents: 3711
diff changeset
176
2313
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
177 void
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
178 playback_initiate(void)
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
179 {
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
180 PlaylistEntry *entry = NULL;
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
181 Playlist *playlist = playlist_get_active();
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
182
2420
b23f52fe132f [svn] Removed util.* from libaudacious
mf0102
parents: 2416
diff changeset
183 g_return_if_fail(playlist_get_length(playlist) != 0);
2313
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
184
3152
c9471f83944c Add playback_eof(). Plugins should use this instead of playback->eof = 1.
William Pitcock <nenolod@atheme-project.org>
parents: 3128
diff changeset
185 /* initialize playback event listeners if not done already */
c9471f83944c Add playback_eof(). Plugins should use this instead of playback->eof = 1.
William Pitcock <nenolod@atheme-project.org>
parents: 3128
diff changeset
186 playback_evlistener_init();
c9471f83944c Add playback_eof(). Plugins should use this instead of playback->eof = 1.
William Pitcock <nenolod@atheme-project.org>
parents: 3128
diff changeset
187
2313
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
188 if (playback_get_playing())
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
189 playback_stop();
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
190
3020
6065d70cb790 replace old Vis with UiVis
Tomasz Mon <desowin@gmail.com>
parents: 3001
diff changeset
191 ui_vis_clear_data(mainwin_vis);
3054
34f37c59e87b replace svis with UiSVis
Tomasz Mon <desowin@gmail.com>
parents: 3042
diff changeset
192 ui_svis_clear_data(mainwin_svis);
2313
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
193 mainwin_disable_seekbar();
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
194
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
195 entry = playlist_get_entry_to_play(playlist);
2420
b23f52fe132f [svn] Removed util.* from libaudacious
mf0102
parents: 2416
diff changeset
196 g_return_if_fail(entry != NULL);
3348
2a081105513c -Set the track "title" in metadata instead of "name"
Ben Tucker <ben.tucker@gmail.com>
parents: 3346
diff changeset
197 #ifdef USE_DBUS
2a081105513c -Set the track "title" in metadata instead of "name"
Ben Tucker <ben.tucker@gmail.com>
parents: 3346
diff changeset
198 mpris_emit_track_change(mpris);
2a081105513c -Set the track "title" in metadata instead of "name"
Ben Tucker <ben.tucker@gmail.com>
parents: 3346
diff changeset
199 #endif
2592
6f7be3702c5f [svn] - revert the penalty-based "find something to play" logic of 1.2. it's
nenolod
parents: 2454
diff changeset
200 playback_play_file(entry);
2313
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
201
3179
d2eae001e562 I don't think that's needed. If you revert it, use playback_get_playing() instead.
Christian Birchinger <joker@netswarm.net>
parents: 3170
diff changeset
202 // if (playback_get_time() != -1) {
2313
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
203 equalizerwin_load_auto_preset(entry->filename);
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
204 input_set_eq(cfg.equalizer_active, cfg.equalizer_preamp,
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
205 cfg.equalizer_bands);
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
206 output_set_eq(cfg.equalizer_active, cfg.equalizer_preamp,
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
207 cfg.equalizer_bands);
3179
d2eae001e562 I don't think that's needed. If you revert it, use playback_get_playing() instead.
Christian Birchinger <joker@netswarm.net>
parents: 3170
diff changeset
208 // }
2313
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
209
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
210 playlist_check_pos_current(playlist);
3018
6a9fdd5aee3a Move timer updates out of the 100hz giant loop.
Daniel Drake <dsd@gentoo.org>
parents: 3001
diff changeset
211 mainwin_update_song_info();
6a9fdd5aee3a Move timer updates out of the 100hz giant loop.
Daniel Drake <dsd@gentoo.org>
parents: 3001
diff changeset
212
6a9fdd5aee3a Move timer updates out of the 100hz giant loop.
Daniel Drake <dsd@gentoo.org>
parents: 3001
diff changeset
213 /* FIXME: use g_timeout_add_seconds when glib-2.14 is required */
6a9fdd5aee3a Move timer updates out of the 100hz giant loop.
Daniel Drake <dsd@gentoo.org>
parents: 3001
diff changeset
214 song_info_timeout_source = g_timeout_add(1000,
6a9fdd5aee3a Move timer updates out of the 100hz giant loop.
Daniel Drake <dsd@gentoo.org>
parents: 3001
diff changeset
215 (GSourceFunc) mainwin_update_song_info, NULL);
2407
1dc1d36d0347 [svn] - add hooks: playback begin, playback end, playlist reached end
nenolod
parents: 2366
diff changeset
216
3677
30ef1287da19 Seperate the 10hz loop dependent code from the rest of the remaining polling loop.
William Pitcock <nenolod@atheme.org>
parents: 3514
diff changeset
217 update_vis_timeout_source = g_timeout_add(10,
30ef1287da19 Seperate the 10hz loop dependent code from the rest of the remaining polling loop.
William Pitcock <nenolod@atheme.org>
parents: 3514
diff changeset
218 (GSourceFunc) playback_update_vis_func, NULL);
30ef1287da19 Seperate the 10hz loop dependent code from the rest of the remaining polling loop.
William Pitcock <nenolod@atheme.org>
parents: 3514
diff changeset
219
3000
bbca1e0e054a don't call gtk_widget_show/hide from func that's called every now and then
Tomasz Mon <desowin@gmail.com>
parents: 2976
diff changeset
220 if (cfg.player_shaded) {
bbca1e0e054a don't call gtk_widget_show/hide from func that's called every now and then
Tomasz Mon <desowin@gmail.com>
parents: 2976
diff changeset
221 gtk_widget_show(mainwin_stime_min);
bbca1e0e054a don't call gtk_widget_show/hide from func that's called every now and then
Tomasz Mon <desowin@gmail.com>
parents: 2976
diff changeset
222 gtk_widget_show(mainwin_stime_sec);
3042
01ae56759d15 mainwin_sposition is now UiSkinnedHorizontalSlider
Tomasz Mon <desowin@gmail.com>
parents: 3041
diff changeset
223 gtk_widget_show(mainwin_sposition);
3001
6d4b7b739232 fully implement UiSkinnedNumber, number.c no longer needed
Tomasz Mon <desowin@gmail.com>
parents: 3000
diff changeset
224 } else {
6d4b7b739232 fully implement UiSkinnedNumber, number.c no longer needed
Tomasz Mon <desowin@gmail.com>
parents: 3000
diff changeset
225 gtk_widget_show(mainwin_minus_num);
6d4b7b739232 fully implement UiSkinnedNumber, number.c no longer needed
Tomasz Mon <desowin@gmail.com>
parents: 3000
diff changeset
226 gtk_widget_show(mainwin_10min_num);
6d4b7b739232 fully implement UiSkinnedNumber, number.c no longer needed
Tomasz Mon <desowin@gmail.com>
parents: 3000
diff changeset
227 gtk_widget_show(mainwin_min_num);
6d4b7b739232 fully implement UiSkinnedNumber, number.c no longer needed
Tomasz Mon <desowin@gmail.com>
parents: 3000
diff changeset
228 gtk_widget_show(mainwin_10sec_num);
6d4b7b739232 fully implement UiSkinnedNumber, number.c no longer needed
Tomasz Mon <desowin@gmail.com>
parents: 3000
diff changeset
229 gtk_widget_show(mainwin_sec_num);
3040
067e0186623a import UiSkinnedHorizontalSlider
Tomasz Mon <desowin@gmail.com>
parents: 3021
diff changeset
230 gtk_widget_show(mainwin_position);
3000
bbca1e0e054a don't call gtk_widget_show/hide from func that's called every now and then
Tomasz Mon <desowin@gmail.com>
parents: 2976
diff changeset
231 }
bbca1e0e054a don't call gtk_widget_show/hide from func that's called every now and then
Tomasz Mon <desowin@gmail.com>
parents: 2976
diff changeset
232
3082
91df04ec5ef7 From: Daniel Drake <dsd@gentoo.org>
William Pitcock <nenolod@atheme-project.org>
parents: 3042
diff changeset
233 vis_playback_start();
3092
1ddeb9f068ab Backed out changeset b87335249c69d45d253b1b960a9ec7f60b68e5a5
William Pitcock <nenolod@atheme-project.org>
parents: 3089
diff changeset
234
1ddeb9f068ab Backed out changeset b87335249c69d45d253b1b960a9ec7f60b68e5a5
William Pitcock <nenolod@atheme-project.org>
parents: 3089
diff changeset
235 hook_call("playback begin", entry);
2313
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
236 }
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
237
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
238 void
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
239 playback_pause(void)
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
240 {
3170
603577228518 get_current_input_playback() cleanups.
William Pitcock <nenolod@atheme-project.org>
parents: 3168
diff changeset
241 InputPlayback *playback;
603577228518 get_current_input_playback() cleanups.
William Pitcock <nenolod@atheme-project.org>
parents: 3168
diff changeset
242
2313
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
243 if (!playback_get_playing())
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
244 return;
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
245
3170
603577228518 get_current_input_playback() cleanups.
William Pitcock <nenolod@atheme-project.org>
parents: 3168
diff changeset
246 if ((playback = get_current_input_playback()) == NULL)
2313
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
247 return;
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
248
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
249 ip_data.paused = !ip_data.paused;
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
250
3170
603577228518 get_current_input_playback() cleanups.
William Pitcock <nenolod@atheme-project.org>
parents: 3168
diff changeset
251 if (playback->plugin->pause)
603577228518 get_current_input_playback() cleanups.
William Pitcock <nenolod@atheme-project.org>
parents: 3168
diff changeset
252 playback->plugin->pause(playback, ip_data.paused);
2313
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
253
3039
c981df78db8c added hooks: playback pause and playback unpause
Giacomo Lozito <james@develia.org>
parents: 3021
diff changeset
254 if (ip_data.paused)
c981df78db8c added hooks: playback pause and playback unpause
Giacomo Lozito <james@develia.org>
parents: 3021
diff changeset
255 hook_call("playback pause", NULL);
c981df78db8c added hooks: playback pause and playback unpause
Giacomo Lozito <james@develia.org>
parents: 3021
diff changeset
256 else
c981df78db8c added hooks: playback pause and playback unpause
Giacomo Lozito <james@develia.org>
parents: 3021
diff changeset
257 hook_call("playback unpause", NULL);
c981df78db8c added hooks: playback pause and playback unpause
Giacomo Lozito <james@develia.org>
parents: 3021
diff changeset
258
2313
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
259 g_return_if_fail(mainwin_playstatus != NULL);
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
260
3346
71d8d93f1bad Implemented TrackChange, StatusChange signals, with a stub for CapsChange.
Ben Tucker <ben.tucker@gmail.com>
parents: 3326
diff changeset
261 if (ip_data.paused) {
3072
84de3244aeaa replace Playstatus with UiSkinnedPlaystatus
Tomasz Mon <desowin@gmail.com>
parents: 3054
diff changeset
262 ui_skinned_playstatus_set_status(mainwin_playstatus, STATUS_PAUSE);
3346
71d8d93f1bad Implemented TrackChange, StatusChange signals, with a stub for CapsChange.
Ben Tucker <ben.tucker@gmail.com>
parents: 3326
diff changeset
263 #ifdef USE_DBUS
71d8d93f1bad Implemented TrackChange, StatusChange signals, with a stub for CapsChange.
Ben Tucker <ben.tucker@gmail.com>
parents: 3326
diff changeset
264 mpris_emit_status_change(mpris, MPRIS_STATUS_PAUSE);
71d8d93f1bad Implemented TrackChange, StatusChange signals, with a stub for CapsChange.
Ben Tucker <ben.tucker@gmail.com>
parents: 3326
diff changeset
265 #endif
71d8d93f1bad Implemented TrackChange, StatusChange signals, with a stub for CapsChange.
Ben Tucker <ben.tucker@gmail.com>
parents: 3326
diff changeset
266 } else {
3072
84de3244aeaa replace Playstatus with UiSkinnedPlaystatus
Tomasz Mon <desowin@gmail.com>
parents: 3054
diff changeset
267 ui_skinned_playstatus_set_status(mainwin_playstatus, STATUS_PLAY);
3346
71d8d93f1bad Implemented TrackChange, StatusChange signals, with a stub for CapsChange.
Ben Tucker <ben.tucker@gmail.com>
parents: 3326
diff changeset
268 #ifdef USE_DBUS
71d8d93f1bad Implemented TrackChange, StatusChange signals, with a stub for CapsChange.
Ben Tucker <ben.tucker@gmail.com>
parents: 3326
diff changeset
269 mpris_emit_status_change(mpris, MPRIS_STATUS_PLAY);
71d8d93f1bad Implemented TrackChange, StatusChange signals, with a stub for CapsChange.
Ben Tucker <ben.tucker@gmail.com>
parents: 3326
diff changeset
270 #endif
71d8d93f1bad Implemented TrackChange, StatusChange signals, with a stub for CapsChange.
Ben Tucker <ben.tucker@gmail.com>
parents: 3326
diff changeset
271 }
2313
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
272 }
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
273
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
274 void
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
275 playback_stop(void)
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
276 {
3170
603577228518 get_current_input_playback() cleanups.
William Pitcock <nenolod@atheme-project.org>
parents: 3168
diff changeset
277 InputPlayback *playback;
3252
eddeaebaf1c7 - fixed ugly race condition in new threading model; mutexes are also probably required to handle shared access to ip_data
Giacomo Lozito <james@develia.org>
parents: 3251
diff changeset
278
3253
6374d66ee623 proper race condition fix
William Pitcock <nenolod@atheme-project.org>
parents: 3252
diff changeset
279 if ((playback = get_current_input_playback()) == NULL)
6374d66ee623 proper race condition fix
William Pitcock <nenolod@atheme-project.org>
parents: 3252
diff changeset
280 return;
2313
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
281
3252
eddeaebaf1c7 - fixed ugly race condition in new threading model; mutexes are also probably required to handle shared access to ip_data
Giacomo Lozito <james@develia.org>
parents: 3251
diff changeset
282 if (ip_data.playing)
3170
603577228518 get_current_input_playback() cleanups.
William Pitcock <nenolod@atheme-project.org>
parents: 3168
diff changeset
283 {
3319
d4f6507cded3 added a system that allows input plugins to signal to the core (actually, they MUST do it) that they're ready for playback and it's safe to call their stop function; this fixes a nasty race condition that triggered on fast track-switching
Giacomo Lozito <james@develia.org>
parents: 3259
diff changeset
284 /* wait for plugin to signal it has reached
d4f6507cded3 added a system that allows input plugins to signal to the core (actually, they MUST do it) that they're ready for playback and it's safe to call their stop function; this fixes a nasty race condition that triggered on fast track-switching
Giacomo Lozito <james@develia.org>
parents: 3259
diff changeset
285 the 'playback safe state' before stopping */
d4f6507cded3 added a system that allows input plugins to signal to the core (actually, they MUST do it) that they're ready for playback and it's safe to call their stop function; this fixes a nasty race condition that triggered on fast track-switching
Giacomo Lozito <james@develia.org>
parents: 3259
diff changeset
286 playback_wait_for_pb_ready(playback);
d4f6507cded3 added a system that allows input plugins to signal to the core (actually, they MUST do it) that they're ready for playback and it's safe to call their stop function; this fixes a nasty race condition that triggered on fast track-switching
Giacomo Lozito <james@develia.org>
parents: 3259
diff changeset
287
3170
603577228518 get_current_input_playback() cleanups.
William Pitcock <nenolod@atheme-project.org>
parents: 3168
diff changeset
288 if (playback_get_paused() == TRUE)
603577228518 get_current_input_playback() cleanups.
William Pitcock <nenolod@atheme-project.org>
parents: 3168
diff changeset
289 {
3258
d899694663a7 in playback_stop, do not assume that get_written_time will be always > 0, some plugins do not write to output at all
Giacomo Lozito <james@develia.org>
parents: 3253
diff changeset
290 if (get_written_time() > 0)
d899694663a7 in playback_stop, do not assume that get_written_time will be always > 0, some plugins do not write to output at all
Giacomo Lozito <james@develia.org>
parents: 3253
diff changeset
291 output_flush(get_written_time()); /* to avoid noise */
2313
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
292 playback_pause();
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
293 }
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
294
3319
d4f6507cded3 added a system that allows input plugins to signal to the core (actually, they MUST do it) that they're ready for playback and it's safe to call their stop function; this fixes a nasty race condition that triggered on fast track-switching
Giacomo Lozito <james@develia.org>
parents: 3259
diff changeset
295 ip_data.playing = FALSE;
2313
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
296
3696
418ac922ce87 Use a mutex/condvar/timed wait to allow tickless operation in the output loop.
William Pitcock <nenolod@atheme.org>
parents: 3677
diff changeset
297 /* TODO: i'm unsure if this will work. we might have to
418ac922ce87 Use a mutex/condvar/timed wait to allow tickless operation in the output loop.
William Pitcock <nenolod@atheme.org>
parents: 3677
diff changeset
298 signal the change in stop() (e.g. in the plugins
418ac922ce87 Use a mutex/condvar/timed wait to allow tickless operation in the output loop.
William Pitcock <nenolod@atheme.org>
parents: 3677
diff changeset
299 directly.) --nenolod */
418ac922ce87 Use a mutex/condvar/timed wait to allow tickless operation in the output loop.
William Pitcock <nenolod@atheme.org>
parents: 3677
diff changeset
300 playback->set_pb_change(playback);
418ac922ce87 Use a mutex/condvar/timed wait to allow tickless operation in the output loop.
William Pitcock <nenolod@atheme.org>
parents: 3677
diff changeset
301
3170
603577228518 get_current_input_playback() cleanups.
William Pitcock <nenolod@atheme-project.org>
parents: 3168
diff changeset
302 if (playback->plugin->stop)
603577228518 get_current_input_playback() cleanups.
William Pitcock <nenolod@atheme-project.org>
parents: 3168
diff changeset
303 playback->plugin->stop(playback);
2313
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
304
3181
1596dcb77acd Track playback monitor thread in InputPlayback.thread.
William Pitcock <nenolod@atheme-project.org>
parents: 3170
diff changeset
305 if (playback->thread != NULL)
3326
6dbb73b8e28c thread sanity checks
William Pitcock <nenolod@atheme-project.org>
parents: 3325
diff changeset
306 {
3181
1596dcb77acd Track playback monitor thread in InputPlayback.thread.
William Pitcock <nenolod@atheme-project.org>
parents: 3170
diff changeset
307 g_thread_join(playback->thread);
3326
6dbb73b8e28c thread sanity checks
William Pitcock <nenolod@atheme-project.org>
parents: 3325
diff changeset
308 playback->thread = NULL;
6dbb73b8e28c thread sanity checks
William Pitcock <nenolod@atheme-project.org>
parents: 3325
diff changeset
309 }
3181
1596dcb77acd Track playback monitor thread in InputPlayback.thread.
William Pitcock <nenolod@atheme-project.org>
parents: 3170
diff changeset
310
2313
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
311 free_vis_data();
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
312 ip_data.paused = FALSE;
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
313
4109
2d6fd3f630f0 Adds playback_free() function to fix memory leak in playback_stop(). (Bugzilla #42)
Jussi Judin <jjudin+audacious@iki.fi>
parents: 3926
diff changeset
314 playback_free(playback);
3319
d4f6507cded3 added a system that allows input plugins to signal to the core (actually, they MUST do it) that they're ready for playback and it's safe to call their stop function; this fixes a nasty race condition that triggered on fast track-switching
Giacomo Lozito <james@develia.org>
parents: 3259
diff changeset
315 set_current_input_playback(NULL);
3346
71d8d93f1bad Implemented TrackChange, StatusChange signals, with a stub for CapsChange.
Ben Tucker <ben.tucker@gmail.com>
parents: 3326
diff changeset
316 #ifdef USE_DBUS
71d8d93f1bad Implemented TrackChange, StatusChange signals, with a stub for CapsChange.
Ben Tucker <ben.tucker@gmail.com>
parents: 3326
diff changeset
317 mpris_emit_status_change(mpris, MPRIS_STATUS_STOP);
71d8d93f1bad Implemented TrackChange, StatusChange signals, with a stub for CapsChange.
Ben Tucker <ben.tucker@gmail.com>
parents: 3326
diff changeset
318 #endif
2313
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
319 }
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
320
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
321 ip_data.buffering = FALSE;
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
322 ip_data.playing = FALSE;
3018
6a9fdd5aee3a Move timer updates out of the 100hz giant loop.
Daniel Drake <dsd@gentoo.org>
parents: 3001
diff changeset
323
6a9fdd5aee3a Move timer updates out of the 100hz giant loop.
Daniel Drake <dsd@gentoo.org>
parents: 3001
diff changeset
324 if (song_info_timeout_source)
6a9fdd5aee3a Move timer updates out of the 100hz giant loop.
Daniel Drake <dsd@gentoo.org>
parents: 3001
diff changeset
325 g_source_remove(song_info_timeout_source);
6a9fdd5aee3a Move timer updates out of the 100hz giant loop.
Daniel Drake <dsd@gentoo.org>
parents: 3001
diff changeset
326
3082
91df04ec5ef7 From: Daniel Drake <dsd@gentoo.org>
William Pitcock <nenolod@atheme-project.org>
parents: 3042
diff changeset
327 vis_playback_stop();
91df04ec5ef7 From: Daniel Drake <dsd@gentoo.org>
William Pitcock <nenolod@atheme-project.org>
parents: 3042
diff changeset
328
2313
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
329 g_return_if_fail(mainwin_playstatus != NULL);
3072
84de3244aeaa replace Playstatus with UiSkinnedPlaystatus
Tomasz Mon <desowin@gmail.com>
parents: 3054
diff changeset
330 ui_skinned_playstatus_set_buffering(mainwin_playstatus, FALSE);
2313
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
331 }
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
332
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
333 static void
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
334 run_no_output_plugin_dialog(void)
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
335 {
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
336 const gchar *markup =
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
337 N_("<b><big>No output plugin selected.</big></b>\n"
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
338 "You have not selected an output plugin.");
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
339
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
340 GtkWidget *dialog =
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
341 gtk_message_dialog_new_with_markup(GTK_WINDOW(mainwin),
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
342 GTK_DIALOG_DESTROY_WITH_PARENT,
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
343 GTK_MESSAGE_ERROR,
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
344 GTK_BUTTONS_OK,
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
345 _(markup));
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
346 gtk_dialog_run(GTK_DIALOG(dialog));
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
347 gtk_widget_destroy(dialog);
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
348 }
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
349
3153
e0a2ea93f98f New threading model for playback. Playback code should now *block*.
William Pitcock <nenolod@atheme-project.org>
parents: 3152
diff changeset
350 static gpointer
3154
732cfad87116 Made a mistake here. Fixing!
William Pitcock <nenolod@atheme-project.org>
parents: 3153
diff changeset
351 playback_monitor_thread(gpointer data)
3153
e0a2ea93f98f New threading model for playback. Playback code should now *block*.
William Pitcock <nenolod@atheme-project.org>
parents: 3152
diff changeset
352 {
3253
6374d66ee623 proper race condition fix
William Pitcock <nenolod@atheme-project.org>
parents: 3252
diff changeset
353 InputPlayback *playback = (InputPlayback *) data;
3153
e0a2ea93f98f New threading model for playback. Playback code should now *block*.
William Pitcock <nenolod@atheme-project.org>
parents: 3152
diff changeset
354
3253
6374d66ee623 proper race condition fix
William Pitcock <nenolod@atheme-project.org>
parents: 3252
diff changeset
355 playback->plugin->play_file(playback);
3153
e0a2ea93f98f New threading model for playback. Playback code should now *block*.
William Pitcock <nenolod@atheme-project.org>
parents: 3152
diff changeset
356
3319
d4f6507cded3 added a system that allows input plugins to signal to the core (actually, they MUST do it) that they're ready for playback and it's safe to call their stop function; this fixes a nasty race condition that triggered on fast track-switching
Giacomo Lozito <james@develia.org>
parents: 3259
diff changeset
357 /* if play_file has not reached the 'safe state' before returning (an error
d4f6507cded3 added a system that allows input plugins to signal to the core (actually, they MUST do it) that they're ready for playback and it's safe to call their stop function; this fixes a nasty race condition that triggered on fast track-switching
Giacomo Lozito <james@develia.org>
parents: 3259
diff changeset
358 occurred), set the playback ready value to 1 now, to allow for proper stop */
d4f6507cded3 added a system that allows input plugins to signal to the core (actually, they MUST do it) that they're ready for playback and it's safe to call their stop function; this fixes a nasty race condition that triggered on fast track-switching
Giacomo Lozito <james@develia.org>
parents: 3259
diff changeset
359 playback_set_pb_ready(playback);
d4f6507cded3 added a system that allows input plugins to signal to the core (actually, they MUST do it) that they're ready for playback and it's safe to call their stop function; this fixes a nasty race condition that triggered on fast track-switching
Giacomo Lozito <james@develia.org>
parents: 3259
diff changeset
360
3158
92717dcb09f6 Send a message from the monitor thread to the main thread for audio errors, like we do with eof condition.
William Pitcock <nenolod@atheme-project.org>
parents: 3156
diff changeset
361 if (!playback->error && ip_data.playing)
3156
4a0e216bc825 playback monitor thread: If forced stop occurs, do not queue "playback eof" signal.
William Pitcock <nenolod@atheme-project.org>
parents: 3155
diff changeset
362 playback_eof();
3158
92717dcb09f6 Send a message from the monitor thread to the main thread for audio errors, like we do with eof condition.
William Pitcock <nenolod@atheme-project.org>
parents: 3156
diff changeset
363 else if (playback->error)
92717dcb09f6 Send a message from the monitor thread to the main thread for audio errors, like we do with eof condition.
William Pitcock <nenolod@atheme-project.org>
parents: 3156
diff changeset
364 playback_error();
3153
e0a2ea93f98f New threading model for playback. Playback code should now *block*.
William Pitcock <nenolod@atheme-project.org>
parents: 3152
diff changeset
365
3181
1596dcb77acd Track playback monitor thread in InputPlayback.thread.
William Pitcock <nenolod@atheme-project.org>
parents: 3170
diff changeset
366 playback->thread = NULL;
1596dcb77acd Track playback monitor thread in InputPlayback.thread.
William Pitcock <nenolod@atheme-project.org>
parents: 3170
diff changeset
367 g_thread_exit(NULL);
3153
e0a2ea93f98f New threading model for playback. Playback code should now *block*.
William Pitcock <nenolod@atheme-project.org>
parents: 3152
diff changeset
368 return NULL;
e0a2ea93f98f New threading model for playback. Playback code should now *block*.
William Pitcock <nenolod@atheme-project.org>
parents: 3152
diff changeset
369 }
e0a2ea93f98f New threading model for playback. Playback code should now *block*.
William Pitcock <nenolod@atheme-project.org>
parents: 3152
diff changeset
370
3710
602002306969 generic playback control API
William Pitcock <nenolod@atheme.org>
parents: 3709
diff changeset
371 InputPlayback *
602002306969 generic playback control API
William Pitcock <nenolod@atheme.org>
parents: 3709
diff changeset
372 playback_new(void)
602002306969 generic playback control API
William Pitcock <nenolod@atheme.org>
parents: 3709
diff changeset
373 {
3711
21d58602af29 build fix.
Yoshiki Yazawa <yaz@cc.rim.or.jp>
parents: 3710
diff changeset
374 InputPlayback *playback = (InputPlayback *) g_slice_new0(InputPlayback);
3710
602002306969 generic playback control API
William Pitcock <nenolod@atheme.org>
parents: 3709
diff changeset
375
602002306969 generic playback control API
William Pitcock <nenolod@atheme.org>
parents: 3709
diff changeset
376 playback->pb_ready_mutex = g_mutex_new();
602002306969 generic playback control API
William Pitcock <nenolod@atheme.org>
parents: 3709
diff changeset
377 playback->pb_ready_cond = g_cond_new();
602002306969 generic playback control API
William Pitcock <nenolod@atheme.org>
parents: 3709
diff changeset
378 playback->pb_ready_val = 0;
602002306969 generic playback control API
William Pitcock <nenolod@atheme.org>
parents: 3709
diff changeset
379
602002306969 generic playback control API
William Pitcock <nenolod@atheme.org>
parents: 3709
diff changeset
380 playback->pb_change_mutex = g_mutex_new();
602002306969 generic playback control API
William Pitcock <nenolod@atheme.org>
parents: 3709
diff changeset
381 playback->pb_change_cond = g_cond_new();
602002306969 generic playback control API
William Pitcock <nenolod@atheme.org>
parents: 3709
diff changeset
382
602002306969 generic playback control API
William Pitcock <nenolod@atheme.org>
parents: 3709
diff changeset
383 /* init vtable functors */
602002306969 generic playback control API
William Pitcock <nenolod@atheme.org>
parents: 3709
diff changeset
384 playback->set_pb_ready = playback_set_pb_ready;
602002306969 generic playback control API
William Pitcock <nenolod@atheme.org>
parents: 3709
diff changeset
385 playback->set_pb_change = playback_set_pb_change;
602002306969 generic playback control API
William Pitcock <nenolod@atheme.org>
parents: 3709
diff changeset
386 playback->set_params = playback_set_pb_params;
602002306969 generic playback control API
William Pitcock <nenolod@atheme.org>
parents: 3709
diff changeset
387 playback->set_title = playback_set_pb_title;
602002306969 generic playback control API
William Pitcock <nenolod@atheme.org>
parents: 3709
diff changeset
388 playback->pass_audio = output_pass_audio;
602002306969 generic playback control API
William Pitcock <nenolod@atheme.org>
parents: 3709
diff changeset
389
602002306969 generic playback control API
William Pitcock <nenolod@atheme.org>
parents: 3709
diff changeset
390 return playback;
602002306969 generic playback control API
William Pitcock <nenolod@atheme.org>
parents: 3709
diff changeset
391 }
602002306969 generic playback control API
William Pitcock <nenolod@atheme.org>
parents: 3709
diff changeset
392
4109
2d6fd3f630f0 Adds playback_free() function to fix memory leak in playback_stop(). (Bugzilla #42)
Jussi Judin <jjudin+audacious@iki.fi>
parents: 3926
diff changeset
393 /**
2d6fd3f630f0 Adds playback_free() function to fix memory leak in playback_stop(). (Bugzilla #42)
Jussi Judin <jjudin+audacious@iki.fi>
parents: 3926
diff changeset
394 * Destroys InputPlayback.
2d6fd3f630f0 Adds playback_free() function to fix memory leak in playback_stop(). (Bugzilla #42)
Jussi Judin <jjudin+audacious@iki.fi>
parents: 3926
diff changeset
395 *
2d6fd3f630f0 Adds playback_free() function to fix memory leak in playback_stop(). (Bugzilla #42)
Jussi Judin <jjudin+audacious@iki.fi>
parents: 3926
diff changeset
396 * Playback comes from playback_new() function but there can be also
2d6fd3f630f0 Adds playback_free() function to fix memory leak in playback_stop(). (Bugzilla #42)
Jussi Judin <jjudin+audacious@iki.fi>
parents: 3926
diff changeset
397 * other sources for allocated playback data (like filename and title)
2d6fd3f630f0 Adds playback_free() function to fix memory leak in playback_stop(). (Bugzilla #42)
Jussi Judin <jjudin+audacious@iki.fi>
parents: 3926
diff changeset
398 * and this tries to deallocate all that data.
2d6fd3f630f0 Adds playback_free() function to fix memory leak in playback_stop(). (Bugzilla #42)
Jussi Judin <jjudin+audacious@iki.fi>
parents: 3926
diff changeset
399 */
2d6fd3f630f0 Adds playback_free() function to fix memory leak in playback_stop(). (Bugzilla #42)
Jussi Judin <jjudin+audacious@iki.fi>
parents: 3926
diff changeset
400 void playback_free(InputPlayback *playback)
2d6fd3f630f0 Adds playback_free() function to fix memory leak in playback_stop(). (Bugzilla #42)
Jussi Judin <jjudin+audacious@iki.fi>
parents: 3926
diff changeset
401 {
2d6fd3f630f0 Adds playback_free() function to fix memory leak in playback_stop(). (Bugzilla #42)
Jussi Judin <jjudin+audacious@iki.fi>
parents: 3926
diff changeset
402 g_free(playback->filename);
2d6fd3f630f0 Adds playback_free() function to fix memory leak in playback_stop(). (Bugzilla #42)
Jussi Judin <jjudin+audacious@iki.fi>
parents: 3926
diff changeset
403 g_free(playback->title);
2d6fd3f630f0 Adds playback_free() function to fix memory leak in playback_stop(). (Bugzilla #42)
Jussi Judin <jjudin+audacious@iki.fi>
parents: 3926
diff changeset
404
2d6fd3f630f0 Adds playback_free() function to fix memory leak in playback_stop(). (Bugzilla #42)
Jussi Judin <jjudin+audacious@iki.fi>
parents: 3926
diff changeset
405 g_mutex_free(playback->pb_ready_mutex);
2d6fd3f630f0 Adds playback_free() function to fix memory leak in playback_stop(). (Bugzilla #42)
Jussi Judin <jjudin+audacious@iki.fi>
parents: 3926
diff changeset
406 g_cond_free(playback->pb_ready_cond);
2d6fd3f630f0 Adds playback_free() function to fix memory leak in playback_stop(). (Bugzilla #42)
Jussi Judin <jjudin+audacious@iki.fi>
parents: 3926
diff changeset
407
2d6fd3f630f0 Adds playback_free() function to fix memory leak in playback_stop(). (Bugzilla #42)
Jussi Judin <jjudin+audacious@iki.fi>
parents: 3926
diff changeset
408 g_mutex_free(playback->pb_change_mutex);
2d6fd3f630f0 Adds playback_free() function to fix memory leak in playback_stop(). (Bugzilla #42)
Jussi Judin <jjudin+audacious@iki.fi>
parents: 3926
diff changeset
409 g_cond_free(playback->pb_change_cond);
2d6fd3f630f0 Adds playback_free() function to fix memory leak in playback_stop(). (Bugzilla #42)
Jussi Judin <jjudin+audacious@iki.fi>
parents: 3926
diff changeset
410
2d6fd3f630f0 Adds playback_free() function to fix memory leak in playback_stop(). (Bugzilla #42)
Jussi Judin <jjudin+audacious@iki.fi>
parents: 3926
diff changeset
411 g_slice_free(InputPlayback, playback);
2d6fd3f630f0 Adds playback_free() function to fix memory leak in playback_stop(). (Bugzilla #42)
Jussi Judin <jjudin+audacious@iki.fi>
parents: 3926
diff changeset
412 }
2d6fd3f630f0 Adds playback_free() function to fix memory leak in playback_stop(). (Bugzilla #42)
Jussi Judin <jjudin+audacious@iki.fi>
parents: 3926
diff changeset
413
3710
602002306969 generic playback control API
William Pitcock <nenolod@atheme.org>
parents: 3709
diff changeset
414 void
602002306969 generic playback control API
William Pitcock <nenolod@atheme.org>
parents: 3709
diff changeset
415 playback_run(InputPlayback *playback)
602002306969 generic playback control API
William Pitcock <nenolod@atheme.org>
parents: 3709
diff changeset
416 {
602002306969 generic playback control API
William Pitcock <nenolod@atheme.org>
parents: 3709
diff changeset
417 playback->thread = g_thread_create(playback_monitor_thread, playback, TRUE, NULL);
602002306969 generic playback control API
William Pitcock <nenolod@atheme.org>
parents: 3709
diff changeset
418 }
602002306969 generic playback control API
William Pitcock <nenolod@atheme.org>
parents: 3709
diff changeset
419
2313
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
420 gboolean
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
421 playback_play_file(PlaylistEntry *entry)
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
422 {
3254
William Pitcock <nenolod@atheme-project.org>
parents: 3253
diff changeset
423 InputPlayback *playback;
William Pitcock <nenolod@atheme-project.org>
parents: 3253
diff changeset
424
2313
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
425 g_return_val_if_fail(entry != NULL, FALSE);
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
426
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
427 if (!get_current_output_plugin()) {
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
428 run_no_output_plugin_dialog();
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
429 mainwin_stop_pushed();
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
430 return FALSE;
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
431 }
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
432
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
433 if (cfg.random_skin_on_play)
2443
93686e8815a4 [svn] - Changed playback_set_random_skin() to skin_set_random_skin()
mf0102
parents: 2438
diff changeset
434 skin_set_random_skin();
2313
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
435
3127
c92070f10148 Use ProbeResult (try 1)
William Pitcock <nenolod@atheme-project.org>
parents: 3123
diff changeset
436 if (!entry->decoder)
c92070f10148 Use ProbeResult (try 1)
William Pitcock <nenolod@atheme-project.org>
parents: 3123
diff changeset
437 {
c92070f10148 Use ProbeResult (try 1)
William Pitcock <nenolod@atheme-project.org>
parents: 3123
diff changeset
438 ProbeResult *pr = input_check_file(entry->filename, FALSE);
c92070f10148 Use ProbeResult (try 1)
William Pitcock <nenolod@atheme-project.org>
parents: 3123
diff changeset
439
c92070f10148 Use ProbeResult (try 1)
William Pitcock <nenolod@atheme-project.org>
parents: 3123
diff changeset
440 if (pr != NULL)
c92070f10148 Use ProbeResult (try 1)
William Pitcock <nenolod@atheme-project.org>
parents: 3123
diff changeset
441 {
c92070f10148 Use ProbeResult (try 1)
William Pitcock <nenolod@atheme-project.org>
parents: 3123
diff changeset
442 entry->decoder = pr->ip;
c92070f10148 Use ProbeResult (try 1)
William Pitcock <nenolod@atheme-project.org>
parents: 3123
diff changeset
443 entry->tuple = pr->tuple;
c92070f10148 Use ProbeResult (try 1)
William Pitcock <nenolod@atheme-project.org>
parents: 3123
diff changeset
444
c92070f10148 Use ProbeResult (try 1)
William Pitcock <nenolod@atheme-project.org>
parents: 3123
diff changeset
445 g_free(pr);
c92070f10148 Use ProbeResult (try 1)
William Pitcock <nenolod@atheme-project.org>
parents: 3123
diff changeset
446 }
c92070f10148 Use ProbeResult (try 1)
William Pitcock <nenolod@atheme-project.org>
parents: 3123
diff changeset
447 }
c92070f10148 Use ProbeResult (try 1)
William Pitcock <nenolod@atheme-project.org>
parents: 3123
diff changeset
448
3437
3092a8b3fe34 Big plugin system changes (part 1 of who knows, it's still a big mess):
William Pitcock <nenolod@atheme.org>
parents: 3348
diff changeset
449 if (!entry->decoder || !entry->decoder->enabled)
2313
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
450 {
2436
f346d30bf5ab [svn] Change the input plugin API to use a struct for the currently-playing file.
iabervon
parents: 2420
diff changeset
451 set_current_input_playback(NULL);
2313
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
452
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
453 return FALSE;
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
454 }
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
455
3092
1ddeb9f068ab Backed out changeset b87335249c69d45d253b1b960a9ec7f60b68e5a5
William Pitcock <nenolod@atheme-project.org>
parents: 3089
diff changeset
456 ip_data.playing = TRUE;
3253
6374d66ee623 proper race condition fix
William Pitcock <nenolod@atheme-project.org>
parents: 3252
diff changeset
457
3710
602002306969 generic playback control API
William Pitcock <nenolod@atheme.org>
parents: 3709
diff changeset
458 playback = playback_new();
3253
6374d66ee623 proper race condition fix
William Pitcock <nenolod@atheme-project.org>
parents: 3252
diff changeset
459
6374d66ee623 proper race condition fix
William Pitcock <nenolod@atheme-project.org>
parents: 3252
diff changeset
460 entry->decoder->output = &psuedo_output_plugin;
6374d66ee623 proper race condition fix
William Pitcock <nenolod@atheme-project.org>
parents: 3252
diff changeset
461
6374d66ee623 proper race condition fix
William Pitcock <nenolod@atheme-project.org>
parents: 3252
diff changeset
462 playback->plugin = entry->decoder;
3710
602002306969 generic playback control API
William Pitcock <nenolod@atheme.org>
parents: 3709
diff changeset
463 playback->filename = g_strdup(entry->filename);
3253
6374d66ee623 proper race condition fix
William Pitcock <nenolod@atheme-project.org>
parents: 3252
diff changeset
464 playback->output = &psuedo_output_plugin;
6374d66ee623 proper race condition fix
William Pitcock <nenolod@atheme-project.org>
parents: 3252
diff changeset
465
6374d66ee623 proper race condition fix
William Pitcock <nenolod@atheme-project.org>
parents: 3252
diff changeset
466 set_current_input_playback(playback);
6374d66ee623 proper race condition fix
William Pitcock <nenolod@atheme-project.org>
parents: 3252
diff changeset
467
3710
602002306969 generic playback control API
William Pitcock <nenolod@atheme.org>
parents: 3709
diff changeset
468 playback_run(playback);
602002306969 generic playback control API
William Pitcock <nenolod@atheme.org>
parents: 3709
diff changeset
469
3346
71d8d93f1bad Implemented TrackChange, StatusChange signals, with a stub for CapsChange.
Ben Tucker <ben.tucker@gmail.com>
parents: 3326
diff changeset
470 #ifdef USE_DBUS
71d8d93f1bad Implemented TrackChange, StatusChange signals, with a stub for CapsChange.
Ben Tucker <ben.tucker@gmail.com>
parents: 3326
diff changeset
471 mpris_emit_status_change(mpris, MPRIS_STATUS_PLAY);
71d8d93f1bad Implemented TrackChange, StatusChange signals, with a stub for CapsChange.
Ben Tucker <ben.tucker@gmail.com>
parents: 3326
diff changeset
472 #endif
2313
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
473
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
474 return TRUE;
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
475 }
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
476
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
477 gboolean
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
478 playback_get_playing(void)
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
479 {
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
480 return ip_data.playing;
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
481 }
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
482
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
483 gboolean
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
484 playback_get_paused(void)
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
485 {
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
486 return ip_data.paused;
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
487 }
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
488
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
489 void
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
490 playback_seek(gint time)
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
491 {
3168
ee8a1ed30826 playback_seek(): several WTFs in here (probably from XMMS) fixed
William Pitcock <nenolod@atheme-project.org>
parents: 3165
diff changeset
492 InputPlayback *playback = get_current_input_playback();
2313
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
493 gboolean restore_pause = FALSE;
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
494 gint l=0, r=0;
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
495
2420
b23f52fe132f [svn] Removed util.* from libaudacious
mf0102
parents: 2416
diff changeset
496 g_return_if_fail(ip_data.playing);
3168
ee8a1ed30826 playback_seek(): several WTFs in here (probably from XMMS) fixed
William Pitcock <nenolod@atheme-project.org>
parents: 3165
diff changeset
497 g_return_if_fail(playback != NULL);
2313
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
498
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
499 /* FIXME WORKAROUND...that should work with all plugins
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
500 * mute the volume, start playback again, do the seek, then pause again
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
501 * -Patrick Sudowe
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
502 */
2420
b23f52fe132f [svn] Removed util.* from libaudacious
mf0102
parents: 2416
diff changeset
503 if (ip_data.paused)
2313
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
504 {
2420
b23f52fe132f [svn] Removed util.* from libaudacious
mf0102
parents: 2416
diff changeset
505 restore_pause = TRUE;
b23f52fe132f [svn] Removed util.* from libaudacious
mf0102
parents: 2416
diff changeset
506 output_get_volume(&l, &r);
b23f52fe132f [svn] Removed util.* from libaudacious
mf0102
parents: 2416
diff changeset
507 output_set_volume(0,0);
b23f52fe132f [svn] Removed util.* from libaudacious
mf0102
parents: 2416
diff changeset
508 playback_pause();
2313
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
509 }
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
510
3168
ee8a1ed30826 playback_seek(): several WTFs in here (probably from XMMS) fixed
William Pitcock <nenolod@atheme-project.org>
parents: 3165
diff changeset
511 playback->plugin->seek(playback, time);
3696
418ac922ce87 Use a mutex/condvar/timed wait to allow tickless operation in the output loop.
William Pitcock <nenolod@atheme.org>
parents: 3677
diff changeset
512 playback->set_pb_change(playback);
2313
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
513 free_vis_data();
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
514
2420
b23f52fe132f [svn] Removed util.* from libaudacious
mf0102
parents: 2416
diff changeset
515 if (restore_pause)
2313
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
516 {
2420
b23f52fe132f [svn] Removed util.* from libaudacious
mf0102
parents: 2416
diff changeset
517 playback_pause();
b23f52fe132f [svn] Removed util.* from libaudacious
mf0102
parents: 2416
diff changeset
518 output_set_volume(l, r);
2313
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
519 }
3718
a98b1189777b add new hook, playback seek, and subscribe mainwin_update_song_info() to it.
William Pitcock <nenolod@atheme.org>
parents: 3717
diff changeset
520
3722
57935d8ebefa 100ms delay
William Pitcock <nenolod@atheme.org>
parents: 3720
diff changeset
521 event_queue_timed(100, "playback seek", playback);
2313
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
522 }
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
523
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
524 void
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
525 playback_seek_relative(gint offset)
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
526 {
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
527 gint time = CLAMP(playback_get_time() / 1000 + offset,
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
528 0, playlist_get_current_length(playlist_get_active()) / 1000 - 1);
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
529 playback_seek(time);
3149d4b1a9a9 [svn] - objective-make autodepend fixes
nenolod
parents:
diff changeset
530 }