From 6e927d6e3bfdf02212adc85d9f83341b61373dd8 Mon Sep 17 00:00:00 2001
From: Stefano Ceccherini <stefano.ceccherini@gmail.com>
Date: Sat, 30 Apr 2005 07:04:27 +0000
Subject: [PATCH] BScreens methods didn't work for a number of reasons: fixed.

git-svn-id: file:///srv/svn/repos/haiku/haiku/trunk@12517 a95241bf-73f2-0310-859d-f6bbb57e9c96
---
 src/servers/app/AppServer.cpp | 28 ----------------------------
 src/servers/app/ServerApp.cpp | 31 +++++++++++++++++++++++++++++++
 2 files changed, 31 insertions(+), 28 deletions(-)

diff --git a/src/servers/app/AppServer.cpp b/src/servers/app/AppServer.cpp
index 3031e8be92..3fd54b734c 100644
--- a/src/servers/app/AppServer.cpp
+++ b/src/servers/app/AppServer.cpp
@@ -738,34 +738,6 @@ void AppServer::DispatchMessage(int32 code, BPortLink &msg)
 
 			break;
 		}
-		case AS_SCREEN_GET_MODE:
-		{
-			port_id replyport = -1;
-			if (msg.Read<port_id>(&replyport) < B_OK)
-				break;
-			
-			screen_id id;
-			msg.Read<screen_id>(&id);
-			uint32 workspace;
-			msg.Read<uint32>(&workspace);
-			
-			// TODO: the display_mode can be different between
-			// the various screens.
-			// We have the screen_id and the workspace number, with these
-			// we need to find the corresponding "driver", and call getmode on it
-			display_mode mode;
-			desktop->GetDisplayDriver()->GetMode(&mode);
-			// actually this isn't still enough as different workspaces can
-			// have different display_modes
-			
-			BPortLink replylink(replyport);
-			replylink.StartMessage(AS_SCREEN_GET_MODE);
-			replylink.Attach<display_mode>(mode);
-			replylink.Attach<status_t>(B_OK);
-			replylink.Flush();
-			
-			break;
-		}
 		case B_QUIT_REQUESTED:
 		{
 #if TEST_MODE
diff --git a/src/servers/app/ServerApp.cpp b/src/servers/app/ServerApp.cpp
index 99250a2dd9..c4b81f1fa1 100644
--- a/src/servers/app/ServerApp.cpp
+++ b/src/servers/app/ServerApp.cpp
@@ -1823,6 +1823,37 @@ void ServerApp::DispatchMessage(int32 code, LinkMsgReader &msg)
 			
 			break;
 		}
+		case AS_SCREEN_GET_MODE:
+		{
+			// Attached data
+			// 1) int32 port to reply to
+			// 2) screen_id
+			// 3) workspace index
+			screen_id id;
+			msg.Read<screen_id>(&id);
+			uint32 workspace;
+			msg.Read<uint32>(&workspace);
+			
+			// TODO: the display_mode can be different between
+			// the various screens.
+			// We have the screen_id and the workspace number, with these
+			// we need to find the corresponding "driver", and call getmode on it
+			display_mode mode;
+			desktop->GetDisplayDriver()->GetMode(&mode);
+			// actually this isn't still enough as different workspaces can
+			// have different display_modes
+			
+			int32 replyport;
+			msg.Read<int32>(&replyport);
+			
+			replylink.SetSendPort(replyport);
+			replylink.StartMessage(SERVER_TRUE);
+			replylink.Attach<display_mode>(mode);
+			replylink.Attach<status_t>(B_OK);
+			replylink.Flush();
+			
+			break;
+		}
 		default:
 		{
 			STRACE(("ServerApp %s received unhandled message code offset %s\n",fSignature.String(),