--- deliantra/Deliantra-Client/bin/cfplus	2007/08/01 14:27:21	1.199
+++ deliantra/Deliantra-Client/bin/cfplus	2007/08/09 03:40:44	1.206
@@ -146,6 +146,9 @@
 our $PL_NOTEBOOK;
 our $PL_WINDOW;
 
+our $MUSIC_PLAYING_WIDGET;
+our $LICENSE_WIDGET;
+
 our $INVENTORY_PAGE;
 our $STATS_PAGE;
 our $SKILL_PAGE;
@@ -216,6 +219,9 @@
    $CFG->{effects_enable}
       or return;
 
+   $AUDIO_PLAY{$face}
+      or return;
+
    if (my $chunk = $AUDIO_CHUNK{$face}) {
       for (grep $_->[0] >= Event::time, @{(delete $AUDIO_PLAY{$face}) || []}) {
          my (undef, $dx, $dy, $vol) = @$_;
@@ -238,14 +244,16 @@
 
       # if its a jingle, play it as ambient music
       if ($meta->{meta}{jingle}) {
-         delete $AUDIO_PLAY{$face}; # take the jingle out of the sound queue
-         push @MUSIC_JINGLE, $meta; # push it oto the music/jingle queue
-         &audio_music_push ($face);
+         if (delete $AUDIO_PLAY{$face}) { # take the jingle out of the sound queue
+            push @MUSIC_JINGLE, $meta; # push it oto the music/jingle queue
+            &audio_music_push ($face);
+         }
       } else {
          # fetch from database
          CFPlus::DB::get res_data => $meta->{name}, sub {
             my $rwops = new CFPlus::RW $_[0];
-            my $chunk = new CFPlus::MixChunk $rwops;
+            my $chunk = new CFPlus::MixChunk $rwops
+               or Carp::confess "sound face " . (JSON::XS::to_json $meta) . " unloadable: " . CFPlus::Mix_GetError;
             $chunk->volume (($meta->{meta}{volume} || 1) * 128);
             $AUDIO_CHUNK{$face} = $chunk;
 
@@ -269,6 +277,18 @@
       unless @$queue > 1;
 }
 
+sub audio_music_set_meta {
+   my ($meta) = @_;
+
+   $MUSIC_PLAYING_META = $meta;
+   $MUSIC_PLAYING_WIDGET->set_markup (
+      "<b>Name</b>:	" . (CFPlus::asxml $meta->{meta}{name}) . "\n"
+    . "<b>Author</b>:	" . (CFPlus::asxml $meta->{meta}{author}) . "\n"
+    . "<b>Source</b>:	" . (CFPlus::asxml $meta->{meta}{source}) . "\n"
+    . "<b>License</b>:	" . (CFPlus::asxml $meta->{meta}{license})
+   );
+}
+
 sub audio_music_update_volume {
    return unless $MUSIC_PLAYING_META;
    my $volume = $MUSIC_PLAYING_META->{meta}{volume} || 1;
@@ -295,7 +315,7 @@
          : new CFPlus::RW $$MUSIC_PLAYING_DATA;
 
       $MUSIC_PLAYER = new CFPlus::MixMusic $rwops
-         or ((warn CFPlus::Mix_GetError), return); # pretty fatal error
+         or Carp::confess "music face $meta->{face} unloadable: " . CFPlus::Mix_GetError;
 
       my $NOW = time;
 
@@ -308,8 +328,8 @@
          $MUSIC_START = time;
       }
 
-      delete $MUSIC_PLAYING_META->{stop_time};
-      delete $MUSIC_PLAYING_META->{stop_pos};
+      delete $meta->{stop_time};
+      delete $meta->{stop_pos};
    }
 }
 
@@ -338,8 +358,7 @@
    }
    
    # if the currently playing song is acceptable, let it continue
-   return if $MUSIC_PLAYING_META
-             && grep $MUSIC_PLAYING_META == $_, @MUSIC_HAVE;
+   return if grep $MUSIC_PLAYING_META == $_, @MUSIC_HAVE;
 
    my $NOW = time;
 
@@ -353,7 +372,7 @@
 
       # if the most recently-played piece played very recently,
       # resume it, else choose the oldest piece for rotation.
-      $MUSIC_PLAYING_META =
+      audio_music_set_meta
          $MUSIC_HAVE[-1]{stop_time} > $NOW - $MUSIC_RESUME
             ? $MUSIC_HAVE[-1]
             : $MUSIC_HAVE[0];
@@ -374,6 +393,8 @@
    shift @MUSIC_JINGLE
       while @MUSIC_JINGLE && $MUSIC_PLAYING_META == $MUSIC_JINGLE[0];
 
+   $MUSIC_PLAYING_WIDGET->clear;
+
    undef $MUSIC_PLAYER;
    undef $MUSIC_PLAYING_META;
    undef $MUSIC_PLAYING_DATA;
@@ -861,16 +882,20 @@
 #      $CFG->{effects_volume} = $_[1];
 #   });
 
-   $table->add_at (0, $row, new CFPlus::UI::Label valign => 0, align => 1, text => "Effects Volume");
+   $table->add_at (0, $row, new CFPlus::UI::Label valign => 0, align => 1, text => "Sound Effects");
    $table->add_at (1, $row, new CFPlus::UI::CheckBox
       expand => 1, state => $CFG->{effects_enable},
       tooltip => "If enabled, sound effects are enabled. If disabled, no sound effects will be played.",
-      on_changed => sub { $CFG->{effects_enable} = $_[1]; 0 }
+      on_changed => sub {
+         $CFG->{effects_enable} = $_[1];
+         $CONN->update_fx_want if $CONN;
+         0
+      }
    );
    $table->add_at (2, $row++, new CFPlus::UI::Slider
       expand => 1, range => [$CFG->{effects_volume}, 0, 1, 0, 1/128],
       tooltip => "The relative volume of sound effects. Best audio quality is achieved if this "
-                 . "is set highest and you use your operating system volume setting. Changes are instant.",
+                 . "is set highest (rightmost) and you use your operating system volume setting. Changes are instant.",
       on_changed => sub { $CFG->{effects_volume} = $_[1]; 0 }
    );
 
@@ -878,7 +903,11 @@
    $table->add_at (1, $row, new CFPlus::UI::CheckBox
       expand => 1, state => $CFG->{bgm_enable},
       tooltip => "If enabled, playing of background music is enabled. If disabled, no background music will be played.",
-      on_changed => sub { $CFG->{bgm_enable} = $_[1]; 0 }
+      on_changed => sub {
+         $CFG->{bgm_enable} = $_[1];
+         $CONN->update_fx_want if $CONN;
+         0
+      }
    );
    $table->add_at (2, $row++, new CFPlus::UI::Slider
       expand => 1, range => [$CFG->{bgm_volume}, 0, 1, 0, 1/128],
@@ -1588,6 +1617,46 @@
    $hb
 }
 
+sub media_window {
+   my $vb = new CFPlus::UI::VBox;
+
+   $vb->add (new CFPlus::UI::FancyFrame
+      label => "Currently playing music",
+      child => new CFPlus::UI::ScrolledWindow scroll_x => 1, scroll_y => 0,
+                  child => ($MUSIC_PLAYING_WIDGET = new CFPlus::UI::Label ellipsise => 0, fontsize => 0.8),
+   );
+
+   $vb->add (new CFPlus::UI::FancyFrame
+      label  => "Other media used in this session",
+      expand => 1,
+      child  => ($LICENSE_WIDGET = new CFPlus::UI::TextScroller
+                expand => 1, fontsize => 0.8, padding_x => 4, padding_y => 4),
+   );
+
+   $vb
+}
+
+sub add_license {
+   my ($meta) = @_;
+
+   $meta = $meta->{meta}
+      or return;
+
+   $meta->{license} || $meta->{author} || $meta->{source}
+      or return;
+
+   $LICENSE_WIDGET->add_paragraph ({
+      fg     => [1, 1, 1, 1],
+      markup => "<small>"
+              . "<b>Name:</b>	" . (CFPlus::asxml $meta->{name}) . "\n"
+              . "<b>Author:</b>	" . (CFPlus::asxml $meta->{author}) . "\n"
+              . "<b>Source:</b>	" . (CFPlus::asxml $meta->{source}) . "\n"
+              . "<b>License:</b>	" . (CFPlus::asxml $meta->{license}) . "\n"
+              . "</small>",
+   });
+   $LICENSE_WIDGET->scroll_to_bottom;
+}
+
 sub toggle_player_page {
    my ($widget) = @_;
 
@@ -1634,9 +1703,12 @@
       "Toggles the inventory window, where you can manage your loot (or treasures :). "
       . "You can also hit the <b>Tab</b>-key to show/hide the Inventory."
    );
-   $ntb->add_tab (Pickup   => autopickup_setup,
+   $ntb->add_tab (Pickup => autopickup_setup,
       "Configure autopickup settings, i.e. which items you will pick up automatically when walking (or running) over them.");
 
+   $ntb->add_tab (Media => media_window,
+      "License, Author and Source info for media sent by the server.");
+
    $ntb->set_current_page ($INVENTORY_PAGE);
 
    $plwin->add ($ntb);
@@ -2224,13 +2296,15 @@
 #CFPlus::_exit 0;
 
 END {
+   video_shutdown;
+   audio_shutdown;
    CFPlus::SDL_Quit;
    CFPlus::DB::Server::stop;
 }
 
 =head1 NAME
 
-cfplus - A Crossfire+ and Crossfire game client
+cfplus - A Crossfire TRT and Crossfire game client
 
 =head1 SYNOPSIS
 
@@ -2238,8 +2312,8 @@
 
 =head1 USAGE
 
-cfplus utilises OpenGL for all UI elements and the game. It is supposed to be used
-fullscreen and interactively.
+cfplus utilises OpenGL for all UI elements and the game. It is supposed to
+be used in fullscreen mode and interactively.
 
 =head1 DEBUGGING