Mercurial > audlegacy
annotate doc/TODO @ 4779:aa79a84627b9
g_basename is deprecated, replace with g_path_get_basename.
author | Tony Vroon <chainsaw@gentoo.org> |
---|---|
date | Fri, 26 Sep 2008 21:02:49 +0100 |
parents | dde7262d0a35 |
children |
rev | line source |
---|---|
4535
2cfd5d48d787
Added a TODO (mostly my own semi-organized ramblings, feel free to add your own.)
Matti Hamalainen <ccr@tnsp.org>
parents:
diff
changeset
|
1 Long-term "TODO" |
2cfd5d48d787
Added a TODO (mostly my own semi-organized ramblings, feel free to add your own.)
Matti Hamalainen <ccr@tnsp.org>
parents:
diff
changeset
|
2 ================ |
4622
a615322374c8
Add notes about possible cleanups into TODO.
Matti Hamalainen <ccr@tnsp.org>
parents:
4617
diff
changeset
|
3 - output plugins should have some kind of (optional) "query available formats" |
a615322374c8
Add notes about possible cleanups into TODO.
Matti Hamalainen <ccr@tnsp.org>
parents:
4617
diff
changeset
|
4 function, so that the core could determine what should be used. perhaps |
a615322374c8
Add notes about possible cleanups into TODO.
Matti Hamalainen <ccr@tnsp.org>
parents:
4617
diff
changeset
|
5 offer users this info? |
a615322374c8
Add notes about possible cleanups into TODO.
Matti Hamalainen <ccr@tnsp.org>
parents:
4617
diff
changeset
|
6 |
4617
de510eb8165a
Added a note about XMMS compat output plugin hack.
Matti Hamalainen <ccr@tnsp.org>
parents:
4603
diff
changeset
|
7 - remove the idiotic XMMS compatibility output plugin hack. |
de510eb8165a
Added a note about XMMS compat output plugin hack.
Matti Hamalainen <ccr@tnsp.org>
parents:
4603
diff
changeset
|
8 |
4622
a615322374c8
Add notes about possible cleanups into TODO.
Matti Hamalainen <ccr@tnsp.org>
parents:
4617
diff
changeset
|
9 - all audio plugins (input, output, effect) should be made completely |
a615322374c8
Add notes about possible cleanups into TODO.
Matti Hamalainen <ccr@tnsp.org>
parents:
4617
diff
changeset
|
10 re-entrant. this probably means severely breaking the API. not very |
a615322374c8
Add notes about possible cleanups into TODO.
Matti Hamalainen <ccr@tnsp.org>
parents:
4617
diff
changeset
|
11 trivial either :( |
a615322374c8
Add notes about possible cleanups into TODO.
Matti Hamalainen <ccr@tnsp.org>
parents:
4617
diff
changeset
|
12 * basically plugin init() should allocate / init a state struct to |
a615322374c8
Add notes about possible cleanups into TODO.
Matti Hamalainen <ccr@tnsp.org>
parents:
4617
diff
changeset
|
13 be used in all operations, etc. |
a615322374c8
Add notes about possible cleanups into TODO.
Matti Hamalainen <ccr@tnsp.org>
parents:
4617
diff
changeset
|
14 |
4617
de510eb8165a
Added a note about XMMS compat output plugin hack.
Matti Hamalainen <ccr@tnsp.org>
parents:
4603
diff
changeset
|
15 |
4603
fba10959a743
Note about equalizer problems.
Matti Hamalainen <ccr@tnsp.org>
parents:
4538
diff
changeset
|
16 - the equalizer should be fixed .. in more than one way: |
fba10959a743
Note about equalizer problems.
Matti Hamalainen <ccr@tnsp.org>
parents:
4538
diff
changeset
|
17 * equalizer was broken by asphyx's libSAD |
fba10959a743
Note about equalizer problems.
Matti Hamalainen <ccr@tnsp.org>
parents:
4538
diff
changeset
|
18 * lots of things are hardcoded, like number of EQ bands. this should |
fba10959a743
Note about equalizer problems.
Matti Hamalainen <ccr@tnsp.org>
parents:
4538
diff
changeset
|
19 be changed. |
fba10959a743
Note about equalizer problems.
Matti Hamalainen <ccr@tnsp.org>
parents:
4538
diff
changeset
|
20 |
fba10959a743
Note about equalizer problems.
Matti Hamalainen <ccr@tnsp.org>
parents:
4538
diff
changeset
|
21 |
4537
9fb29ee322d3
Added a note about problems with certain URIs.
Matti Hamalainen <ccr@tnsp.org>
parents:
4535
diff
changeset
|
22 - URIs with "?" are stripped (the subtune support uses that), which is bad, |
9fb29ee322d3
Added a note about problems with certain URIs.
Matti Hamalainen <ccr@tnsp.org>
parents:
4535
diff
changeset
|
23 because some (a lot) of real-world URLs use 'GET' arguments. this breaks |
9fb29ee322d3
Added a note about problems with certain URIs.
Matti Hamalainen <ccr@tnsp.org>
parents:
4535
diff
changeset
|
24 probing, but current input probing is confusing (and possibly broken anyway) |
9fb29ee322d3
Added a note about problems with certain URIs.
Matti Hamalainen <ccr@tnsp.org>
parents:
4535
diff
changeset
|
25 and should be refactored. |
9fb29ee322d3
Added a note about problems with certain URIs.
Matti Hamalainen <ccr@tnsp.org>
parents:
4535
diff
changeset
|
26 |
9fb29ee322d3
Added a note about problems with certain URIs.
Matti Hamalainen <ccr@tnsp.org>
parents:
4535
diff
changeset
|
27 |
4535
2cfd5d48d787
Added a TODO (mostly my own semi-organized ramblings, feel free to add your own.)
Matti Hamalainen <ccr@tnsp.org>
parents:
diff
changeset
|
28 - mime-types support: |
2cfd5d48d787
Added a TODO (mostly my own semi-organized ramblings, feel free to add your own.)
Matti Hamalainen <ccr@tnsp.org>
parents:
diff
changeset
|
29 * there is already code for mime support, but I think it may not be |
2cfd5d48d787
Added a TODO (mostly my own semi-organized ramblings, feel free to add your own.)
Matti Hamalainen <ccr@tnsp.org>
parents:
diff
changeset
|
30 sufficient as it is designed for input plugins only -- |
2cfd5d48d787
Added a TODO (mostly my own semi-organized ramblings, feel free to add your own.)
Matti Hamalainen <ccr@tnsp.org>
parents:
diff
changeset
|
31 also playlist containers etc. need this (IMHO) |
2cfd5d48d787
Added a TODO (mostly my own semi-organized ramblings, feel free to add your own.)
Matti Hamalainen <ccr@tnsp.org>
parents:
diff
changeset
|
32 |
2cfd5d48d787
Added a TODO (mostly my own semi-organized ramblings, feel free to add your own.)
Matti Hamalainen <ccr@tnsp.org>
parents:
diff
changeset
|
33 * might be nicer to have the type registrations in plugin struct |
2cfd5d48d787
Added a TODO (mostly my own semi-organized ramblings, feel free to add your own.)
Matti Hamalainen <ccr@tnsp.org>
parents:
diff
changeset
|
34 instead of a separate function, a'la vfs_extensions. |
2cfd5d48d787
Added a TODO (mostly my own semi-organized ramblings, feel free to add your own.)
Matti Hamalainen <ccr@tnsp.org>
parents:
diff
changeset
|
35 |
2cfd5d48d787
Added a TODO (mostly my own semi-organized ramblings, feel free to add your own.)
Matti Hamalainen <ccr@tnsp.org>
parents:
diff
changeset
|
36 |
2cfd5d48d787
Added a TODO (mostly my own semi-organized ramblings, feel free to add your own.)
Matti Hamalainen <ccr@tnsp.org>
parents:
diff
changeset
|
37 - document the different APIs via Doxygen |
2cfd5d48d787
Added a TODO (mostly my own semi-organized ramblings, feel free to add your own.)
Matti Hamalainen <ccr@tnsp.org>
parents:
diff
changeset
|
38 |
2cfd5d48d787
Added a TODO (mostly my own semi-organized ramblings, feel free to add your own.)
Matti Hamalainen <ccr@tnsp.org>
parents:
diff
changeset
|
39 |
2cfd5d48d787
Added a TODO (mostly my own semi-organized ramblings, feel free to add your own.)
Matti Hamalainen <ccr@tnsp.org>
parents:
diff
changeset
|
40 - unified debugging/message system, for core and plugins |
2cfd5d48d787
Added a TODO (mostly my own semi-organized ramblings, feel free to add your own.)
Matti Hamalainen <ccr@tnsp.org>
parents:
diff
changeset
|
41 * something like glib logging system, with logging levels |
2cfd5d48d787
Added a TODO (mostly my own semi-organized ramblings, feel free to add your own.)
Matti Hamalainen <ccr@tnsp.org>
parents:
diff
changeset
|
42 * hardcoded "debugging" levels and/or macro-wrappers that would |
2cfd5d48d787
Added a TODO (mostly my own semi-organized ramblings, feel free to add your own.)
Matti Hamalainen <ccr@tnsp.org>
parents:
diff
changeset
|
43 disable superfluous debugging output compile-time. |
2cfd5d48d787
Added a TODO (mostly my own semi-organized ramblings, feel free to add your own.)
Matti Hamalainen <ccr@tnsp.org>
parents:
diff
changeset
|
44 --- being planned by ccr |
2cfd5d48d787
Added a TODO (mostly my own semi-organized ramblings, feel free to add your own.)
Matti Hamalainen <ccr@tnsp.org>
parents:
diff
changeset
|
45 |
2cfd5d48d787
Added a TODO (mostly my own semi-organized ramblings, feel free to add your own.)
Matti Hamalainen <ccr@tnsp.org>
parents:
diff
changeset
|
46 |
2cfd5d48d787
Added a TODO (mostly my own semi-organized ramblings, feel free to add your own.)
Matti Hamalainen <ccr@tnsp.org>
parents:
diff
changeset
|
47 - audacious VFS is not 64-bit offset safe, breakage will most likely occur, |
2cfd5d48d787
Added a TODO (mostly my own semi-organized ramblings, feel free to add your own.)
Matti Hamalainen <ccr@tnsp.org>
parents:
diff
changeset
|
48 if files larger than 2^31 are used (rather unlikely, tho, but still...) |
2cfd5d48d787
Added a TODO (mostly my own semi-organized ramblings, feel free to add your own.)
Matti Hamalainen <ccr@tnsp.org>
parents:
diff
changeset
|
49 |
2cfd5d48d787
Added a TODO (mostly my own semi-organized ramblings, feel free to add your own.)
Matti Hamalainen <ccr@tnsp.org>
parents:
diff
changeset
|
50 * nenolod says: current vfs sucks, it needs a "rewrite": |
2cfd5d48d787
Added a TODO (mostly my own semi-organized ramblings, feel free to add your own.)
Matti Hamalainen <ccr@tnsp.org>
parents:
diff
changeset
|
51 - buffering support |
2cfd5d48d787
Added a TODO (mostly my own semi-organized ramblings, feel free to add your own.)
Matti Hamalainen <ccr@tnsp.org>
parents:
diff
changeset
|
52 - non-blocking I/O support |
2cfd5d48d787
Added a TODO (mostly my own semi-organized ramblings, feel free to add your own.)
Matti Hamalainen <ccr@tnsp.org>
parents:
diff
changeset
|
53 |
2cfd5d48d787
Added a TODO (mostly my own semi-organized ramblings, feel free to add your own.)
Matti Hamalainen <ccr@tnsp.org>
parents:
diff
changeset
|
54 |
2cfd5d48d787
Added a TODO (mostly my own semi-organized ramblings, feel free to add your own.)
Matti Hamalainen <ccr@tnsp.org>
parents:
diff
changeset
|
55 - {core,plugins}/configure.ac need some cleanup loving. |
4645
e36c207de2c1
Added a separate chapter for playlist code issues and ranting about them (including my own ideas
Matti Hamalainen <ccr@tnsp.org>
parents:
4622
diff
changeset
|
56 * make session management (SM) optional. (done) |
e36c207de2c1
Added a separate chapter for playlist code issues and ranting about them (including my own ideas
Matti Hamalainen <ccr@tnsp.org>
parents:
4622
diff
changeset
|
57 * build system cleanups .. extra.mk.in? wtf? (done in core, progressing |
e36c207de2c1
Added a separate chapter for playlist code issues and ranting about them (including my own ideas
Matti Hamalainen <ccr@tnsp.org>
parents:
4622
diff
changeset
|
58 for plugins) |
4535
2cfd5d48d787
Added a TODO (mostly my own semi-organized ramblings, feel free to add your own.)
Matti Hamalainen <ccr@tnsp.org>
parents:
diff
changeset
|
59 --- this is in progress, worked on by ccr |
2cfd5d48d787
Added a TODO (mostly my own semi-organized ramblings, feel free to add your own.)
Matti Hamalainen <ccr@tnsp.org>
parents:
diff
changeset
|
60 |
2cfd5d48d787
Added a TODO (mostly my own semi-organized ramblings, feel free to add your own.)
Matti Hamalainen <ccr@tnsp.org>
parents:
diff
changeset
|
61 |
2cfd5d48d787
Added a TODO (mostly my own semi-organized ramblings, feel free to add your own.)
Matti Hamalainen <ccr@tnsp.org>
parents:
diff
changeset
|
62 - plugin rewrites: |
2cfd5d48d787
Added a TODO (mostly my own semi-organized ramblings, feel free to add your own.)
Matti Hamalainen <ccr@tnsp.org>
parents:
diff
changeset
|
63 * madplug |
2cfd5d48d787
Added a TODO (mostly my own semi-organized ramblings, feel free to add your own.)
Matti Hamalainen <ccr@tnsp.org>
parents:
diff
changeset
|
64 * modplug (in progress by ccr) |
2cfd5d48d787
Added a TODO (mostly my own semi-organized ramblings, feel free to add your own.)
Matti Hamalainen <ccr@tnsp.org>
parents:
diff
changeset
|
65 * scrobbler |
2cfd5d48d787
Added a TODO (mostly my own semi-organized ramblings, feel free to add your own.)
Matti Hamalainen <ccr@tnsp.org>
parents:
diff
changeset
|
66 |
2cfd5d48d787
Added a TODO (mostly my own semi-organized ramblings, feel free to add your own.)
Matti Hamalainen <ccr@tnsp.org>
parents:
diff
changeset
|
67 |
4645
e36c207de2c1
Added a separate chapter for playlist code issues and ranting about them (including my own ideas
Matti Hamalainen <ccr@tnsp.org>
parents:
4622
diff
changeset
|
68 Playlist code issues |
e36c207de2c1
Added a separate chapter for playlist code issues and ranting about them (including my own ideas
Matti Hamalainen <ccr@tnsp.org>
parents:
4622
diff
changeset
|
69 ==================== |
e36c207de2c1
Added a separate chapter for playlist code issues and ranting about them (including my own ideas
Matti Hamalainen <ccr@tnsp.org>
parents:
4622
diff
changeset
|
70 It's been determined that the playlist code should be completely refactored. |
e36c207de2c1
Added a separate chapter for playlist code issues and ranting about them (including my own ideas
Matti Hamalainen <ccr@tnsp.org>
parents:
4622
diff
changeset
|
71 |
e36c207de2c1
Added a separate chapter for playlist code issues and ranting about them (including my own ideas
Matti Hamalainen <ccr@tnsp.org>
parents:
4622
diff
changeset
|
72 ATTENTION! Add more stuff here! Issues that you can think of, suggestions |
e36c207de2c1
Added a separate chapter for playlist code issues and ranting about them (including my own ideas
Matti Hamalainen <ccr@tnsp.org>
parents:
4622
diff
changeset
|
73 for solutions, etc. References to algorithms... |
e36c207de2c1
Added a separate chapter for playlist code issues and ranting about them (including my own ideas
Matti Hamalainen <ccr@tnsp.org>
parents:
4622
diff
changeset
|
74 |
e36c207de2c1
Added a separate chapter for playlist code issues and ranting about them (including my own ideas
Matti Hamalainen <ccr@tnsp.org>
parents:
4622
diff
changeset
|
75 |
e36c207de2c1
Added a separate chapter for playlist code issues and ranting about them (including my own ideas
Matti Hamalainen <ccr@tnsp.org>
parents:
4622
diff
changeset
|
76 - code (mostly in playlist.c) is unnecessarily complex in some places and |
e36c207de2c1
Added a separate chapter for playlist code issues and ranting about them (including my own ideas
Matti Hamalainen <ccr@tnsp.org>
parents:
4622
diff
changeset
|
77 there is lots of useless duplication. |
e36c207de2c1
Added a separate chapter for playlist code issues and ranting about them (including my own ideas
Matti Hamalainen <ccr@tnsp.org>
parents:
4622
diff
changeset
|
78 |
e36c207de2c1
Added a separate chapter for playlist code issues and ranting about them (including my own ideas
Matti Hamalainen <ccr@tnsp.org>
parents:
4622
diff
changeset
|
79 - what data structures should be used? things that factor in are |
e36c207de2c1
Added a separate chapter for playlist code issues and ranting about them (including my own ideas
Matti Hamalainen <ccr@tnsp.org>
parents:
4622
diff
changeset
|
80 efficiency and scalability - memory usage and speed. |
e36c207de2c1
Added a separate chapter for playlist code issues and ranting about them (including my own ideas
Matti Hamalainen <ccr@tnsp.org>
parents:
4622
diff
changeset
|
81 see also below and notice the manipulation bottleneck(s). |
e36c207de2c1
Added a separate chapter for playlist code issues and ranting about them (including my own ideas
Matti Hamalainen <ccr@tnsp.org>
parents:
4622
diff
changeset
|
82 |
e36c207de2c1
Added a separate chapter for playlist code issues and ranting about them (including my own ideas
Matti Hamalainen <ccr@tnsp.org>
parents:
4622
diff
changeset
|
83 (different structures typically have bottlenecks in different places.) |
e36c207de2c1
Added a separate chapter for playlist code issues and ranting about them (including my own ideas
Matti Hamalainen <ccr@tnsp.org>
parents:
4622
diff
changeset
|
84 |
e36c207de2c1
Added a separate chapter for playlist code issues and ranting about them (including my own ideas
Matti Hamalainen <ccr@tnsp.org>
parents:
4622
diff
changeset
|
85 - playlist_clear() is VERY slow with large playlists, possible reasons? |
e36c207de2c1
Added a separate chapter for playlist code issues and ranting about them (including my own ideas
Matti Hamalainen <ccr@tnsp.org>
parents:
4622
diff
changeset
|
86 * GList scalability sucks? (it does, but is it the real reason here? need |
e36c207de2c1
Added a separate chapter for playlist code issues and ranting about them (including my own ideas
Matti Hamalainen <ccr@tnsp.org>
parents:
4622
diff
changeset
|
87 to profile this...) |
e36c207de2c1
Added a separate chapter for playlist code issues and ranting about them (including my own ideas
Matti Hamalainen <ccr@tnsp.org>
parents:
4622
diff
changeset
|
88 * maybe playlist_entry_free() is at fault instead? (see also below) |
4535
2cfd5d48d787
Added a TODO (mostly my own semi-organized ramblings, feel free to add your own.)
Matti Hamalainen <ccr@tnsp.org>
parents:
diff
changeset
|
89 * mowgli_heap_free()? |
2cfd5d48d787
Added a TODO (mostly my own semi-organized ramblings, feel free to add your own.)
Matti Hamalainen <ccr@tnsp.org>
parents:
diff
changeset
|
90 |
2cfd5d48d787
Added a TODO (mostly my own semi-organized ramblings, feel free to add your own.)
Matti Hamalainen <ccr@tnsp.org>
parents:
diff
changeset
|
91 possible solutions? |
2cfd5d48d787
Added a TODO (mostly my own semi-organized ramblings, feel free to add your own.)
Matti Hamalainen <ccr@tnsp.org>
parents:
diff
changeset
|
92 * mempools for playlist data? not sure if possible, the tuple heaps |
2cfd5d48d787
Added a TODO (mostly my own semi-organized ramblings, feel free to add your own.)
Matti Hamalainen <ccr@tnsp.org>
parents:
diff
changeset
|
93 should then somehow be collected into pools... |
4536
b0def426de68
Add note about GtkTreeStore being fairly fast.
William Pitcock <nenolod@atheme.org>
parents:
4535
diff
changeset
|
94 * storing the playlist as a GtkTreeModel seems to be fairly fast (at least |
b0def426de68
Add note about GtkTreeStore being fairly fast.
William Pitcock <nenolod@atheme.org>
parents:
4535
diff
changeset
|
95 in mudkip-player it is) --nenolod |
4535
2cfd5d48d787
Added a TODO (mostly my own semi-organized ramblings, feel free to add your own.)
Matti Hamalainen <ccr@tnsp.org>
parents:
diff
changeset
|
96 |
4645
e36c207de2c1
Added a separate chapter for playlist code issues and ranting about them (including my own ideas
Matti Hamalainen <ccr@tnsp.org>
parents:
4622
diff
changeset
|
97 - current implementation has a nasty racecondition in scanner thread vs. playlist |
e36c207de2c1
Added a separate chapter for playlist code issues and ranting about them (including my own ideas
Matti Hamalainen <ccr@tnsp.org>
parents:
4622
diff
changeset
|
98 manipulation (playlist_clear(), free, etc.) |
e36c207de2c1
Added a separate chapter for playlist code issues and ranting about them (including my own ideas
Matti Hamalainen <ccr@tnsp.org>
parents:
4622
diff
changeset
|
99 |
e36c207de2c1
Added a separate chapter for playlist code issues and ranting about them (including my own ideas
Matti Hamalainen <ccr@tnsp.org>
parents:
4622
diff
changeset
|
100 * new code should be water-tight when it comes to locking. simpler code |
e36c207de2c1
Added a separate chapter for playlist code issues and ranting about them (including my own ideas
Matti Hamalainen <ccr@tnsp.org>
parents:
4622
diff
changeset
|
101 with more "atomic operations" should help. |
e36c207de2c1
Added a separate chapter for playlist code issues and ranting about them (including my own ideas
Matti Hamalainen <ccr@tnsp.org>
parents:
4622
diff
changeset
|
102 |
e36c207de2c1
Added a separate chapter for playlist code issues and ranting about them (including my own ideas
Matti Hamalainen <ccr@tnsp.org>
parents:
4622
diff
changeset
|
103 |
e36c207de2c1
Added a separate chapter for playlist code issues and ranting about them (including my own ideas
Matti Hamalainen <ccr@tnsp.org>
parents:
4622
diff
changeset
|
104 ccr rants: |
e36c207de2c1
Added a separate chapter for playlist code issues and ranting about them (including my own ideas
Matti Hamalainen <ccr@tnsp.org>
parents:
4622
diff
changeset
|
105 ---------- |
e36c207de2c1
Added a separate chapter for playlist code issues and ranting about them (including my own ideas
Matti Hamalainen <ccr@tnsp.org>
parents:
4622
diff
changeset
|
106 I suggest that the whole playlist manipulation should be moved into separate |
e36c207de2c1
Added a separate chapter for playlist code issues and ranting about them (including my own ideas
Matti Hamalainen <ccr@tnsp.org>
parents:
4622
diff
changeset
|
107 thread. What I mean, is that the playlist scanning (e.g. probing for |
e36c207de2c1
Added a separate chapter for playlist code issues and ranting about them (including my own ideas
Matti Hamalainen <ccr@tnsp.org>
parents:
4622
diff
changeset
|
108 metadata) AND adding files into playlist should be in a thread. |
e36c207de2c1
Added a separate chapter for playlist code issues and ranting about them (including my own ideas
Matti Hamalainen <ccr@tnsp.org>
parents:
4622
diff
changeset
|
109 |
e36c207de2c1
Added a separate chapter for playlist code issues and ranting about them (including my own ideas
Matti Hamalainen <ccr@tnsp.org>
parents:
4622
diff
changeset
|
110 Adding files would work like this: |
e36c207de2c1
Added a separate chapter for playlist code issues and ranting about them (including my own ideas
Matti Hamalainen <ccr@tnsp.org>
parents:
4622
diff
changeset
|
111 1) feed URIs to playlist handler thread |
e36c207de2c1
Added a separate chapter for playlist code issues and ranting about them (including my own ideas
Matti Hamalainen <ccr@tnsp.org>
parents:
4622
diff
changeset
|
112 2) playlist handler notices new URIs and starts checking if they are valid |
e36c207de2c1
Added a separate chapter for playlist code issues and ranting about them (including my own ideas
Matti Hamalainen <ccr@tnsp.org>
parents:
4622
diff
changeset
|
113 aka probing or other checks. |
e36c207de2c1
Added a separate chapter for playlist code issues and ranting about them (including my own ideas
Matti Hamalainen <ccr@tnsp.org>
parents:
4622
diff
changeset
|
114 3) valid files/URIs get added to the playlist. unsupported files get discarded. |
e36c207de2c1
Added a separate chapter for playlist code issues and ranting about them (including my own ideas
Matti Hamalainen <ccr@tnsp.org>
parents:
4622
diff
changeset
|
115 |
e36c207de2c1
Added a separate chapter for playlist code issues and ranting about them (including my own ideas
Matti Hamalainen <ccr@tnsp.org>
parents:
4622
diff
changeset
|
116 Possible benefits of this approach: |
e36c207de2c1
Added a separate chapter for playlist code issues and ranting about them (including my own ideas
Matti Hamalainen <ccr@tnsp.org>
parents:
4622
diff
changeset
|
117 - checking for "can we play this?", which currently happens in main |
e36c207de2c1
Added a separate chapter for playlist code issues and ranting about them (including my own ideas
Matti Hamalainen <ccr@tnsp.org>
parents:
4622
diff
changeset
|
118 thread and blocks the GUI etc, would now happen asynchronously. |
e36c207de2c1
Added a separate chapter for playlist code issues and ranting about them (including my own ideas
Matti Hamalainen <ccr@tnsp.org>
parents:
4622
diff
changeset
|
119 - improved user experience: less blocking generally, and it would be even |
e36c207de2c1
Added a separate chapter for playlist code issues and ranting about them (including my own ideas
Matti Hamalainen <ccr@tnsp.org>
parents:
4622
diff
changeset
|
120 possible to _parallelize_ probing, if we wanted to, getting metadata |
e36c207de2c1
Added a separate chapter for playlist code issues and ranting about them (including my own ideas
Matti Hamalainen <ccr@tnsp.org>
parents:
4622
diff
changeset
|
121 for multiple files at the same time. (this could be a user-settable |
e36c207de2c1
Added a separate chapter for playlist code issues and ranting about them (including my own ideas
Matti Hamalainen <ccr@tnsp.org>
parents:
4622
diff
changeset
|
122 option.) |
e36c207de2c1
Added a separate chapter for playlist code issues and ranting about them (including my own ideas
Matti Hamalainen <ccr@tnsp.org>
parents:
4622
diff
changeset
|
123 |
e36c207de2c1
Added a separate chapter for playlist code issues and ranting about them (including my own ideas
Matti Hamalainen <ccr@tnsp.org>
parents:
4622
diff
changeset
|
124 Bad sides: |
e36c207de2c1
Added a separate chapter for playlist code issues and ranting about them (including my own ideas
Matti Hamalainen <ccr@tnsp.org>
parents:
4622
diff
changeset
|
125 - requires careful design and thread locking .. but that we need anyway. |
e36c207de2c1
Added a separate chapter for playlist code issues and ranting about them (including my own ideas
Matti Hamalainen <ccr@tnsp.org>
parents:
4622
diff
changeset
|
126 - possibly might make playlist GUI representation harder. I am not sure |
e36c207de2c1
Added a separate chapter for playlist code issues and ranting about them (including my own ideas
Matti Hamalainen <ccr@tnsp.org>
parents:
4622
diff
changeset
|
127 what effects this might have, feel free to pitch in thoughts... |
e36c207de2c1
Added a separate chapter for playlist code issues and ranting about them (including my own ideas
Matti Hamalainen <ccr@tnsp.org>
parents:
4622
diff
changeset
|
128 |
e36c207de2c1
Added a separate chapter for playlist code issues and ranting about them (including my own ideas
Matti Hamalainen <ccr@tnsp.org>
parents:
4622
diff
changeset
|
129 |