Mesa (master): st/nine: Queries: return S_FALSE instead of INVALIDCALL when in building query state
Ilia Mirkin
imirkin at kemper.freedesktop.org
Wed Dec 3 15:43:24 UTC 2014
Module: Mesa
Branch: master
Commit: ca0588d1a1e61404687fb6a6bc4857819fa46e2e
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=ca0588d1a1e61404687fb6a6bc4857819fa46e2e
Author: Axel Davy <axel.davy at ens.fr>
Date: Sun Nov 23 11:44:59 2014 +0100
st/nine: Queries: return S_FALSE instead of INVALIDCALL when in building query state
It is the same behaviour as wine has.
Reviewed-by: Ilia Mirkin <imirkin at alum.mit.edu>
Reviewed-by: David Heidelberg <david at ixit.cz>
Signed-off-by: Axel Davy <axel.davy at ens.fr>
---
src/gallium/state_trackers/nine/query9.c | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/src/gallium/state_trackers/nine/query9.c b/src/gallium/state_trackers/nine/query9.c
index 17c36e5..b74a707 100644
--- a/src/gallium/state_trackers/nine/query9.c
+++ b/src/gallium/state_trackers/nine/query9.c
@@ -205,7 +205,10 @@ NineQuery9_GetData( struct NineQuery9 *This,
DBG("This=%p pData=%p dwSize=%d dwGetDataFlags=%d\n",
This, pData, dwSize, dwGetDataFlags);
- user_assert(This->state != NINE_QUERY_STATE_RUNNING, D3DERR_INVALIDCALL);
+ /* according to spec we should return D3DERR_INVALIDCALL here, but
+ * wine returns S_FALSE because it is apparently the behaviour
+ * on windows */
+ user_assert(This->state != NINE_QUERY_STATE_RUNNING, S_FALSE);
user_assert(dwSize == 0 || pData, D3DERR_INVALIDCALL);
user_assert(dwGetDataFlags == 0 ||
dwGetDataFlags == D3DGETDATA_FLUSH, D3DERR_INVALIDCALL);
More information about the mesa-commit
mailing list