commit ebe29ef37611f246fcb4900a2f351c6ef69660bb
author Alexandre Julliard <julliard@winehq.org> Sat Jun 26 08:43:26 1999 +0000
committer Alexandre Julliard <julliard@winehq.org> Sat Jun 26 08:43:26 1999 +0000
tree bac91dfb746c21e002d179754299eec4db93a53a
parent f26c89e83ab45f34fb56b6935f35f9e13584ae0d

Further server optimizations:
- merged request and reply structures
- build requests directly into the buffer to avoid a copy

server/change.c

diff --git a/server/change.c b/server/change.c
index fb1f0cb..50f193e 100644
--- a/server/change.c
+++ b/server/change.c
@@ -41,13 +41,15 @@
 };
 
 
-static struct object *create_change_notification( int subtree, int filter )
+static struct change *create_change_notification( int subtree, int filter )
 {
     struct change *change;
-    if (!(change = alloc_object( &change_ops ))) return NULL;
-    change->subtree = subtree;
-    change->filter  = filter;
-    return &change->obj;
+    if ((change = alloc_object( &change_ops )))
+    {
+        change->subtree = subtree;
+        change->filter  = filter;
+    }
+    return change;
 }
 
 static void change_dump( struct object *obj, int verbose )
@@ -68,13 +70,13 @@
 /* create a change notification */
 DECL_HANDLER(create_change_notification)
 {
-    struct object *obj;
-    struct create_change_notification_reply *reply = push_reply_data( current, sizeof(*reply) );
+    struct change *change;
 
-    if ((obj = create_change_notification( req->subtree, req->filter )))
+    req->handle = -1;
+    if ((change = create_change_notification( req->subtree, req->filter )))
     {
-        reply->handle = alloc_handle( current->process, obj,
-                                      STANDARD_RIGHTS_REQUIRED|SYNCHRONIZE, 0 );
-        release_object( obj );
+        req->handle = alloc_handle( current->process, change,
+                                    STANDARD_RIGHTS_REQUIRED|SYNCHRONIZE, 0 );
+        release_object( change );
     }
 }