From c119239e0e8869ab4cfd73681921f82750238e0a Mon Sep 17 00:00:00 2001 From: "Enrico Weigelt, metux IT consult" Date: Fri, 28 Jun 2024 13:24:15 +0200 Subject: [PATCH] (submit/xext-saver-cleanup) Xext: saver: drop New() macro We can directly use calloc() as all other places do. If we wanna have an convenient macro for struct allocation, that would be the job of a separate patch queue and should be done consequently, treewide. Signed-off-by: Enrico Weigelt, metux IT consult --- Xext/saver.c | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/Xext/saver.c b/Xext/saver.c index af20d15a4..21f668d14 100644 --- a/Xext/saver.c +++ b/Xext/saver.c @@ -195,8 +195,6 @@ static DevPrivateKeyRec ScreenPrivateKeyRec; dixSetPrivate(&(s)->devPrivates, ScreenPrivateKey, v); #define SetupScreen(s) ScreenSaverScreenPrivatePtr pPriv = (s ? GetScreenPrivate(s) : NULL) -#define New(t) (malloc(sizeof (t))) - static void CheckScreenPrivate(ScreenPtr pScreen) { @@ -219,7 +217,7 @@ MakeScreenPrivate(ScreenPtr pScreen) if (pPriv) return pPriv; - pPriv = New(ScreenSaverScreenPrivateRec); + pPriv = calloc(1, sizeof(ScreenSaverScreenPrivateRec)); if (!pPriv) return 0; pPriv->events = 0; @@ -269,7 +267,7 @@ setEventMask(ScreenPtr pScreen, ClientPtr client, unsigned long mask) } else { if (!pEv) { - pEv = New(ScreenSaverEventRec); + pEv = calloc(1, sizeof(ScreenSaverEventRec)); if (!pEv) { CheckScreenPrivate(pScreen); return FALSE; @@ -848,7 +846,7 @@ ScreenSaverSetAttributes(ClientPtr client) if (!pPriv) return FALSE; } - pAttr = New(ScreenSaverAttrRec); + pAttr = calloc(1, sizeof(ScreenSaverAttrRec)); if (!pAttr) { ret = BadAlloc; goto bail;