Hydrogenaudio Forums

anza

May 30 2005, 08:12

QUOTE(NEMO7538 @ May 30 2005, 04:02 PM)

QUOTE(dano @ May 30 2005, 05:56 AM)

......

You can do this in the Layout options with splitters etc..

Not exactly. Duplicating the "columns playlist" works but both views have the same content.

What I requested would enable to have different selections (e.g. history for one, playlist switcher for another, ...) applied to these views.

I'd be delighted to ear about developper's opinion and position on this feature.

I really don't understand what you're after. It is possible to have history and/or albumlist above the playlist. The playback queue isn't possible yet (well neither is history with foobar 0.9/columns 0.1.3).

filon

May 30 2005, 08:24

QUOTE(NEMO7538 @ May 30 2005, 06:02 AM)

QUOTE(dano @ May 30 2005, 05:56 AM)

......

You can do this in the Layout options with splitters etc..

Not exactly. Duplicating the "columns playlist" works but both views have the same content.

What I requested would enable to have different selections (e.g. history for one, playlist switcher for another, ...) applied to these views.

I'd be delighted to ear about developper's opinion and position on this feature.

Looks that right now multiple playlist views are bound to the same data and our request should be a matter of per-view data (active playlist, view configuration).
I see this as one "master" playlist view controlled by playlist switcher etc. and possibly muliple other playlist views for selected playlists.

For example, album list would fill "Database View" playlist which will have it's own playlist view, and apart from that there would be main playlist view where some of the tracks from "Database View" could be copied etc.

DotNoir

May 30 2005, 08:26

I'm not completely sure if you're after having multiple playlists displaying different material or (like I undersood from your post) that you'd be able to have albumlist on top half and playlist on lower half of the window. The second option is already possible -> http://tritonusrock.com/columns.png

filon

May 30 2005, 08:34

QUOTE(musicmusic @ May 29 2005, 03:21 PM)

Album list panel 0.2 beta for columns 0.1.3 pre-alpha 11 and 0.9 beta 4 is out:
Download

When media library is empty, album list crashes.

CODE

Illegal operation:
Code: C0000005h, flags: 00000000h, address: 00E92A79h
Access violation, operation: read, address: 00000000h
Call path:
WinMain=>app_mainloop=>album_list_panel_refresh_tree=>album_list_panel_refresh_tree=>album_list_panel_refresh_tree=>album_list_panel_refresh_tree=>album_list_panel_refresh_tree=>album_list_panel_refresh_tree=>album_list_panel_refresh_tree=>album_list_panel_refresh_tree=>album_list_panel_refresh_tree=>album_list_panel_refresh_tree=>album_list_panel_refresh_tree=>album_list_panel_refresh_tree=>album_list_panel_refresh_tree
This is the first crash logged by this instance.
Code bytes (00E92A79h):
00E92A39h:  8B 7C 24 0C 57 8B F1 E8 CB FB FF FF 8B 46 04 8B
00E92A49h:  04 B8 5F C6 40 21 00 5E C2 10 00 56 57 8B 7C 24
00E92A59h:  0C 57 8B F1 E8 AE FB FF FF 8B 46 04 8B 04 B8 5F
00E92A69h:  C6 40 21 00 5E C2 08 00 8B 51 04 8B 44 24 04 56
00E92A79h:  8B 34 82 80 7E 20 00 75 06 50 E8 FF F9 FF FF 8D
00E92A89h:  46 14 5E C2 04 00 55 8B EC 53 56 57 FF 75 08 E8
00E92A99h:  D4 FF FF FF 33 F6 B9 BC 87 EB 00 8B F8 89 75 08
00E92AA9h:  E8 0B F7 FD FF 8B D8 3B DE 76 1B 8B 07 56 8B CF
Stack (0011FA30h):
0011FA10h:  00000001 0011FC4C 7C90EB94 00000001
0011FA20h:  F52A457C 0000003B 00000083 006A54B0
0011FA30h:  00B453C0 00E92AEA 00000000 46010000
0011FA40h:  00B453C0 0011FA74 77D39056 00E9389E
0011FA50h:  00000000 00B453C0 00B453C0 77D39056
0011FA60h:  FFFFFFFF 00E93883 0011FB08 00EA7760
0011FA70h:  FFFFFFFF 0011FB14 00E8E22B 0011FB90
0011FA80h:  00000000 00B453C0 00000000 00000000
0011FA90h:  00E70000 0011FBD4 40000400 00EAF038
0011FAA0h:  00F74C60 00EAF038 0000001C 00000004
0011FAB0h:  00000000 00000000 00000000 00000000
0011FAC0h:  00000000 0000001C 00000007 00000000
0011FAD0h:  FFFFFFFF 00000000 00000000 00000000
0011FAE0h:  00000000 00000000 88527726 00000000
0011FAF0h:  00000000 00000000 004A8CDC 00000000
0011FB00h:  46010000 000000F1 0011FC8C 00EA7351
0011FB10h:  00000000 00000001 00E7F619 00000001
0011FB20h:  00000100 00000000 00B453C0 0011FC98
0011FB30h:  00000000 FFFFFFF5 00000000 00000000
0011FB40h:  00000000 00000190 01000000 00000000
Registers:
EAX: 00000000, EBX: 77D39056, ECX: 00EB882C, EDX: 00000000
ESI: 00B453C0, EDI: 46010000, EBP: 0011FA74, ESP: 0011FA30
Crash location: "foo_ui_columns", loaded at 00E70000h - 00EC6000h

Loaded modules:
foobar2000                       loaded at 00400000h - 004B8000h
ntdll                            loaded at 7C900000h - 7C9B2000h
kernel32                         loaded at 7C800000h - 7C8FB000h
COMCTL32                         loaded at 773C0000h - 774C2000h
msvcrt                           loaded at 77C00000h - 77C58000h
ADVAPI32                         loaded at 77DC0000h - 77E6C000h
RPCRT4                           loaded at 77E70000h - 77F01000h
GDI32                            loaded at 77F10000h - 77F56000h
USER32                           loaded at 77D30000h - 77DC0000h
SHLWAPI                          loaded at 77F60000h - 77FD6000h
DSOUND                           loaded at 73EA0000h - 73EFC000h
ole32                            loaded at 774D0000h - 7760D000h
WINMM                            loaded at 76B20000h - 76B4E000h
VERSION                          loaded at 77BF0000h - 77BF8000h
SHELL32                          loaded at 7C9C0000h - 7D1DB000h
shared                           loaded at 10000000h - 10010000h
comdlg32                         loaded at 76380000h - 763C9000h
foo_ac3                          loaded at 00A20000h - 00A35000h
foo_alac                         loaded at 003F0000h - 003F9000h
foo_albumlist                    loaded at 00A40000h - 00A54000h
foo_autoplaylist                 loaded at 00A60000h - 00A72000h
foo_benchmark                    loaded at 00A80000h - 00A91000h
foo_burninate                    loaded at 00AA0000h - 00AB4000h
foo_cdda                         loaded at 00AC0000h - 00AD3000h
foo_converter                    loaded at 00AE0000h - 00B02000h
foo_dbsearch                     loaded at 00B10000h - 00B38000h
MSVCR71                          loaded at 7C340000h - 7C396000h
foo_dsp_crossfeed                loaded at 00B50000h - 00B56000h
foo_dsp_std                      loaded at 00B60000h - 00B78000h
foo_dumb                         loaded at 00B80000h - 00BB4000h
foo_input_std                    loaded at 00BC0000h - 00CF4000h
MSACM32                          loaded at 77BD0000h - 77BE5000h
WS2_32                           loaded at 71A50000h - 71A67000h
WS2HELP                          loaded at 71A40000h - 71A48000h
foo_masstag                      loaded at 00D10000h - 00D38000h
foo_midi                         loaded at 00D40000h - 00DF7000h
WINSPOOL                         loaded at 72F90000h - 72FB6000h
OLEAUT32                         loaded at 77110000h - 7719C000h
foo_playlistgen                  loaded at 00E20000h - 00E30000h
foo_rgscan                       loaded at 00E30000h - 00E47000h
foo_uie_albumlist                loaded at 00E50000h - 00E66000h
foo_ui_columns                   loaded at 00E70000h - 00EC6000h
foo_ui_std                       loaded at 00ED0000h - 00EF4000h
OLEACC                           loaded at 74C30000h - 74C5C000h
MSVCP60                          loaded at 76050000h - 760B5000h
foo_unpack                       loaded at 00F10000h - 00F45000h
foo_utils                        loaded at 00F50000h - 00F65000h
uxtheme                          loaded at 5B1D0000h - 5B208000h
imagehlp                         loaded at 76C80000h - 76CA8000h
DBGHELP                          loaded at 59BC0000h - 59C61000h

Stack dump analysis:
Address: 00E92AEAh, location: "foo_ui_columns", loaded at 00E70000h - 00EC6000h
Address: 77D39056h, location: "USER32", loaded at 77D30000h - 77DC0000h
Symbol: "SetScrollInfo" (+00000000h)
Address: 00E9389Eh, location: "foo_ui_columns", loaded at 00E70000h - 00EC6000h
Address: 77D39056h, location: "USER32", loaded at 77D30000h - 77DC0000h
Symbol: "SetScrollInfo" (+00000000h)
Address: 00E93883h, location: "foo_ui_columns", loaded at 00E70000h - 00EC6000h
Address: 00EA7760h, location: "foo_ui_columns", loaded at 00E70000h - 00EC6000h
Address: 00E8E22Bh, location: "foo_ui_columns", loaded at 00E70000h - 00EC6000h
Address: 00E70000h, location: "foo_ui_columns", loaded at 00E70000h - 00EC6000h
Address: 00EAF038h, location: "foo_ui_columns", loaded at 00E70000h - 00EC6000h
Address: 00EAF038h, location: "foo_ui_columns", loaded at 00E70000h - 00EC6000h
Address: 004A8CDCh, location: "foobar2000", loaded at 00400000h - 004B8000h
Address: 00EA7351h, location: "foo_ui_columns", loaded at 00E70000h - 00EC6000h
Address: 00E7F619h, location: "foo_ui_columns", loaded at 00E70000h - 00EC6000h
Address: 00E8FA04h, location: "foo_ui_columns", loaded at 00E70000h - 00EC6000h
Address: 77D3B3F9h, location: "USER32", loaded at 77D30000h - 77DC0000h
Symbol: "DefWindowProcW" (+000000BDh)
Address: 77D38BD9h, location: "USER32", loaded at 77D30000h - 77DC0000h
Symbol: "GetWindowThreadProcessId" (+00000159h)
Address: 77D90088h, location: "USER32", loaded at 77D30000h - 77DC0000h
Address: 77D3B3CCh, location: "USER32", loaded at 77D30000h - 77DC0000h
Symbol: "DefWindowProcW" (+00000090h)
Address: 77D3B3A7h, location: "USER32", loaded at 77D30000h - 77DC0000h
Symbol: "DefWindowProcW" (+0000006Bh)
Address: 77D388A6h, location: "USER32", loaded at 77D30000h - 77DC0000h
Symbol: "GetWindowLongW" (+00000000h)
Address: 77D388A6h, location: "USER32", loaded at 77D30000h - 77DC0000h
Symbol: "GetWindowLongW" (+00000000h)
Address: 77D3C06Bh, location: "USER32", loaded at 77D30000h - 77DC0000h
Symbol: "SetWindowPos" (+00000050h)
Address: 77D3C138h, location: "USER32", loaded at 77D30000h - 77DC0000h
Symbol: "GetClassLongW" (+00000027h)
Address: 77D3C141h, location: "USER32", loaded at 77D30000h - 77DC0000h
Symbol: "GetClassLongW" (+00000030h)
Address: 77D388A6h, location: "USER32", loaded at 77D30000h - 77DC0000h
Symbol: "GetWindowLongW" (+00000000h)
Address: 77D3D3DEh, location: "USER32", loaded at 77D30000h - 77DC0000h
Symbol: "EnumDisplayMonitors" (+0000047Bh)
Address: 77D3D453h, location: "USER32", loaded at 77D30000h - 77DC0000h
Symbol: "SetPropW" (+0000002Ah)
Address: 77D38B26h, location: "USER32", loaded at 77D30000h - 77DC0000h
Symbol: "GetWindowThreadProcessId" (+000000A6h)
Address: 00E9D290h, location: "foo_ui_columns", loaded at 00E70000h - 00EC6000h
Address: 77D388D1h, location: "USER32", loaded at 77D30000h - 77DC0000h
Symbol: "GetWindowLongW" (+0000002Bh)
Address: 77D388DAh, location: "USER32", loaded at 77D30000h - 77DC0000h
Symbol: "GetWindowLongW" (+00000034h)
Address: 00E9D290h, location: "foo_ui_columns", loaded at 00E70000h - 00EC6000h
Address: 00E8F79Ch, location: "foo_ui_columns", loaded at 00E70000h - 00EC6000h
Address: 00EA74F3h, location: "foo_ui_columns", loaded at 00E70000h - 00EC6000h
Address: 00E9D380h, location: "foo_ui_columns", loaded at 00E70000h - 00EC6000h
Address: 00E9D290h, location: "foo_ui_columns", loaded at 00E70000h - 00EC6000h
Address: 77D38734h, location: "USER32", loaded at 77D30000h - 77DC0000h
Symbol: "GetDC" (+0000006Dh)
Address: 00E9D290h, location: "foo_ui_columns", loaded at 00E70000h - 00EC6000h
Address: 00E9D290h, location: "foo_ui_columns", loaded at 00E70000h - 00EC6000h
Address: 77D3D05Bh, location: "USER32", loaded at 77D30000h - 77DC0000h
Symbol: "EnumDisplayMonitors" (+000000F8h)
Address: 00E9D290h, location: "foo_ui_columns", loaded at 00E70000h - 00EC6000h
Address: 77D60467h, location: "USER32", loaded at 77D30000h - 77DC0000h
Symbol: "DeregisterShellHookWindow" (+000001CFh)
Address: 77D38830h, location: "USER32", loaded at 77D30000h - 77DC0000h
Symbol: "GetDC" (+00000169h)
Address: 77D3B4C0h, location: "USER32", loaded at 77D30000h - 77DC0000h
Symbol: "DefWindowProcW" (+00000184h)
Address: 00E9D290h, location: "foo_ui_columns", loaded at 00E70000h - 00EC6000h
Address: 77D38BD9h, location: "USER32", loaded at 77D30000h - 77DC0000h
Symbol: "GetWindowThreadProcessId" (+00000159h)
Address: 77D60467h, location: "USER32", loaded at 77D30000h - 77DC0000h
Symbol: "DeregisterShellHookWindow" (+000001CFh)
Address: 77D3B4D0h, location: "USER32", loaded at 77D30000h - 77DC0000h
Symbol: "DefWindowProcW" (+00000194h)
Address: 77D3FD29h, location: "USER32", loaded at 77D30000h - 77DC0000h
Symbol: "UserClientDllInitialize" (+000007F1h)
Address: 00E9D290h, location: "foo_ui_columns", loaded at 00E70000h - 00EC6000h
Address: 7C90EAE3h, location: "ntdll", loaded at 7C900000h - 7C9B2000h
Symbol: "KiUserCallbackDispatcher" (+00000013h)
Address: 00E70000h, location: "foo_ui_columns", loaded at 00E70000h - 00EC6000h

Version info: 
foobar2000 v0.9 beta 4
UNICODE

Additional info:
RAR reader 1.1  (foo_unpack)
HTTP Reader 1.1  (foo_input_std)
ALAC decoder 1.0  (foo_alac)
FLAC decoder 1.1.0  (foo_input_std)
WavPack Decoder 2.1  (foo_input_std)
DXi MIDI synthesizer host 1.2.3  (foo_midi)
Columns UI 0.1.3 pre-alpha 11v3  (foo_ui_columns)
Audio CD Writer 1.4  (foo_burninate)
Masstagger 1.5  (foo_masstag)
Album List 3.0.0  (foo_albumlist)
Album list panel 0.2.0 beta  (foo_uie_albumlist)
7-Zip reader 1.10  (foo_unpack)
MPEG-4 AAC decoder 2.1  (foo_input_std)
Database search 1.3.0wip  (foo_dbsearch)
ZIP/GZIP reader 1.0  (foo_unpack)
Utilities 0.5 alpha 16  (foo_utils)
Monkey's Audio decoder 1.2  (foo_input_std)
Converter 1.0  (foo_converter)
CD Audio decoder 1.1  (foo_cdda)
Crossfeed 0.70  (foo_dsp_crossfeed)
DUMB module decoder 0.9.6  (foo_dumb)
CLI encoder 0.4  (foo_converter)
AC3 decoder 0.6  (foo_ac3)
Standard DSP array 1.0  (foo_dsp_std)
ReplayGain Scanner 1.3  (foo_rgscan)
Standard input array 1.0  (foo_input_std)
Default User Interface 0.9acc  (foo_ui_std)

DotNoir

May 30 2005, 08:40

Oh, i get "Album list needs Media Library to be configured first." when i start foobar2000 and have album list panel enabled. Media library is enabled and album list panel populates when i do a refresh on it...

NEMO7538

May 30 2005, 08:52

QUOTE(DotNoir @ May 30 2005, 06:26 AM)

I'm not completely sure if you're after having multiple playlists displaying different material or (like I undersood from your post) that you'd be able to have albumlist on top half and playlist on lower half of the window. The second option is already possible -> http://tritonusrock.com/columns.png

I was speaking of multiple "playlist views" displaying different data (several instances base on different sources).

NEMO7538

May 30 2005, 08:53

QUOTE(filon @ May 30 2005, 06:24 AM)

Looks that right now multiple playlist views are bound to the same data and our request should be a matter of per-view data (active playlist, view configuration).
I see this as one "master" playlist view controlled by playlist switcher etc. and possibly muliple other playlist views for selected playlists.

For example, album list would fill "Database View" playlist which will have it's own playlist view, and apart from that there would be main playlist view where some of the tracks from "Database View" could be copied etc.

Yes .... apart that I don't see the point with master or not.

We shoud just have the capability to associate distinct sources (eg switcher, album list, history, enqueued files, etc. ...) for the content of several instances of the playlist view.

duncan

May 30 2005, 10:28

QUOTE(musicmusic @ May 29 2005, 07:21 PM)

QUOTE(duncan @ May 29 2005, 12:53 AM)

For version Pre-Alpha 11v3, in 'Display > Columns UI', under the 'Status bar / Systray' tab, the option 'Popup now playing balloon top' has no checkbox or anything, it's just the text. Am I missing something here, or is that a bug

?

It does here. Im not sure why it wouldn't (screenshot?)

See image:
user posted image

It looks a bit odd to me.

anza

May 30 2005, 10:32

QUOTE(duncan @ May 30 2005, 06:28 PM)

QUOTE(musicmusic @ May 29 2005, 07:21 PM)

QUOTE(duncan @ May 29 2005, 12:53 AM)

For version Pre-Alpha 11v3, in 'Display > Columns UI', under the 'Status bar / Systray' tab, the option 'Popup now playing balloon top' has no checkbox or anything, it's just the text. Am I missing something here, or is that a bug

?

It does here. Im not sure why it wouldn't (screenshot?)

See image:
user posted image

It looks a bit odd to me.

That part just tells what the selected action in "Action to perform when doubleclicking on status bar" does.

upNorth

May 30 2005, 10:35

Questions/request:
1) Could you please lock the vertical size of the "seekbar" when used in the sidebar? I can't seem to find a way of doing it at the moment...

2) As far as I can gather, it's currently not possible put toolbars side by side in the sidebar. Is this a planned feature, and if not, would you consider adding it? It's not a bit deal, but I would just like to put the "seekbar" and "playback order" side by side, as neither need the full width by itself. The reason I ask, is that I have put all menu items in the sidebar so they can easilly be toggled out of the way.

3) Is it possible for you to add something to enable us to mark playlists that are locked/autoplaylists, or does this have to be added "in the core"? What I have in mind is something like %_islocked% ond/or %_isautoplaylist% for use in the titleformatting of playlist titles.

The Link

May 30 2005, 10:52

Is the sidebar still there because of backwards compatability reasons or is there sth. I missed?

@upNorth:
ad 1) I think you can lock the size if you use the new layout.

ad 2) Is possible using layout here.

upNorth

May 30 2005, 11:04

QUOTE(The Link @ May 30 2005, 05:52 PM)

Is the sidebar still there because of backwards compatability reasons or is there sth. I missed?

Maybe you missed something, see below.

QUOTE(The Link @ May 30 2005, 05:52 PM)

@upNorth:
ad 1) I think you can lock the size if you use the new layout.

ad 2) Is possible using layout here.

Ok, but just to make sure. Does this also apply to the sidebar itself? The one to the left with an optional toggle all along the left side. Because as far as I can see, it doesn't, and I'm really attached to that toggle feature (a feature seen in the Opera browser, that I requested a while back).

I haven't tested the new "Layout" all that much yet, as there isn't alot of panels to play with at the moment, and I also feel kind of limited with my 1024x768 resolution as I like to have a lot of vertical room for my playlist. Hence, I'm not quite sure I'll be adding panels above the playlist.

The Link

May 30 2005, 11:12

QUOTE(upNorth @ May 30 2005, 06:04 PM)

Does this also apply to the sidebar itself?

I'm just referring to the new layout. I never used this toggle feature of the old sidebar so I wasn't aware of any additional use. For me the new layout replaced the sidebar.

QUOTE(upNorth @ May 30 2005, 06:04 PM)

... Hence, I'm not quite sure I'll be adding panels above the playlist.

I'm also not using any panels above the playlist. The new layout just makes configuring more flexible and easier imho (no more problems if you have hidden all the captions, more possibilities to arrange toolbars/panels).

The Link

edit: uhh...i lied! I have the spectrum analyzer above my playlist! I just got that quickly used to the new layout system that I forgot about that.

But it's not taking as much additional space as it would have been using the old system.

duncan

May 30 2005, 12:57

QUOTE(anza @ May 30 2005, 06:32 PM)

That part just tells what the selected action in "Action to perform when doubleclicking on status bar" does.

I don't mean the dropdown, but the text below that.

anza

May 30 2005, 13:01

QUOTE(duncan @ May 30 2005, 08:57 PM)

QUOTE(anza @ May 30 2005, 06:32 PM)

That part just tells what the selected action in "Action to perform when doubleclicking on status bar" does.

I don't mean the dropdown, but the text below that.

Yea I know. And that's just the part I meant.

Haggis

May 30 2005, 13:14

Using Album List Panel 0.2.0 beta, Columns UI 0.1.3 pre-alpha 11v3, and foobar2000 0.9b4, I can't seem to disable the Album List Panel option "Auto send to playlist". If I remove the check, it comes back if I close the window or even if I click "Save All".

mimeryme

May 30 2005, 18:02

Is there a way to have the Playlist Switcher & Playlist View to always adopt the current Windows color scheme or at least to reset it automatically instead of manually selecting the colors to match?

desigrid

May 30 2005, 21:33

AFAIK, currently no. But it's in the works. See quote below.

QUOTE(musicmusic @ May 26 2005, 01:02 PM)

QUOTE(janjan @ May 26 2005, 01:45 PM)

And yet another thing:
Perhaps you could add a 'Reset Colors'-Button under which resets the colors to the current systems defaults. That could be usefull for people who often change their windows skins.

I would rather add a 'Use custom colours' checkbox like album list panel, in fact I already thought about that, probably will be implemented soon, though it will still be possible to override them in the colour string (which i will rename to the style string).

jahty

May 30 2005, 21:46

When I have the album list sitting above the playlist view, the playlist tabs sit above the album list. I don't know if this is by design or not, but it makes a lot more sense to me if the tabs always sit above the playlist view.

user posted image

jahty

May 30 2005, 21:56

Another minor thing I'm seeing with the album list panel beta is that the "|" character is displayed as an underscore in the album list. It displays properly in 0.1.8.

sPeziFisH

May 31 2005, 02:51

The new ColumnUI for the 0.9-line is great

musicmusic, could you implement a color-setup for the highlighting of the playlist, of which the current song is played - of course maybe more than one playlist can contain this song, but at least it was started at a certain playlist (if so)
Sometimes I get lost when I want to switch to the right playlist to have a look which song will be next..
AFAIK it's not possible right now and playlist-switcher is part of ColumnUI...
The possibilty to add a music-note behind the playlist-name would also be great but I guess it wouldl require further changes of the SDK to format playlist-names, like

CODE

$if(%is_playing%,'new - ' ...whatever.. ,new)

anza

May 31 2005, 03:09

QUOTE(sPeziFisH @ May 31 2005, 10:51 AM)

Sometimes I get lost when I want to switch to the right playlist to have a look which song will be next..

Set a keyboard shortcut to Main|Playlist|Highlight playing item. It changes the playlist to the currently playing one and -of course- highlights and sets focus to the currently playing file.

sPeziFisH

May 31 2005, 03:32

Thanks anza, interesting that I lost the awarenes of this menu-item.
...it's when searching too much at preferences, rarely use adding of new keyboard-shortcuts (sure I got some) or rarely using playlistmenu (has also killed this item some time ago while cleaning and sorting the playlist-menu

) - everything's fine right now, thx

musicmusic

May 31 2005, 04:13

QUOTE(desigrid @ May 30 2005, 04:31 AM)

QUOTE(musicmusic @ May 29 2005, 03:21 PM)

Album list panel 0.2 beta for columns 0.1.3 pre-alpha 11 and 0.9 beta 4 is out:
Download

Great work on columns 0.1.3 musicmusic. Love the customisability of the new layout feature.

Been playing around with the beta album list panel and have a few comments:

1. Tagz without the % no longer work as expected e.g. artist used by itself would previously separate multiple artists fields into it's own tree index. I recall reading someone posting elsewhere that the album list component that comes with fb2k 0.9 beta acts the same way. Does the new core not support this feature anymore?

Well, album list panel is based upon the standard album list, that's the reason they both have some same behaviours.

QUOTE(desigrid @ May 30 2005, 04:31 AM)

2. I noticed an option to "Select added items." I believe this feature will solve a problem interfacing with future versions of the album art and track info panels (whether intended or not), whereby selecting a tree item would load the appropriate info in the panels. However, would it be possible to add another option to select first item only?

You can just tab to the playlist and press down for that.

QUOTE(desigrid @ May 30 2005, 04:31 AM)

3. Would it also be possible to add an option to focus the auto-sent playlist if it currently isn't, when an already selected index on the album list index is clicked on? As it is now, if I select an artist with the album list panel, the associated files will be auto-sent to a playlist called Library View as expected. If I then activate another playlist and subsequently click on the currently selected artist on the album list panel, the auto-sent playlist remains unfocused.

I'll look into it.

QUOTE(filon @ May 30 2005, 02:34 PM)

QUOTE(musicmusic @ May 29 2005, 03:21 PM)

Album list panel 0.2 beta for columns 0.1.3 pre-alpha 11 and 0.9 beta 4 is out:
Download

When media library is empty, album list crashes.

CODE

Illegal operation:
Code: C0000005h, flags: 00000000h, address: 00E92A79h
Access violation, operation: read, address: 00000000h
Call path:
WinMain=>app_mainloop=>album_list_panel_refresh_tree=>album_list_panel_refresh_tree=>album_list_panel_refresh_tree=>album_list_panel_refresh_tree=>album_list_panel_refresh_tree=>album_list_panel_refresh_tree=>album_list_panel_refresh_tree=>album_list_panel_refresh_tree=>album_list_panel_refresh_tree=>album_list_panel_refresh_tree=>album_list_panel_refresh_tree=>album_list_panel_refresh_tree=>album_list_panel_refresh_tree
This is the first crash logged by this instance.
Code bytes (00E92A79h):
00E92A39h:  8B 7C 24 0C 57 8B F1 E8 CB FB FF FF 8B 46 04 8B
00E92A49h:  04 B8 5F C6 40 21 00 5E C2 10 00 56 57 8B 7C 24
00E92A59h:  0C 57 8B F1 E8 AE FB FF FF 8B 46 04 8B 04 B8 5F
00E92A69h:  C6 40 21 00 5E C2 08 00 8B 51 04 8B 44 24 04 56
00E92A79h:  8B 34 82 80 7E 20 00 75 06 50 E8 FF F9 FF FF 8D
00E92A89h:  46 14 5E C2 04 00 55 8B EC 53 56 57 FF 75 08 E8
00E92A99h:  D4 FF FF FF 33 F6 B9 BC 87 EB 00 8B F8 89 75 08
00E92AA9h:  E8 0B F7 FD FF 8B D8 3B DE 76 1B 8B 07 56 8B CF
Stack (0011FA30h):
0011FA10h:  00000001 0011FC4C 7C90EB94 00000001
0011FA20h:  F52A457C 0000003B 00000083 006A54B0
0011FA30h:  00B453C0 00E92AEA 00000000 46010000
0011FA40h:  00B453C0 0011FA74 77D39056 00E9389E
0011FA50h:  00000000 00B453C0 00B453C0 77D39056
0011FA60h:  FFFFFFFF 00E93883 0011FB08 00EA7760
0011FA70h:  FFFFFFFF 0011FB14 00E8E22B 0011FB90
0011FA80h:  00000000 00B453C0 00000000 00000000
0011FA90h:  00E70000 0011FBD4 40000400 00EAF038
0011FAA0h:  00F74C60 00EAF038 0000001C 00000004
0011FAB0h:  00000000 00000000 00000000 00000000
0011FAC0h:  00000000 0000001C 00000007 00000000
0011FAD0h:  FFFFFFFF 00000000 00000000 00000000
0011FAE0h:  00000000 00000000 88527726 00000000
0011FAF0h:  00000000 00000000 004A8CDC 00000000
0011FB00h:  46010000 000000F1 0011FC8C 00EA7351
0011FB10h:  00000000 00000001 00E7F619 00000001
0011FB20h:  00000100 00000000 00B453C0 0011FC98
0011FB30h:  00000000 FFFFFFF5 00000000 00000000
0011FB40h:  00000000 00000190 01000000 00000000
Registers:
EAX: 00000000, EBX: 77D39056, ECX: 00EB882C, EDX: 00000000
ESI: 00B453C0, EDI: 46010000, EBP: 0011FA74, ESP: 0011FA30
Crash location: "foo_ui_columns", loaded at 00E70000h - 00EC6000h

Loaded modules:
foobar2000                       loaded at 00400000h - 004B8000h
ntdll                            loaded at 7C900000h - 7C9B2000h
kernel32                         loaded at 7C800000h - 7C8FB000h
COMCTL32                         loaded at 773C0000h - 774C2000h
msvcrt                           loaded at 77C00000h - 77C58000h
ADVAPI32                         loaded at 77DC0000h - 77E6C000h
RPCRT4                           loaded at 77E70000h - 77F01000h
GDI32                            loaded at 77F10000h - 77F56000h
USER32                           loaded at 77D30000h - 77DC0000h
SHLWAPI                          loaded at 77F60000h - 77FD6000h
DSOUND                           loaded at 73EA0000h - 73EFC000h
ole32                            loaded at 774D0000h - 7760D000h
WINMM                            loaded at 76B20000h - 76B4E000h
VERSION                          loaded at 77BF0000h - 77BF8000h
SHELL32                          loaded at 7C9C0000h - 7D1DB000h
shared                           loaded at 10000000h - 10010000h
comdlg32                         loaded at 76380000h - 763C9000h
foo_ac3                          loaded at 00A20000h - 00A35000h
foo_alac                         loaded at 003F0000h - 003F9000h
foo_albumlist                    loaded at 00A40000h - 00A54000h
foo_autoplaylist                 loaded at 00A60000h - 00A72000h
foo_benchmark                    loaded at 00A80000h - 00A91000h
foo_burninate                    loaded at 00AA0000h - 00AB4000h
foo_cdda                         loaded at 00AC0000h - 00AD3000h
foo_converter                    loaded at 00AE0000h - 00B02000h
foo_dbsearch                     loaded at 00B10000h - 00B38000h
MSVCR71                          loaded at 7C340000h - 7C396000h
foo_dsp_crossfeed                loaded at 00B50000h - 00B56000h
foo_dsp_std                      loaded at 00B60000h - 00B78000h
foo_dumb                         loaded at 00B80000h - 00BB4000h
foo_input_std                    loaded at 00BC0000h - 00CF4000h
MSACM32                          loaded at 77BD0000h - 77BE5000h
WS2_32                           loaded at 71A50000h - 71A67000h
WS2HELP                          loaded at 71A40000h - 71A48000h
foo_masstag                      loaded at 00D10000h - 00D38000h
foo_midi                         loaded at 00D40000h - 00DF7000h
WINSPOOL                         loaded at 72F90000h - 72FB6000h
OLEAUT32                         loaded at 77110000h - 7719C000h
foo_playlistgen                  loaded at 00E20000h - 00E30000h
foo_rgscan                       loaded at 00E30000h - 00E47000h
foo_uie_albumlist                loaded at 00E50000h - 00E66000h
foo_ui_columns                   loaded at 00E70000h - 00EC6000h
foo_ui_std                       loaded at 00ED0000h - 00EF4000h
OLEACC                           loaded at 74C30000h - 74C5C000h
MSVCP60                          loaded at 76050000h - 760B5000h
foo_unpack                       loaded at 00F10000h - 00F45000h
foo_utils                        loaded at 00F50000h - 00F65000h
uxtheme                          loaded at 5B1D0000h - 5B208000h
imagehlp                         loaded at 76C80000h - 76CA8000h
DBGHELP                          loaded at 59BC0000h - 59C61000h

Stack dump analysis:
Address: 00E92AEAh, location: "foo_ui_columns", loaded at 00E70000h - 00EC6000h
Address: 77D39056h, location: "USER32", loaded at 77D30000h - 77DC0000h
Symbol: "SetScrollInfo" (+00000000h)
Address: 00E9389Eh, location: "foo_ui_columns", loaded at 00E70000h - 00EC6000h
Address: 77D39056h, location: "USER32", loaded at 77D30000h - 77DC0000h
Symbol: "SetScrollInfo" (+00000000h)
Address: 00E93883h, location: "foo_ui_columns", loaded at 00E70000h - 00EC6000h
Address: 00EA7760h, location: "foo_ui_columns", loaded at 00E70000h - 00EC6000h
Address: 00E8E22Bh, location: "foo_ui_columns", loaded at 00E70000h - 00EC6000h
Address: 00E70000h, location: "foo_ui_columns", loaded at 00E70000h - 00EC6000h
Address: 00EAF038h, location: "foo_ui_columns", loaded at 00E70000h - 00EC6000h
Address: 00EAF038h, location: "foo_ui_columns", loaded at 00E70000h - 00EC6000h
Address: 004A8CDCh, location: "foobar2000", loaded at 00400000h - 004B8000h
Address: 00EA7351h, location: "foo_ui_columns", loaded at 00E70000h - 00EC6000h
Address: 00E7F619h, location: "foo_ui_columns", loaded at 00E70000h - 00EC6000h
Address: 00E8FA04h, location: "foo_ui_columns", loaded at 00E70000h - 00EC6000h
Address: 77D3B3F9h, location: "USER32", loaded at 77D30000h - 77DC0000h
Symbol: "DefWindowProcW" (+000000BDh)
Address: 77D38BD9h, location: "USER32", loaded at 77D30000h - 77DC0000h
Symbol: "GetWindowThreadProcessId" (+00000159h)
Address: 77D90088h, location: "USER32", loaded at 77D30000h - 77DC0000h
Address: 77D3B3CCh, location: "USER32", loaded at 77D30000h - 77DC0000h
Symbol: "DefWindowProcW" (+00000090h)
Address: 77D3B3A7h, location: "USER32", loaded at 77D30000h - 77DC0000h
Symbol: "DefWindowProcW" (+0000006Bh)
Address: 77D388A6h, location: "USER32", loaded at 77D30000h - 77DC0000h
Symbol: "GetWindowLongW" (+00000000h)
Address: 77D388A6h, location: "USER32", loaded at 77D30000h - 77DC0000h
Symbol: "GetWindowLongW" (+00000000h)
Address: 77D3C06Bh, location: "USER32", loaded at 77D30000h - 77DC0000h
Symbol: "SetWindowPos" (+00000050h)
Address: 77D3C138h, location: "USER32", loaded at 77D30000h - 77DC0000h
Symbol: "GetClassLongW" (+00000027h)
Address: 77D3C141h, location: "USER32", loaded at 77D30000h - 77DC0000h
Symbol: "GetClassLongW" (+00000030h)
Address: 77D388A6h, location: "USER32", loaded at 77D30000h - 77DC0000h
Symbol: "GetWindowLongW" (+00000000h)
Address: 77D3D3DEh, location: "USER32", loaded at 77D30000h - 77DC0000h
Symbol: "EnumDisplayMonitors" (+0000047Bh)
Address: 77D3D453h, location: "USER32", loaded at 77D30000h - 77DC0000h
Symbol: "SetPropW" (+0000002Ah)
Address: 77D38B26h, location: "USER32", loaded at 77D30000h - 77DC0000h
Symbol: "GetWindowThreadProcessId" (+000000A6h)
Address: 00E9D290h, location: "foo_ui_columns", loaded at 00E70000h - 00EC6000h
Address: 77D388D1h, location: "USER32", loaded at 77D30000h - 77DC0000h
Symbol: "GetWindowLongW" (+0000002Bh)
Address: 77D388DAh, location: "USER32", loaded at 77D30000h - 77DC0000h
Symbol: "GetWindowLongW" (+00000034h)
Address: 00E9D290h, location: "foo_ui_columns", loaded at 00E70000h - 00EC6000h
Address: 00E8F79Ch, location: "foo_ui_columns", loaded at 00E70000h - 00EC6000h
Address: 00EA74F3h, location: "foo_ui_columns", loaded at 00E70000h - 00EC6000h
Address: 00E9D380h, location: "foo_ui_columns", loaded at 00E70000h - 00EC6000h
Address: 00E9D290h, location: "foo_ui_columns", loaded at 00E70000h - 00EC6000h
Address: 77D38734h, location: "USER32", loaded at 77D30000h - 77DC0000h
Symbol: "GetDC" (+0000006Dh)
Address: 00E9D290h, location: "foo_ui_columns", loaded at 00E70000h - 00EC6000h
Address: 00E9D290h, location: "foo_ui_columns", loaded at 00E70000h - 00EC6000h
Address: 77D3D05Bh, location: "USER32", loaded at 77D30000h - 77DC0000h
Symbol: "EnumDisplayMonitors" (+000000F8h)
Address: 00E9D290h, location: "foo_ui_columns", loaded at 00E70000h - 00EC6000h
Address: 77D60467h, location: "USER32", loaded at 77D30000h - 77DC0000h
Symbol: "DeregisterShellHookWindow" (+000001CFh)
Address: 77D38830h, location: "USER32", loaded at 77D30000h - 77DC0000h
Symbol: "GetDC" (+00000169h)
Address: 77D3B4C0h, location: "USER32", loaded at 77D30000h - 77DC0000h
Symbol: "DefWindowProcW" (+00000184h)
Address: 00E9D290h, location: "foo_ui_columns", loaded at 00E70000h - 00EC6000h
Address: 77D38BD9h, location: "USER32", loaded at 77D30000h - 77DC0000h
Symbol: "GetWindowThreadProcessId" (+00000159h)
Address: 77D60467h, location: "USER32", loaded at 77D30000h - 77DC0000h
Symbol: "DeregisterShellHookWindow" (+000001CFh)
Address: 77D3B4D0h, location: "USER32", loaded at 77D30000h - 77DC0000h
Symbol: "DefWindowProcW" (+00000194h)
Address: 77D3FD29h, location: "USER32", loaded at 77D30000h - 77DC0000h
Symbol: "UserClientDllInitialize" (+000007F1h)
Address: 00E9D290h, location: "foo_ui_columns", loaded at 00E70000h - 00EC6000h
Address: 7C90EAE3h, location: "ntdll", loaded at 7C900000h - 7C9B2000h
Symbol: "KiUserCallbackDispatcher" (+00000013h)
Address: 00E70000h, location: "foo_ui_columns", loaded at 00E70000h - 00EC6000h

Version info: 
foobar2000 v0.9 beta 4
UNICODE

Additional info:
RAR reader 1.1  (foo_unpack)
HTTP Reader 1.1  (foo_input_std)
ALAC decoder 1.0  (foo_alac)
FLAC decoder 1.1.0  (foo_input_std)
WavPack Decoder 2.1  (foo_input_std)
DXi MIDI synthesizer host 1.2.3  (foo_midi)
Columns UI 0.1.3 pre-alpha 11v3  (foo_ui_columns)
Audio CD Writer 1.4  (foo_burninate)
Masstagger 1.5  (foo_masstag)
Album List 3.0.0  (foo_albumlist)
Album list panel 0.2.0 beta  (foo_uie_albumlist)
7-Zip reader 1.10  (foo_unpack)
MPEG-4 AAC decoder 2.1  (foo_input_std)
Database search 1.3.0wip  (foo_dbsearch)
ZIP/GZIP reader 1.0  (foo_unpack)
Utilities 0.5 alpha 16  (foo_utils)
Monkey's Audio decoder 1.2  (foo_input_std)
Converter 1.0  (foo_converter)
CD Audio decoder 1.1  (foo_cdda)
Crossfeed 0.70  (foo_dsp_crossfeed)
DUMB module decoder 0.9.6  (foo_dumb)
CLI encoder 0.4  (foo_converter)
AC3 decoder 0.6  (foo_ac3)
Standard DSP array 1.0  (foo_dsp_std)
ReplayGain Scanner 1.3  (foo_rgscan)
Standard input array 1.0  (foo_input_std)
Default User Interface 0.9acc  (foo_ui_std)

This is wierd.
It crashed here:

CODE

PUBLIC ?get_columns_mask@playlist_view_cache@@QAEABVbit_array@@I@Z; playlist_view_cache::get_columns_mask
; Function compile flags: /Ogsy
; COMDAT ?get_columns_mask@playlist_view_cache@@QAEABVbit_array@@I@Z
_TEXT SEGMENT
_playlist$ = 8 ; size = 4
?get_columns_mask@playlist_view_cache@@QAEABVbit_array@@I@Z PROC NEAR; playlist_view_cache::get_columns_mask, COMDAT
; _this$ = ecx

; 422 : playlist_cache * p_cache = get_item(playlist);

00000 8b 51 04 mov edx, DWORD PTR [ecx+4]
00003 8b 44 24 04 mov eax, DWORD PTR _playlist$[esp-4]
00007 56 push esi
00008 8b 34 82 mov esi, DWORD PTR [edx+eax*4] <------there

I guess the malloc failed when allocating the ptr array in the ptr_list_t, which playlist_view_cache inherits from. Was the active playlist the first one?

And, refresh_tree() shouldn't really recursively call itself. It is only called in four places: WM_CREATE, WM_COMMAND, WM_CONTEXTMENU and the panel menu. (And, of all the possible numbers, it happened 13 times for you..)

The rest of the stack is columns creating a playlist window. Which just adds to the confusion, rather than giving much indication as to wtf is going on.

Did you have multiple instances of albumlist panel open? And can you reproduce this? (I haven't been able to, even with an empty media library) And what exactly did you do when this happened?

Thanks.

musicmusic

May 31 2005, 04:13

part 2.....

QUOTE(DotNoir @ May 30 2005, 02:40 PM)

Oh, i get "Album list needs Media Library to be configured first." when i start foobar2000 and have album list panel enabled. Media library is enabled and album list panel populates when i do a refresh on it...

Did you not update foobar to beta 4 ?

QUOTE(upNorth @ May 30 2005, 04:35 PM)

Questions/request:
1) Could you please lock the vertical size of the "seekbar" when used in the sidebar? I can't seem to find a way of doing it at the moment...

I guess I could/should set a minimum height for it.

QUOTE(upNorth @ May 30 2005, 04:35 PM)

3) Is it possible for you to add something to enable us to mark playlists that are locked/autoplaylists, or does this have to be added "in the core"? What I have in mind is something like %_islocked% ond/or %_isautoplaylist% for use in the titleformatting of playlist titles.

Yes, two ways of indicating it I thought of were a lock icon in the playlist switcher, or in the status bar. Im intersted on any opinion on this.

QUOTE(The Link @ May 30 2005, 04:52 PM)

Is the sidebar still there because of backwards compatability reasons or is there sth. I missed?

It still there until all its features are added/capable from the layout. Also the playlist tabs needs to be turned into a panel, so the layout isn't always contained within them.

QUOTE(Haggis @ May 30 2005, 07:14 PM)

Using Album List Panel 0.2.0 beta, Columns UI 0.1.3 pre-alpha 11v3, and foobar2000 0.9b4, I can't seem to disable the Album List Panel option "Auto send to playlist". If I remove the check, it comes back if I close the window or even if I click "Save All".

Thanks, fixed for next version.

QUOTE(jahty @ May 31 2005, 03:46 AM)

When I have the album list sitting above the playlist view, the playlist tabs sit above the album list. I don't know if this is by design or not, but it makes a lot more sense to me if the tabs always sit above the playlist view.

user posted image

See above.

QUOTE(jahty @ May 31 2005, 03:56 AM)

Another minor thing I'm seeing with the album list panel beta is that the "|" character is displayed as an underscore in the album list. It displays properly in 0.1.8.

I guess this is because it is used as a separator; and the way the media library is process to create the tree changed a lot in album list 3.0.0 (and is far faster). Does this happen in normal album list?

QUOTE(sPeziFisH @ May 31 2005, 08:51 AM)

The new ColumnUI for the 0.9-line is great

musicmusic, could you implement a color-setup for the highlighting of the playlist, of which the current song is played - of course maybe more than one playlist can contain this song, but at least it was started at a certain playlist (if so)

You mean in the default config? I will do something about that.

QUOTE(sPeziFisH @ May 31 2005, 08:51 AM)

The possibilty to add a music-note behind the playlist-name would also be great but I guess it wouldl require further changes of the SDK to format playlist-names, like

CODE

$if(%is_playing%,'new - ' ...whatever.. ,new)

Well, it is possible as long as it is possible to detect when the playing playlist changes.

DotNoir

May 31 2005, 04:41

QUOTE(musicmusic @ May 31 2005, 01:13 PM)

part 2.....

QUOTE(DotNoir @ May 30 2005, 02:40 PM)

Oh, i get "Album list needs Media Library to be configured first." when i start foobar2000 and have album list panel enabled. Media library is enabled and album list panel populates when i do a refresh on it...

Did you not update foobar to beta 4 ?

Ewww, I'm stupid.

filon

May 31 2005, 06:29

QUOTE(musicmusic @ May 31 2005, 02:13 AM)

QUOTE(filon @ May 30 2005, 02:34 PM)

QUOTE(musicmusic @ May 29 2005, 03:21 PM)

Album list panel 0.2 beta for columns 0.1.3 pre-alpha 11 and 0.9 beta 4 is out:
Download

When media library is empty, album list crashes.

...

This is wierd.
It crashed here:

CODE

PUBLIC ?get_columns_mask@playlist_view_cache@@QAEABVbit_array@@I@Z; playlist_view_cache::get_columns_mask
; Function compile flags: /Ogsy
; COMDAT ?get_columns_mask@playlist_view_cache@@QAEABVbit_array@@I@Z
_TEXT SEGMENT
_playlist$ = 8 ; size = 4
?get_columns_mask@playlist_view_cache@@QAEABVbit_array@@I@Z PROC NEAR; playlist_view_cache::get_columns_mask, COMDAT
; _this$ = ecx

; 422 : playlist_cache * p_cache = get_item(playlist);

00000 8b 51 04 mov edx, DWORD PTR [ecx+4]
00003 8b 44 24 04 mov eax, DWORD PTR _playlist$[esp-4]
00007 56 push esi
00008 8b 34 82 mov esi, DWORD PTR [edx+eax*4] <------there

I guess the malloc failed when allocating the ptr array in the ptr_list_t, which playlist_view_cache inherits from. Was the active playlist the first one?

And, refresh_tree() shouldn't really recursively call itself. It is only called in four places: WM_CREATE, WM_COMMAND, WM_CONTEXTMENU and the panel menu. (And, of all the possible numbers, it happened 13 times for you..)

The rest of the stack is columns creating a playlist window. Which just adds to the confusion, rather than giving much indication as to wtf is going on.

Did you have multiple instances of albumlist panel open? And can you reproduce this? (I haven't been able to, even with an empty media library) And what exactly did you do when this happened?

Thanks.

Hi!

To reproduce the bug, follow steps
1. remove all entries from media library
2. remove all playlists (including database) (so now, there is only one playlist = "default")
3. restart foobar
4. disable sidebar
5. make following laylout:

Splitter
- Album List
- Columns Playlist

6. Click "Apply" and volia, crash is ready

What's interesting, it only happens when both panels are added. Maybe it's some kind of mutual recursion between album list and playlist panel?

Frank Bicking

May 31 2005, 06:33

I hope this wasn't mentioned before:

Album list panel 0.2.0 beta doesn't remember the "Auto send to playlist" setting, it automatically resets to enabled.

musicmusic

May 31 2005, 07:14

QUOTE(filon @ May 31 2005, 12:29 PM)

QUOTE(musicmusic @ May 31 2005, 02:13 AM)

QUOTE(filon @ May 30 2005, 02:34 PM)

QUOTE(musicmusic @ May 29 2005, 03:21 PM)

Album list panel 0.2 beta for columns 0.1.3 pre-alpha 11 and 0.9 beta 4 is out:
Download

When media library is empty, album list crashes.

...

This is wierd.
It crashed here:

CODE

PUBLIC ?get_columns_mask@playlist_view_cache@@QAEABVbit_array@@I@Z; playlist_view_cache::get_columns_mask
; Function compile flags: /Ogsy
; COMDAT ?get_columns_mask@playlist_view_cache@@QAEABVbit_array@@I@Z
_TEXT SEGMENT
_playlist$ = 8; size = 4
?get_columns_mask@playlist_view_cache@@QAEABVbit_array@@I@Z PROC NEAR; playlist_view_cache::get_columns_mask, COMDAT
; _this$ = ecx

; 422 : playlist_cache * p_cache = get_item(playlist);

00000 8b 51 04 mov edx, DWORD PTR [ecx+4]
00003 8b 44 24 04 mov eax, DWORD PTR _playlist$[esp-4]
00007 56 push esi
00008 8b 34 82 mov esi, DWORD PTR [edx+eax*4] <------there

I guess the malloc failed when allocating the ptr array in the ptr_list_t, which playlist_view_cache inherits from. Was the active playlist the first one?

And, refresh_tree() shouldn't really recursively call itself. It is only called in four places: WM_CREATE, WM_COMMAND, WM_CONTEXTMENU and the panel menu. (And, of all the possible numbers, it happened 13 times for you..)

The rest of the stack is columns creating a playlist window. Which just adds to the confusion, rather than giving much indication as to wtf is going on.

Did you have multiple instances of albumlist panel open? And can you reproduce this? (I haven't been able to, even with an empty media library) And what exactly did you do when this happened?

Thanks.

Hi!

To reproduce the bug, follow steps
1. remove all entries from media library
2. remove all playlists (including database) (so now, there is only one playlist = "default")
3. restart foobar
4. disable sidebar
5. make following laylout:

Splitter
- Album List
- Columns Playlist

6. Click "Apply" and volia, crash is ready

What's interesting, it only happens when both panels are added. Maybe it's some kind of mutual recursion between album list and playlist panel?

Thanks, I managed to reproduce this.

QUOTE(Frank_Bicking @ May 31 2005, 12:33 PM)

I hope this wasn't mentioned before:

Album list panel 0.2.0 beta doesn't remember the "Auto send to playlist" setting, it automatically resets to enabled.

It was

musicmusic

May 31 2005, 07:52

QUOTE(musicmusic @ May 31 2005, 01:14 PM)

QUOTE(filon @ May 31 2005, 12:29 PM)

QUOTE(musicmusic @ May 31 2005, 02:13 AM)

QUOTE(filon @ May 30 2005, 02:34 PM)

QUOTE(musicmusic @ May 29 2005, 03:21 PM)

Album list panel 0.2 beta for columns 0.1.3 pre-alpha 11 and 0.9 beta 4 is out:
Download

When media library is empty, album list crashes.

...

This is wierd.
It crashed here:

CODE

PUBLIC ?get_columns_mask@playlist_view_cache@@QAEABVbit_array@@I@Z; playlist_view_cache::get_columns_mask
; Function compile flags: /Ogsy
; COMDAT ?get_columns_mask@playlist_view_cache@@QAEABVbit_array@@I@Z
_TEXT SEGMENT
_playlist$ = 8; size = 4
?get_columns_mask@playlist_view_cache@@QAEABVbit_array@@I@Z PROC NEAR; playlist_view_cache::get_columns_mask, COMDAT
; _this$ = ecx

; 422 : playlist_cache * p_cache = get_item(playlist);

00000 8b 51 04 mov edx, DWORD PTR [ecx+4]
00003 8b 44 24 04 mov eax, DWORD PTR _playlist$[esp-4]
00007 56 push esi
00008 8b 34 82 mov esi, DWORD PTR [edx+eax*4] <------there

I guess the malloc failed when allocating the ptr array in the ptr_list_t, which playlist_view_cache inherits from. Was the active playlist the first one?

And, refresh_tree() shouldn't really recursively call itself. It is only called in four places: WM_CREATE, WM_COMMAND, WM_CONTEXTMENU and the panel menu. (And, of all the possible numbers, it happened 13 times for you..)

The rest of the stack is columns creating a playlist window. Which just adds to the confusion, rather than giving much indication as to wtf is going on.

Did you have multiple instances of albumlist panel open? And can you reproduce this? (I haven't been able to, even with an empty media library) And what exactly did you do when this happened?

Thanks.

Hi!

To reproduce the bug, follow steps
1. remove all entries from media library
2. remove all playlists (including database) (so now, there is only one playlist = "default")
3. restart foobar
4. disable sidebar
5. make following laylout:

Splitter
- Album List
- Columns Playlist

6. Click "Apply" and volia, crash is ready

What's interesting, it only happens when both panels are added. Maybe it's some kind of mutual recursion between album list and playlist panel?

Thanks, I managed to reproduce this.

I found the cause. When you apply the changes, a new album list window is created. But media library is disabled, so it attempts to show media library prefs page. But when current prefs page is being destroyed, it again tries to apply the changes, and you can see how it enters this loop..

I'll have a fixed build up soon.

jahty

May 31 2005, 10:01

QUOTE(musicmusic @ May 31 2005, 02:13 AM)

QUOTE(jahty @ May 31 2005, 03:56 AM)

Another minor thing I'm seeing with the album list panel beta is that the "|" character is displayed as an underscore in the album list. It displays properly in 0.1.8.

I guess this is because it is used as a separator; and the way the media library is process to create the tree changed a lot in album list 3.0.0 (and is far faster). Does this happen in normal album list?

Ah, yes it does, I'll report it in the 0.9 thread. Thanks.

CJC

May 31 2005, 18:20

QUOTE(musicmusic @ May 29 2005, 06:21 PM)

Album list panel 0.2 beta for columns 0.1.3 pre-alpha 11 and 0.9 beta 4 is out:
Download

Thank you very much sir

sPeziFisH

Jun 1 2005, 02:15

Don't know if this is up to ColumnUI, but I will post here first as I don't want to garbage the 0.9-beta-bug-thread with possible ColumnUI-up-to-s:
when I select some songs, press CTRL and drop them to another playlist the files get sorted alphabetically. I would like to keep the existing order.

upNorth

Jun 1 2005, 10:30

QUOTE(musicmusic @ May 31 2005, 11:13 AM)

QUOTE(upNorth @ May 30 2005, 04:35 PM)

Questions/request:
1) Could you please lock the vertical size of the "seekbar" when used in the sidebar? I can't seem to find a way of doing it at the moment...

I guess I could/should set a minimum height for it.

I was implying a fixed vertical size/height. If you for some reason don't want it to be fixed size (vertically), a max size setting would be great. It's a toolbar, not a panel, and the slider itself is very much fixed size, so I don't really see the usefulness of being able to resize the vertical "padding" below it. It's just annoying when resizing panels, and also looks strange when panels are removed, as you're left with a mostly empty "slider panel" with the slider at the top. I'll stop elaborating now, as it's just a minior detail anyway...

QUOTE(musicmusic @ May 31 2005, 11:13 AM)

QUOTE(upNorth @ May 30 2005, 04:35 PM)

3) Is it possible for you to add something to enable us to mark playlists that are locked/autoplaylists, or does this have to be added "in the core"? What I have in mind is something like %_islocked% ond/or %_isautoplaylist% for use in the titleformatting of playlist titles.

Yes, two ways of indicating it I thought of were a lock icon in the playlist switcher, or in the status bar. Im intersted on any opinion on this.

I would personally prefer a way to make use of it in the formatting code for playlist titles. There is currently no icons in your playlist switcher, so an icon might look out of place. The status bar is too far away for my taste. Besides, foosion's new foo_playlist_manager, already has icons that displays this and more. I don't know how much further he wants to develop it, but a panel version would give us a nice alternative to the standard switcher. I would most likely use them both with foo_uie_tabs, as I use the "title formatting for playlist name" feature extensively, and for that reason, will continue to use the current one. Just do whatever you see fit.

QUOTE(musicmusic @ May 31 2005, 11:13 AM)

QUOTE(The Link @ May 30 2005, 04:52 PM)

Is the sidebar still there because of backwards compatability reasons or is there sth. I missed?

It still there until all its features are added/capable from the layout. Also the playlist tabs needs to be turned into a panel, so the layout isn't always contained within them.

I hope the toggle along the left side will survive this. And with it, the ability to toggle parts in and out/on off.

Just fighting for another belowed feature

musicmusic

Jun 2 2005, 17:19

QUOTE(upNorth @ Jun 1 2005, 04:30 PM)

QUOTE(musicmusic @ May 31 2005, 11:13 AM)

QUOTE(upNorth @ May 30 2005, 04:35 PM)

Questions/request:
1) Could you please lock the vertical size of the "seekbar" when used in the sidebar? I can't seem to find a way of doing it at the moment...

I guess I could/should set a minimum height for it.

I was implying a fixed vertical size/height. If you for some reason don't want it to be fixed size (vertically), a max size setting would be great. It's a toolbar, not a panel, and the slider itself is very much fixed size, so I don't really see the usefulness of being able to resize the vertical "padding" below it. It's just annoying when resizing panels, and also looks strange when panels are removed, as you're left with a mostly empty "slider panel" with the slider at the top. I'll stop elaborating now, as it's just a minior detail anyway...

Well, the fact that resizing most of the standard toolbars vertically is useless (fixed height as you say) is why they have max heights. I did have some reason for not setting one for the seekbar, but I forget what it was. So rather than think about it I'll just change it to have a max height since it makes sense. [edit] yes, i did actually mean max height in the post you quoted..

QUOTE(upNorth @ Jun 1 2005, 04:30 PM)

QUOTE(musicmusic @ May 31 2005, 11:13 AM)

QUOTE(upNorth @ May 30 2005, 04:35 PM)

3) Is it possible for you to add something to enable us to mark playlists that are locked/autoplaylists, or does this have to be added "in the core"? What I have in mind is something like %_islocked% ond/or %_isautoplaylist% for use in the titleformatting of playlist titles.

Yes, two ways of indicating it I thought of were a lock icon in the playlist switcher, or in the status bar. Im intersted on any opinion on this.

I would personally prefer a way to make use of it in the formatting code for playlist titles. There is currently no icons in your playlist switcher, so an icon might look out of place. The status bar is too far away for my taste. Besides, foosion's new foo_playlist_manager, already has icons that displays this and more. I don't know how much further he wants to develop it, but a panel version would give us a nice alternative to the standard switcher. I would most likely use them both with foo_uie_tabs, as I use the "title formatting for playlist name" feature extensively, and for that reason, will continue to use the current one. Just do whatever you see fit.

Well, I will add %_is_locked%, and then you can colour or whatnot those playlists. True, an icon in my playlist switcher would probably look wierd, but I was thinking of more intuitive ways of showing it (it is clear what such an icon means, rather than random colours and whatnot). So I think I will add an icon in the status bar as well.

QUOTE(upNorth @ Jun 1 2005, 04:30 PM)

QUOTE(musicmusic @ May 31 2005, 11:13 AM)

QUOTE(The Link @ May 30 2005, 04:52 PM)

Is the sidebar still there because of backwards compatability reasons or is there sth. I missed?

It still there until all its features are added/capable from the layout. Also the playlist tabs needs to be turned into a panel, so the layout isn't always contained within them.

I hope the toggle along the left side will survive this. And with it, the ability to toggle parts in and out/on off.

Just fighting for another belowed feature

Well, Im not going to remove any features already implemented unless they become redundant

. But, I cant really implement most little features requested like I was at one point, I have enough options at it is and I hate the way they now span across three prefs pages

musicmusic

Jun 2 2005, 17:19

QUOTE(sPeziFisH @ Jun 1 2005, 08:15 AM)

Don't know if this is up to ColumnUI, but I will post here first as I don't want to garbage the 0.9-beta-bug-thread with possible ColumnUI-up-to-s:
when I select some songs, press CTRL and drop them to another playlist the files get sorted alphabetically. I would like to keep the existing order.

Makes sense, noted.

Olive

Jun 2 2005, 17:42

musicmusic, I seem to recall you said it was not possible to make a console panel, do you know whether Peter has made this possible in 0.9 ?

fabiospark

Jun 3 2005, 00:14

QUOTE

musicmusic ....

QUOTE(upNorth @ Jun 1 2005, 04:30 PM)

QUOTE(musicmusic @ May 31 2005, 11:13 AM)

QUOTE(upNorth @ May 30 2005, 04:35 PM)

3) Is it possible for you to add something to enable us to mark playlists that are locked/autoplaylists, or does this have to be added "in the core"? What I have in mind is something like %_islocked% ond/or %_isautoplaylist% for use in the titleformatting of playlist titles.

Yes, two ways of indicating it I thought of were a lock icon in the playlist switcher, or in the status bar. Im intersted on any opinion on this.

I would personally prefer a way to make use of it in the formatting code for playlist titles. There is currently no icons in your playlist switcher, so an icon might look out of place. The status bar is too far away for my taste. Besides, foosion's new foo_playlist_manager, already has icons that displays this and more. I don't know how much further he wants to develop it, but a panel version would give us a nice alternative to the standard switcher. I would most likely use them both with foo_uie_tabs, as I use the "title formatting for playlist name" feature extensively, and for that reason, will continue to use the current one. Just do whatever you see fit.

Well, I will add %_is_locked%, and then you can colour or whatnot those playlists. True, an icon in my playlist switcher would probably look wierd, but I was thinking of more intuitive ways of showing it (it is clear what such an icon means, rather than random colours and whatnot). So I think I will add an icon in the status bar as well.

I think the best way to indicate the status of playlists would be a conditional formatting of their tabs. That would also be the right place for icons to show up. It would be nice to automatically have highlighted the active playlist - the one where a playing FB will pick up the next track to play (of course with follow cursor disabled). Another useful info would be to highlight, at will, all the playlists that contain the now playing track.

I know one can disable playlist tabs but you can repeat the same formatting in the switcher panel.

----------

Please consider the often requested ability to use the playlist variables outside the playlist window (status bar, trackinfo etc) and, if you can do something about ........

Thanks.

Killmaster

Jun 3 2005, 00:27

Ok, here's some more stuff:

1. Your definition of "horizonal" and "vertical" splitters is reversed - "horizonal" should split it straight across and "vertical" should split it at a 90 degree angle. If you don't know what I'm talking about then please see Windows' "Tile Windows" function.

2. I realize that it's experimental so I'm not going to analyze it too much, but the autohide option doesn't actually do anything useful so far. Right now I have it set up like so:

CODE

V. splitter (autohide enabled)
Horizonal Splitter (autohide enabled)
album list
Columns pl
Seekbar

With autohide enabled on the v. splitter, dragging the albumlist downward will make the seekbar go upward. The albumlist doesn't go down at all, just the seekbar that goes up. It seems that instead of actually autohiding anything, it just makes it impossible to resize any panels.. very strange.

3. Instead of having a sidebar (you were talking about simplifying prefs..) why not add all the options in there back into the main layout page? It'd be nice if you could define certain panels as hideable, and I guess you're already planning on useable autohide, so it would make sense to break away completely from the old sidebar.

4. Not sure if it's already there or not, but it would make a certain amount of sense to have all the exclusively paneled plugins listed under the columns UI in the prefs. Also what's the purpose of the "libraries info" page?

filon

Jun 3 2005, 01:50

QUOTE(musicmusic @ May 31 2005, 05:52 AM)

I found the cause. When you apply the changes, a new album list window is created. But media library is disabled, so it attempts to show media library prefs page. But when current prefs page is being destroyed, it again tries to apply the changes, and you can see how it enters this loop..

I'll have a fixed build up soon.

Good work with finding the bug.

And a question...
As you've told, splitters are not panels itself. Are you going to panelize them later? That would be very nice to have the possibility of just swapping them laster with other "layout manager" type panel (for example, tabbed panel stack or something completely different).

sPeziFisH

Jun 3 2005, 02:22

QUOTE(Killmaster @ Jun 2 2005, 10:27 PM)

1. Your definition of "horizonal" and "vertical" splitters is reversed - "horizonal" should split it straight across and "vertical" should split it at a 90 degree angle. If you don't know what I'm talking about then please see Windows' "Tile Windows" function.

indeed, it's a bit confusing, it took some minutes too me to realize that a 'vertical splitter' is not a splitter which is vertical aligned (so that it splits in horizontal) but is splitting the vertical - 'splitter vertical' vs. 'vertical splitter'.
At least the meaning (expression) of what it is right now is correct, but it's confusing and still depending on the way the user is thinking.
A look at 'Orientation' tells you it's orientated vertical, which is not the case - it splits vertical but is orientated horizontal...
Some text on the layout-page may help ...or the user is going to learn to think about those experiences, might also be okay

always expand yourself !

The Link

Jun 3 2005, 02:32

QUOTE(Killmaster @ Jun 3 2005, 07:27 AM)

4. Not sure if it's already there or not, but it would make a certain amount of sense to have all the exclusively paneled plugins listed under the columns UI in the prefs. ...

I agree. Imho it would make sense to list all panels under columns ui in the preferences as playlist switcher and playlist view already are (though i know they are not separate plugins). It would make things cleaner and more clear where each plugin belongs to.

musicmusic

Jun 3 2005, 05:13

QUOTE(fabiospark @ Jun 3 2005, 06:14 AM)

Please consider the often requested ability to use the playlist variables outside the playlist window (status bar, trackinfo etc) and, if you can do something about ........

Thanks.

The only issue I have with this is that these strings are shared, and other things that use them may not be global var aware, and the other issues that arise as a result..

QUOTE(Killmaster @ Jun 3 2005, 06:27 AM)

Ok, here's some more stuff:

1. Your definition of "horizonal" and "vertical" splitters is reversed - "horizonal" should split it straight across and "vertical" should split it at a 90 degree angle. If you don't know what I'm talking about then please see Windows' "Tile Windows" function.

I agree that it is reveresed for panels, and that will be changed. But for splitters it makes sense to me - a vertical splitter looks very vertical to me and also split the area vertically.

QUOTE(Killmaster @ Jun 3 2005, 06:27 AM)

2. I realize that it's experimental so I'm not going to analyze it too much, but the autohide option doesn't actually do anything useful so far. Right now I have it set up like so:

CODE

V. splitter (autohide enabled)
Horizonal Splitter (autohide enabled)
album list
Columns pl
Seekbar

With autohide enabled on the v. splitter, dragging the albumlist downward will make the seekbar go upward. The albumlist doesn't go down at all, just the seekbar that goes up. It seems that instead of actually autohiding anything, it just makes it impossible to resize any panels.. very strange.

Yes hidden splitters (and as a result, auto-hide) was broken in pa11. This is already fixed for the next version.

QUOTE(Killmaster @ Jun 3 2005, 06:27 AM)

4. Not sure if it's already there or not, but it would make a certain amount of sense to have all the exclusively paneled plugins listed under the columns UI in the prefs. Also what's the purpose of the "libraries info" page?

Well, panels can be used by other components apart from Columns. If this happened, then you would then be forced to have Columns installed just to see their config.

QUOTE(filon @ Jun 3 2005, 07:50 AM)

QUOTE(musicmusic @ May 31 2005, 05:52 AM)

I found the cause. When you apply the changes, a new album list window is created. But media library is disabled, so it attempts to show media library prefs page. But when current prefs page is being destroyed, it again tries to apply the changes, and you can see how it enters this loop..

I'll have a fixed build up soon.

Good work with finding the bug.

And a question...
As you've told, splitters are not panels itself. Are you going to panelize them later? That would be very nice to have the possibility of just swapping them laster with other "layout manager" type panel (for example, tabbed panel stack or something completely different).

They won't be "panelised", but I may add similar panels eventually, so you can use them inside of panel stack if desired.

pIv

Jun 3 2005, 05:31

To musicmusic

It is possible add transparency option to ui_columns as it is in standart ui?

filon

Jun 3 2005, 10:54

I found kind of bug.

If someone set "Show Toolbars" off, did the same with system tray icon and removed "CTRL + P" shortcut for preferences, then he can't show preferences anymore.

Maybe it's not really smart to do all that, but we shouldn't be able to get to such irreversible state...

anza

Jun 3 2005, 12:30

Well one could then just remove foo_ui_columns.dll, restart foobar and set the shortcut for preferences and after that just install foo_ui_columns again and restart.

musicmusic

Jun 3 2005, 15:58

QUOTE(pIv @ Jun 3 2005, 11:31 AM)

To musicmusic

It is possible add transparency option to ui_columns as it is in standart ui?

I don't think it is particulary useful; but maybe at some point later on.

QUOTE(filon @ Jun 3 2005, 04:54 PM)

I found kind of bug.

If someone set "Show Toolbars" off, did the same with system tray icon and removed "CTRL + P" shortcut for preferences, then he can't show preferences anymore.

Maybe it's not really smart to do all that, but we shouldn't be able to get to such irreversible state...

Its not "irreversible"; you can use command line to bring up the prefs.

ephemeros

Jun 4 2005, 04:17

Hi musicmusic,

I don't think this has been reported before:
Both captions and the playback buttons panel show up completely black -> pic here
I 'fixed' the playback buttons by using a bitmap, so it just seems that both background and text color are set to black. Might have something to do with my Windows color scheme, still an easy way to change caption background color would be nice..

BTW, Thanks for your outstanding plug-ins! They make me rediscover my music collection time after time

musicmusic

Jun 4 2005, 07:26

QUOTE(ephemeros @ Jun 4 2005, 10:17 AM)

Hi musicmusic,

I don't think this has been reported before:
Both captions and the playback buttons panel show up completely black -> pic here
I 'fixed' the playback buttons by using a bitmap, so it just seems that both background and text color are set to black. Might have something to do with my Windows color scheme, still an easy way to change caption background color would be nice..

BTW, Thanks for your outstanding plug-ins! They make me rediscover my music collection time after time

Thanks, i think I know what the problem is, it should be fixed for the next version.

Insolent

Jun 4 2005, 19:49

Are we going to see something like $set_style(frame,left,top,right,bottom) to edit the frames in the next release? At the moment, using the new $set_style() functions, you have to do something like this:

CODE

$set_style(text,blah,blah)
$set_style(back,blah,blah)
|||||left|top|right|bottom

...which looks seriously dodgy.