--- Screen.pm	2007/08/05 01:31:41	142
+++ Screen.pm	2007/08/05 19:44:20	161
@@ -23,6 +23,33 @@
 
 Screen - simulated 256*256 pixels monochrome screen using SDL
 
+=head1 Architecture dependent
+
+You may override following methods if you want to implement keyboard on each
+keypress event. Alternative is to use <read> hook and trap memory access.
+
+=head2 key_down
+
+  $self->key_down( 'a' );
+
+=cut
+
+sub key_down {}
+
+=head2 key_up
+
+  $self->key_up( 'a' );
+
+=cut
+
+sub key_up {}
+
+
+=head1 Architecture independent
+
+You don't need to override any of following function in your architecture,
+but you might want to call them.
+
 =head2 open_screen
 
 Open simulated screen
@@ -49,6 +76,7 @@
 	);
 	#$app->grab_input( SDL_GRAB_QUERY );
 	$app->grab_input( SDL_GRAB_OFF );
+	$app->title( ref($self) );
 
 	$self->app( $app );
 
@@ -65,8 +93,6 @@
 my $green	= SDL::Color->new( -r => 0x00, -g => 0xff, -b => 0x00 );
 my $blue	= SDL::Color->new( -r => 0x00, -g => 0x00, -b => 0xff );
 
-my $rect_mem = SDL::Rect->new( -x => 256, -y => 0, -width => 256, -height => 256 );
-
 =head2 mem_xy
 
 Helper to return x and y coordinates in memory map
@@ -155,7 +181,7 @@
 
 	my $scale = $self->scale || confess "no scale?";
 
-	my $rect		= SDL::Rect->new( -x => 0, -y => 0, -width => 256 * $scale, -height => 256 * $scale );
+	my $rect        = SDL::Rect->new( -x => 0, -y => 0, -width => 256 * $scale, -height => 256 * $scale );
 	my $rect_screen = SDL::Rect->new( -x => 0, -y => 0, -width => 256 * $scale, -height => 256 * $scale );
 
 	if ( $scale > 1 ) {
@@ -197,7 +223,9 @@
 
 	$vram->display_format;
 
-	my $rect = SDL::Rect->new( -x => 0, -y => 0, -width => 256, -height => 256 );
+	my $rect     = SDL::Rect->new( -x => 0, -y => 0, -width => 256, -height => 256 );
+	my $rect_mem = SDL::Rect->new( -x => 256 * $self->scale, -y => 0, -width => 256, -height => 256 );
+
 	$vram->blit( $rect, $app, $rect_mem );
 
 	$app->sync;
@@ -219,15 +247,6 @@
 my $pending_key;
 my $run_for = 2000;
 
-my $key_down;
-
-sub key_down {
-	my $self = shift;
-	my $key = shift;
-	warn "key_down($key) = ",$key_down->{$key}, "\n" if $self->debug;
-	return $key_down->{$key};
-}
-
 sub key_pressed {
 	my $self = shift;
 
@@ -236,7 +255,20 @@
 
 	my $event = $self->event || confess "no event?";
 
-	$event->poll || return $pending_key;
+	if ( ! $event->poll ) {
+		return $pending_key unless $self->can('session_event');
+		if ( my $h = $self->session_event('key_pressed') ) {
+			my ( $key, $state ) = %$h;
+			if ( $state ) {
+				$pending_key = $key;
+				$self->key_down( $key );
+			} else {
+				undef $pending_key;
+				$self->key_up( $key );
+			}
+		}
+		return $pending_key;
+	}
 
 	my $type = $event->type();
 
@@ -246,7 +278,6 @@
 
 	if ($type == SDL_KEYDOWN) {
 		$k = $event->key_name();
-		$key_down->{$k}++;
 		if ( $k eq 'escape' ) {
 			$run_for = $self->cli;
 			warn "will check event loop every $run_for cycles\n";
@@ -254,15 +285,18 @@
 		} else {
 			warn "SDL_KEYDOWN ($type) = '$k'", $just_checking ? ' fake' : '', "\n";
 			$pending_key = $k;
+			$self->key_down( $k );
+			$self->record_session('key_pressed', { $k => 1 });
 		}
 	} elsif ( $type == SDL_KEYUP ) {
 		my $up = $event->key_name();
-		$key_down->{$up} = 0;
 		warn "SDL_KEYUP ($type) = '$up'", $just_checking ? ' fake' : '', "\n";
+		$self->key_up( $up );
+		$self->record_session('key_pressed', { $up => 0 });
 		undef $pending_key;
 	}
 
-	warn "key_pressed = $pending_key\n" if $pending_key;
+	warn "key_pressed = $pending_key\n" if ( $pending_key );
 
 	return $pending_key;
 }
@@ -308,4 +342,5 @@
 under the same terms as Perl itself.
 
 =cut
+
 1;