On Tue, Aug 22, 2017 at 12:19 PM, Adam Jackson <a...@redhat.com> wrote: > Discarding the return value here is just wrong. > > Signed-off-by: Adam Jackson <a...@redhat.com>
Reviewed-by: Alex Deucher <alexander.deuc...@amd.com> > --- > glx/glxcmds.c | 3 +-- > glx/glxcmdsswap.c | 3 +-- > 2 files changed, 2 insertions(+), 4 deletions(-) > > diff --git a/glx/glxcmds.c b/glx/glxcmds.c > index 6a763970da..241abc6a58 100644 > --- a/glx/glxcmds.c > +++ b/glx/glxcmds.c > @@ -2376,8 +2376,7 @@ __glXDisp_VendorPrivate(__GLXclientState * cl, GLbyte * > pc) > __glXGetProtocolDecodeFunction(&VendorPriv_dispatch_info, > vendorcode, 0); > if (proc != NULL) { > - (*proc) (cl, (GLbyte *) req); > - return Success; > + return (*proc) (cl, (GLbyte *) req); > } > > cl->client->errorValue = req->vendorCode; > diff --git a/glx/glxcmdsswap.c b/glx/glxcmdsswap.c > index 44a09e61cb..10d84062ed 100644 > --- a/glx/glxcmdsswap.c > +++ b/glx/glxcmdsswap.c > @@ -873,8 +873,7 @@ __glXDispSwap_VendorPrivate(__GLXclientState * cl, GLbyte > * pc) > __glXGetProtocolDecodeFunction(&VendorPriv_dispatch_info, > vendorcode, 1); > if (proc != NULL) { > - (*proc) (cl, (GLbyte *) req); > - return Success; > + return (*proc) (cl, (GLbyte *) req); > } > > cl->client->errorValue = req->vendorCode; > -- > 2.13.5 > > _______________________________________________ > xorg-devel@lists.x.org: X.Org development > Archives: http://lists.x.org/archives/xorg-devel > Info: https://lists.x.org/mailman/listinfo/xorg-devel _______________________________________________ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: https://lists.x.org/mailman/listinfo/xorg-devel