barriers: Don't loop over the server to destroy a barrier
This is completely pointless as far as I can tell. Signed-off-by: Jasper St. Pierre <jstpierre@mecheye.net> Reviewed-by: Peter Hutterer <peter.hutterer@who-t.net>
This commit is contained in:
		
							parent
							
								
									482e0cb352
								
							
						
					
					
						commit
						2868a93945
					
				| 
						 | 
					@ -437,22 +437,10 @@ CreatePointerBarrierClient(ClientPtr client,
 | 
				
			||||||
static int
 | 
					static int
 | 
				
			||||||
BarrierFreeBarrier(void *data, XID id)
 | 
					BarrierFreeBarrier(void *data, XID id)
 | 
				
			||||||
{
 | 
					{
 | 
				
			||||||
    struct PointerBarrierClient *b = NULL, *barrier;
 | 
					    struct PointerBarrierClient *barrier;
 | 
				
			||||||
    ScreenPtr screen;
 | 
					 | 
				
			||||||
    BarrierScreenPtr cs;
 | 
					 | 
				
			||||||
 | 
					
 | 
				
			||||||
    barrier = container_of(data, struct PointerBarrierClient, barrier);
 | 
					    barrier = container_of(data, struct PointerBarrierClient, barrier);
 | 
				
			||||||
 | 
					    xorg_list_del(&barrier->entry);
 | 
				
			||||||
    screen = barrier->screen;
 | 
					 | 
				
			||||||
    cs = GetBarrierScreen(screen);
 | 
					 | 
				
			||||||
 | 
					 | 
				
			||||||
    /* find and unlink from the screen private */
 | 
					 | 
				
			||||||
    xorg_list_for_each_entry(b, &cs->barriers, entry) {
 | 
					 | 
				
			||||||
        if (b == barrier) {
 | 
					 | 
				
			||||||
            xorg_list_del(&b->entry);
 | 
					 | 
				
			||||||
            break;
 | 
					 | 
				
			||||||
        }
 | 
					 | 
				
			||||||
    }
 | 
					 | 
				
			||||||
 | 
					
 | 
				
			||||||
    free(barrier);
 | 
					    free(barrier);
 | 
				
			||||||
    return Success;
 | 
					    return Success;
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
		Loading…
	
		Reference in New Issue