[RFC v2 03/12] sync: Move fence destroy call to object specific function
Adam Jackson
ajax at nwnk.net
Fri Oct 20 18:36:40 UTC 2017
On Wed, 2017-09-27 at 01:19 -0400, Louis-Francis Ratté-Boulianne wrote:
> @@ -41,11 +41,10 @@ miSyncScreenCreateFence(ScreenPtr pScreen,
> SyncFence * pFence,
> pFence->triggered = initially_triggered;
> }
>
> -void
> -miSyncScreenDestroyFence(ScreenPtr pScreen, SyncFence * pFence)
> +int
> +miSyncGetFenceType(SyncFence *pFence)
> {
> - (void) pScreen;
> - (void) pFence;
> + return pFence->type;
> }
This function is not referenced in this commit, even the header decl
isn't present until 4/12. Since this function appears to be unrelated
to the this refactor it should probably be moved into 4/12. (But my
comments on 4/12 itself may make this point moot.)
> @@ -94,7 +101,8 @@ miSyncInitFence(ScreenPtr pScreen, SyncFence *
> pFence, Bool initially_triggered)
> &miSyncFenceReset,
> &miSyncFenceCheckTriggered,
> &miSyncFenceAddTrigger,
> - &miSyncFenceDeleteTrigger
> + &miSyncFenceDeleteTrigger,
> + &miSyncFenceDestroy
> };
>
> pFence->pScreen = pScreen;
Please use a , even on the last item in a list (yes even though the
original code didn't, it should have).
- ajax
More information about the xorg-devel
mailing list