wine-commits December 2010

wine-commits@winehq.org

2 participants
1099 discussions

Alexandre Julliard : comctl32: Fix positioning of tracking tooltips at the bottom of the screen.
by Alexandre Julliard 13 Dec '10

13 Dec '10

Module: wine Branch: master Commit: 84a86d4897991360c6dc17f2ad9765282c32e06c URL: http://source.winehq.org/git/wine.git/?a=commit;h=84a86d4897991360c6dc17f2a… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Mon Dec 13 16:24:49 2010 +0100 comctl32: Fix positioning of tracking tooltips at the bottom of the screen. --- dlls/comctl32/tooltips.c | 3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/dlls/comctl32/tooltips.c b/dlls/comctl32/tooltips.c index 5951796..774387a 100644 --- a/dlls/comctl32/tooltips.c +++ b/dlls/comctl32/tooltips.c @@ -636,7 +636,8 @@ TOOLTIPS_Show (TOOLTIPS_INFO *infoPtr, BOOL track_activate) if (!(style & TTS_BALLOON)) rect.top -= (size.cy / 2); } - infoPtr->bToolBelow = TRUE; + if (!(infoPtr->bToolBelow = (infoPtr->yTrackPos + size.cy <= GetSystemMetrics(SM_CYSCREEN)))) + rect.top -= size.cy; if (!(toolPtr->uFlags & TTF_ABSOLUTE)) {

1 0

Henri Verbeet : wined3d: Pass an IWineD3DBaseShaderImpl pointer to shader_cleanup().
by Alexandre Julliard 13 Dec '10

13 Dec '10

Module: wine Branch: master Commit: bbbab10c8e647bba23929ca7db3a4803adf3fe29 URL: http://source.winehq.org/git/wine.git/?a=commit;h=bbbab10c8e647bba23929ca7d… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Mon Dec 13 13:40:33 2010 +0100 wined3d: Pass an IWineD3DBaseShaderImpl pointer to shader_cleanup(). --- dlls/wined3d/shader.c | 16 +++++++--------- 1 files changed, 7 insertions(+), 9 deletions(-) diff --git a/dlls/wined3d/shader.c b/dlls/wined3d/shader.c index 1220e21..31ece02 100644 --- a/dlls/wined3d/shader.c +++ b/dlls/wined3d/shader.c @@ -1475,10 +1475,8 @@ static void shader_trace_init(const struct wined3d_shader_frontend *fe, void *fe } } -static void shader_cleanup(IWineD3DBaseShader *iface) +static void shader_cleanup(IWineD3DBaseShaderImpl *shader) { - IWineD3DBaseShaderImpl *shader = (IWineD3DBaseShaderImpl *)iface; - shader->baseShader.device->shader_backend->shader_destroy(shader); HeapFree(GetProcessHeap(), 0, shader->baseShader.reg_maps.constf); HeapFree(GetProcessHeap(), 0, shader->baseShader.function); @@ -1661,7 +1659,7 @@ static ULONG STDMETHODCALLTYPE vertexshader_Release(IWineD3DVertexShader *iface) if (!refcount) { - shader_cleanup((IWineD3DBaseShader *)iface); + shader_cleanup((IWineD3DBaseShaderImpl *)shader); shader->baseShader.parent_ops->wined3d_object_destroyed(shader->baseShader.parent); HeapFree(GetProcessHeap(), 0, shader); } @@ -1860,7 +1858,7 @@ HRESULT vertexshader_init(IWineD3DVertexShaderImpl *shader, IWineD3DDeviceImpl * if (FAILED(hr)) { WARN("Failed to set function, hr %#x.\n", hr); - shader_cleanup((IWineD3DBaseShader *)shader); + shader_cleanup((IWineD3DBaseShaderImpl *)shader); return hr; } @@ -1933,7 +1931,7 @@ static ULONG STDMETHODCALLTYPE geometryshader_Release(IWineD3DGeometryShader *if if (!refcount) { - shader_cleanup((IWineD3DBaseShader *)iface); + shader_cleanup((IWineD3DBaseShaderImpl *)shader); shader->base_shader.parent_ops->wined3d_object_destroyed(shader->base_shader.parent); HeapFree(GetProcessHeap(), 0, shader); } @@ -1980,7 +1978,7 @@ HRESULT geometryshader_init(struct wined3d_geometryshader *shader, IWineD3DDevic if (FAILED(hr)) { WARN("Failed to set function, hr %#x.\n", hr); - shader_cleanup((IWineD3DBaseShader *)shader); + shader_cleanup((IWineD3DBaseShaderImpl *)shader); return hr; } @@ -2029,7 +2027,7 @@ static ULONG STDMETHODCALLTYPE pixelshader_Release(IWineD3DPixelShader *iface) if (!refcount) { - shader_cleanup((IWineD3DBaseShader *)iface); + shader_cleanup((IWineD3DBaseShaderImpl *)shader); shader->baseShader.parent_ops->wined3d_object_destroyed(shader->baseShader.parent); HeapFree(GetProcessHeap(), 0, shader); } @@ -2267,7 +2265,7 @@ HRESULT pixelshader_init(IWineD3DPixelShaderImpl *shader, IWineD3DDeviceImpl *de if (FAILED(hr)) { WARN("Failed to set function, hr %#x.\n", hr); - shader_cleanup((IWineD3DBaseShader *)shader); + shader_cleanup((IWineD3DBaseShaderImpl *)shader); return hr; }

1 0

Henri Verbeet : wined3d: Pass an IWineD3DBaseShaderImpl pointer to shader_destroy().
by Alexandre Julliard 13 Dec '10

13 Dec '10

Module: wine Branch: master Commit: fe2ffe1e6a47f190c3e3b508267bf1bbd96da4fb URL: http://source.winehq.org/git/wine.git/?a=commit;h=fe2ffe1e6a47f190c3e3b5082… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Mon Dec 13 13:40:32 2010 +0100 wined3d: Pass an IWineD3DBaseShaderImpl pointer to shader_destroy(). --- dlls/wined3d/arb_program_shader.c | 16 ++++++------ dlls/wined3d/device.c | 5 ++- dlls/wined3d/glsl_shader.c | 44 ++++++++++++++++++++---------------- dlls/wined3d/shader.c | 4 +- dlls/wined3d/wined3d_private.h | 3 +- 5 files changed, 39 insertions(+), 33 deletions(-) diff --git a/dlls/wined3d/arb_program_shader.c b/dlls/wined3d/arb_program_shader.c index bd5b197..1526e70 100644 --- a/dlls/wined3d/arb_program_shader.c +++ b/dlls/wined3d/arb_program_shader.c @@ -4702,14 +4702,14 @@ static void shader_arb_deselect_depth_blt(void *shader_priv, const struct wined3 } } -static void shader_arb_destroy(IWineD3DBaseShader *iface) { - IWineD3DBaseShaderImpl *baseShader = (IWineD3DBaseShaderImpl *) iface; - IWineD3DDeviceImpl *device = baseShader->baseShader.device; +static void shader_arb_destroy(IWineD3DBaseShaderImpl *shader) +{ + IWineD3DDeviceImpl *device = shader->baseShader.device; const struct wined3d_gl_info *gl_info = &device->adapter->gl_info; - if (shader_is_pshader_version(baseShader->baseShader.reg_maps.shader_version.type)) + if (shader_is_pshader_version(shader->baseShader.reg_maps.shader_version.type)) { - struct arb_pshader_private *shader_data = baseShader->baseShader.backend_data; + struct arb_pshader_private *shader_data = shader->baseShader.backend_data; UINT i; if(!shader_data) return; /* This can happen if a shader was never compiled */ @@ -4731,11 +4731,11 @@ static void shader_arb_destroy(IWineD3DBaseShader *iface) { HeapFree(GetProcessHeap(), 0, shader_data->gl_shaders); HeapFree(GetProcessHeap(), 0, shader_data); - baseShader->baseShader.backend_data = NULL; + shader->baseShader.backend_data = NULL; } else { - struct arb_vshader_private *shader_data = baseShader->baseShader.backend_data; + struct arb_vshader_private *shader_data = shader->baseShader.backend_data; UINT i; if(!shader_data) return; /* This can happen if a shader was never compiled */ @@ -4757,7 +4757,7 @@ static void shader_arb_destroy(IWineD3DBaseShader *iface) { HeapFree(GetProcessHeap(), 0, shader_data->gl_shaders); HeapFree(GetProcessHeap(), 0, shader_data); - baseShader->baseShader.backend_data = NULL; + shader->baseShader.backend_data = NULL; } } diff --git a/dlls/wined3d/device.c b/dlls/wined3d/device.c index 15b9020..0789ed9 100644 --- a/dlls/wined3d/device.c +++ b/dlls/wined3d/device.c @@ -6182,8 +6182,9 @@ static void delete_opengl_contexts(IWineD3DDevice *iface, IWineD3DSwapChainImpl gl_info = context->gl_info; IWineD3DDevice_EnumResources(iface, reset_unload_resources, NULL); - LIST_FOR_EACH_ENTRY(shader, &This->shaders, IWineD3DBaseShaderImpl, baseShader.shader_list_entry) { - This->shader_backend->shader_destroy((IWineD3DBaseShader *) shader); + LIST_FOR_EACH_ENTRY(shader, &This->shaders, IWineD3DBaseShaderImpl, baseShader.shader_list_entry) + { + This->shader_backend->shader_destroy(shader); } ENTER_GL(); diff --git a/dlls/wined3d/glsl_shader.c b/dlls/wined3d/glsl_shader.c index a28e9ce..a2fb79a 100644 --- a/dlls/wined3d/glsl_shader.c +++ b/dlls/wined3d/glsl_shader.c @@ -4672,10 +4672,10 @@ static void shader_glsl_deselect_depth_blt(void *shader_priv, const struct wined checkGLcall("glUseProgramObjectARB"); } -static void shader_glsl_destroy(IWineD3DBaseShader *iface) { +static void shader_glsl_destroy(IWineD3DBaseShaderImpl *shader) +{ const struct list *linked_programs; - IWineD3DBaseShaderImpl *This = (IWineD3DBaseShaderImpl *) iface; - IWineD3DDeviceImpl *device = This->baseShader.device; + IWineD3DDeviceImpl *device = shader->baseShader.device; struct shader_glsl_priv *priv = device->shader_priv; const struct wined3d_gl_info *gl_info; struct wined3d_context *context; @@ -4683,41 +4683,44 @@ static void shader_glsl_destroy(IWineD3DBaseShader *iface) { /* Note: Do not use QueryInterface here to find out which shader type this is because this code * can be called from IWineD3DBaseShader::Release */ - char pshader = shader_is_pshader_version(This->baseShader.reg_maps.shader_version.type); + char pshader = shader_is_pshader_version(shader->baseShader.reg_maps.shader_version.type); + + if (pshader) + { + struct glsl_pshader_private *shader_data = shader->baseShader.backend_data; - if(pshader) { - struct glsl_pshader_private *shader_data; - shader_data = This->baseShader.backend_data; if (!shader_data || !shader_data->num_gl_shaders) { HeapFree(GetProcessHeap(), 0, shader_data); - This->baseShader.backend_data = NULL; + shader->baseShader.backend_data = NULL; return; } context = context_acquire(device, NULL); gl_info = context->gl_info; - if (priv->glsl_program && (IWineD3DBaseShader *)priv->glsl_program->pshader == iface) + if (priv->glsl_program && (IWineD3DBaseShaderImpl *)priv->glsl_program->pshader == shader) { ENTER_GL(); shader_glsl_select(context, FALSE, FALSE); LEAVE_GL(); } - } else { - struct glsl_vshader_private *shader_data; - shader_data = This->baseShader.backend_data; + } + else + { + struct glsl_vshader_private *shader_data = shader->baseShader.backend_data; + if (!shader_data || !shader_data->num_gl_shaders) { HeapFree(GetProcessHeap(), 0, shader_data); - This->baseShader.backend_data = NULL; + shader->baseShader.backend_data = NULL; return; } context = context_acquire(device, NULL); gl_info = context->gl_info; - if (priv->glsl_program && (IWineD3DBaseShader *)priv->glsl_program->vshader == iface) + if (priv->glsl_program && (IWineD3DBaseShaderImpl *)priv->glsl_program->vshader == shader) { ENTER_GL(); shader_glsl_select(context, FALSE, FALSE); @@ -4725,7 +4728,7 @@ static void shader_glsl_destroy(IWineD3DBaseShader *iface) { } } - linked_programs = &This->baseShader.linked_programs; + linked_programs = &shader->baseShader.linked_programs; TRACE("Deleting linked programs\n"); if (linked_programs->next) { @@ -4744,9 +4747,10 @@ static void shader_glsl_destroy(IWineD3DBaseShader *iface) { LEAVE_GL(); } - if(pshader) { + if (pshader) + { + struct glsl_pshader_private *shader_data = shader->baseShader.backend_data; UINT i; - struct glsl_pshader_private *shader_data = This->baseShader.backend_data; ENTER_GL(); for(i = 0; i < shader_data->num_gl_shaders; i++) { @@ -4759,8 +4763,8 @@ static void shader_glsl_destroy(IWineD3DBaseShader *iface) { } else { + struct glsl_vshader_private *shader_data = shader->baseShader.backend_data; UINT i; - struct glsl_vshader_private *shader_data = This->baseShader.backend_data; ENTER_GL(); for(i = 0; i < shader_data->num_gl_shaders; i++) { @@ -4772,8 +4776,8 @@ static void shader_glsl_destroy(IWineD3DBaseShader *iface) { HeapFree(GetProcessHeap(), 0, shader_data->gl_shaders); } - HeapFree(GetProcessHeap(), 0, This->baseShader.backend_data); - This->baseShader.backend_data = NULL; + HeapFree(GetProcessHeap(), 0, shader->baseShader.backend_data); + shader->baseShader.backend_data = NULL; context_release(context); } diff --git a/dlls/wined3d/shader.c b/dlls/wined3d/shader.c index 4a72e88..1220e21 100644 --- a/dlls/wined3d/shader.c +++ b/dlls/wined3d/shader.c @@ -1479,7 +1479,7 @@ static void shader_cleanup(IWineD3DBaseShader *iface) { IWineD3DBaseShaderImpl *shader = (IWineD3DBaseShaderImpl *)iface; - shader->baseShader.device->shader_backend->shader_destroy(iface); + shader->baseShader.device->shader_backend->shader_destroy(shader); HeapFree(GetProcessHeap(), 0, shader->baseShader.reg_maps.constf); HeapFree(GetProcessHeap(), 0, shader->baseShader.function); shader_delete_constant_list(&shader->baseShader.constantsF); @@ -1503,7 +1503,7 @@ static void shader_none_update_float_pixel_constants(IWineD3DDeviceImpl *device, static void shader_none_load_constants(const struct wined3d_context *context, char usePS, char useVS) {} static void shader_none_load_np2fixup_constants(void *shader_priv, const struct wined3d_gl_info *gl_info, const struct wined3d_state *state) {} -static void shader_none_destroy(IWineD3DBaseShader *iface) {} +static void shader_none_destroy(IWineD3DBaseShaderImpl *shader) {} static HRESULT shader_none_alloc(IWineD3DDeviceImpl *device) {return WINED3D_OK;} static void shader_none_free(IWineD3DDeviceImpl *device) {} static BOOL shader_none_dirty_const(void) {return FALSE;} diff --git a/dlls/wined3d/wined3d_private.h b/dlls/wined3d/wined3d_private.h index d7612eb..6769956 100644 --- a/dlls/wined3d/wined3d_private.h +++ b/dlls/wined3d/wined3d_private.h @@ -57,6 +57,7 @@ typedef struct IWineD3DSurfaceImpl IWineD3DSurfaceImpl; typedef struct IWineD3DPaletteImpl IWineD3DPaletteImpl; typedef struct IWineD3DDeviceImpl IWineD3DDeviceImpl; typedef struct IWineD3DSwapChainImpl IWineD3DSwapChainImpl; +struct IWineD3DBaseShaderImpl; /* Texture format fixups */ @@ -758,7 +759,7 @@ typedef struct { void (*shader_load_constants)(const struct wined3d_context *context, char usePS, char useVS); void (*shader_load_np2fixup_constants)(void *shader_priv, const struct wined3d_gl_info *gl_info, const struct wined3d_state *state); - void (*shader_destroy)(IWineD3DBaseShader *iface); + void (*shader_destroy)(struct IWineD3DBaseShaderImpl *shader); HRESULT (*shader_alloc_private)(IWineD3DDeviceImpl *device); void (*shader_free_private)(IWineD3DDeviceImpl *device); BOOL (*shader_dirtifyable_constants)(void);

1 0

Henri Verbeet : wined3d: Pass an IWineD3DBaseShaderImpl pointer to shader_generate_glsl_declarations ().
by Alexandre Julliard 13 Dec '10

13 Dec '10

Module: wine Branch: master Commit: 97ca0e6863d0c5fa6f3481744d27f41f76604495 URL: http://source.winehq.org/git/wine.git/?a=commit;h=97ca0e6863d0c5fa6f3481744… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Mon Dec 13 13:40:31 2010 +0100 wined3d: Pass an IWineD3DBaseShaderImpl pointer to shader_generate_glsl_declarations(). --- dlls/wined3d/glsl_shader.c | 52 +++++++++++++++++++++++-------------------- 1 files changed, 28 insertions(+), 24 deletions(-) diff --git a/dlls/wined3d/glsl_shader.c b/dlls/wined3d/glsl_shader.c index 19f0cc4..a28e9ce 100644 --- a/dlls/wined3d/glsl_shader.c +++ b/dlls/wined3d/glsl_shader.c @@ -906,11 +906,10 @@ static unsigned int vec4_varyings(DWORD shader_major, const struct wined3d_gl_in /** Generate the variable & register declarations for the GLSL output target */ static void shader_generate_glsl_declarations(const struct wined3d_context *context, - struct wined3d_shader_buffer *buffer, IWineD3DBaseShader *iface, + struct wined3d_shader_buffer *buffer, IWineD3DBaseShaderImpl *shader, const struct wined3d_shader_reg_maps *reg_maps, struct shader_glsl_ctx_priv *ctx_priv) { - IWineD3DBaseShaderImpl* This = (IWineD3DBaseShaderImpl*) iface; - IWineD3DDeviceImpl *device = This->baseShader.device; + IWineD3DDeviceImpl *device = shader->baseShader.device; const struct wined3d_state *state = &device->stateBlock->state; const struct ps_compile_args *ps_args = ctx_priv->cur_ps_args; const struct wined3d_gl_info *gl_info = context->gl_info; @@ -929,7 +928,8 @@ static void shader_generate_glsl_declarations(const struct wined3d_context *cont } /* Declare the constants (aka uniforms) */ - if (This->baseShader.limits.constant_float > 0) { + if (shader->baseShader.limits.constant_float > 0) + { unsigned max_constantsF; /* Unless the shader uses indirect addressing, always declare the maximum array size and ignore that we need some * uniforms privately. E.g. if GL supports 256 uniforms, and we need 2 for the pos fixup and immediate values, still @@ -982,18 +982,18 @@ static void shader_generate_glsl_declarations(const struct wined3d_context *cont max_constantsF = gl_info->limits.glsl_vs_float_constants; } } - max_constantsF = min(This->baseShader.limits.constant_float, max_constantsF); + max_constantsF = min(shader->baseShader.limits.constant_float, max_constantsF); shader_addline(buffer, "uniform vec4 %cC[%u];\n", prefix, max_constantsF); } - /* Always declare the full set of constants, the compiler can remove the unused ones because d3d doesn't(yet) - * support indirect int and bool constant addressing. This avoids problems if the app uses e.g. i0 and i9. - */ - if (This->baseShader.limits.constant_int > 0 && reg_maps->integer_constants) - shader_addline(buffer, "uniform ivec4 %cI[%u];\n", prefix, This->baseShader.limits.constant_int); + /* Always declare the full set of constants, the compiler can remove the + * unused ones because d3d doesn't (yet) support indirect int and bool + * constant addressing. This avoids problems if the app uses e.g. i0 and i9. */ + if (shader->baseShader.limits.constant_int > 0 && reg_maps->integer_constants) + shader_addline(buffer, "uniform ivec4 %cI[%u];\n", prefix, shader->baseShader.limits.constant_int); - if (This->baseShader.limits.constant_bool > 0 && reg_maps->boolean_constants) - shader_addline(buffer, "uniform bool %cB[%u];\n", prefix, This->baseShader.limits.constant_bool); + if (shader->baseShader.limits.constant_bool > 0 && reg_maps->boolean_constants) + shader_addline(buffer, "uniform bool %cB[%u];\n", prefix, shader->baseShader.limits.constant_bool); if (!pshader) { @@ -1027,11 +1027,11 @@ static void shader_generate_glsl_declarations(const struct wined3d_context *cont } if (reg_maps->vpos || reg_maps->usesdsy) { - if (This->baseShader.limits.constant_float + extra_constants_needed + if (shader->baseShader.limits.constant_float + extra_constants_needed + 1 < gl_info->limits.glsl_ps_float_constants) { shader_addline(buffer, "uniform vec4 ycorrection;\n"); - ((IWineD3DPixelShaderImpl *) This)->vpos_uniform = 1; + ((IWineD3DPixelShaderImpl *)shader)->vpos_uniform = 1; extra_constants_needed++; } else { /* This happens because we do not have proper tracking of the constant registers that are @@ -1047,7 +1047,8 @@ static void shader_generate_glsl_declarations(const struct wined3d_context *cont } /* Declare texture samplers */ - for (i = 0; i < This->baseShader.limits.sampler; i++) { + for (i = 0; i < shader->baseShader.limits.sampler; ++i) + { if (reg_maps->sampler_type[i]) { IWineD3DBaseTextureImpl *texture; @@ -1107,8 +1108,10 @@ static void shader_generate_glsl_declarations(const struct wined3d_context *cont * samplerNP2Fixup stores texture dimensions and is updated through * shader_glsl_load_np2fixup_constants when the sampler changes. */ - for (i = 0; i < This->baseShader.limits.sampler; ++i) { - if (reg_maps->sampler_type[i]) { + for (i = 0; i < shader->baseShader.limits.sampler; ++i) + { + if (reg_maps->sampler_type[i]) + { if (!(ps_args->np2_fixup & (1 << i))) continue; if (WINED3DSTT_2D != reg_maps->sampler_type[i]) { @@ -1154,9 +1157,8 @@ static void shader_generate_glsl_declarations(const struct wined3d_context *cont } /* Declare output register temporaries */ - if(This->baseShader.limits.packed_output) { - shader_addline(buffer, "vec4 OUT[%u];\n", This->baseShader.limits.packed_output); - } + if (shader->baseShader.limits.packed_output) + shader_addline(buffer, "vec4 OUT[%u];\n", shader->baseShader.limits.packed_output); /* Declare temporary variables */ for (i = 0, map = reg_maps->temporary; map; map >>= 1, ++i) @@ -1187,8 +1189,10 @@ static void shader_generate_glsl_declarations(const struct wined3d_context *cont * They can't be hardcoded into the shader text via LC = {x, y, z, w}; because the * float -> string conversion can cause precision loss. */ - if(!This->baseShader.load_local_constsF) { - LIST_FOR_EACH_ENTRY(lconst, &This->baseShader.constantsF, local_constant, entry) { + if (!shader->baseShader.load_local_constsF) + { + LIST_FOR_EACH_ENTRY(lconst, &shader->baseShader.constantsF, local_constant, entry) + { shader_addline(buffer, "uniform vec4 %cLC%u;\n", prefix, lconst->idx); } } @@ -3991,7 +3995,7 @@ static GLuint shader_glsl_generate_pshader(const struct wined3d_context *context } /* Base Declarations */ - shader_generate_glsl_declarations(context, buffer, (IWineD3DBaseShader *)This, reg_maps, &priv_ctx); + shader_generate_glsl_declarations(context, buffer, (IWineD3DBaseShaderImpl *)This, reg_maps, &priv_ctx); /* Pack 3.0 inputs */ if (reg_maps->shader_version.major >= 3 && args->vp_mode != vertexshader) @@ -4083,7 +4087,7 @@ static GLuint shader_glsl_generate_vshader(const struct wined3d_context *context priv_ctx.cur_vs_args = args; /* Base Declarations */ - shader_generate_glsl_declarations(context, buffer, (IWineD3DBaseShader *)This, reg_maps, &priv_ctx); + shader_generate_glsl_declarations(context, buffer, (IWineD3DBaseShaderImpl *)This, reg_maps, &priv_ctx); /* Base Shader Body */ shader_generate_main((IWineD3DBaseShader*)This, buffer, reg_maps, function, &priv_ctx);

1 0

Henri Verbeet : wined3d: Pass an IWineD3DBaseShaderImpl pointer to shader_generate_arb_declarations ().
by Alexandre Julliard 13 Dec '10

13 Dec '10

Module: wine Branch: master Commit: 8ea7d1ef71af870235eea4a474e30d7f85243a70 URL: http://source.winehq.org/git/wine.git/?a=commit;h=8ea7d1ef71af870235eea4a47… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Mon Dec 13 13:40:30 2010 +0100 wined3d: Pass an IWineD3DBaseShaderImpl pointer to shader_generate_arb_declarations(). --- dlls/wined3d/arb_program_shader.c | 21 +++++++++++---------- 1 files changed, 11 insertions(+), 10 deletions(-) diff --git a/dlls/wined3d/arb_program_shader.c b/dlls/wined3d/arb_program_shader.c index 6c10fa6..bd5b197 100644 --- a/dlls/wined3d/arb_program_shader.c +++ b/dlls/wined3d/arb_program_shader.c @@ -694,12 +694,11 @@ static DWORD *local_const_mapping(IWineD3DBaseShaderImpl *This) } /* Generate the variable & register declarations for the ARB_vertex_program output target */ -static DWORD shader_generate_arb_declarations(IWineD3DBaseShader *iface, +static DWORD shader_generate_arb_declarations(IWineD3DBaseShaderImpl *shader, const struct wined3d_shader_reg_maps *reg_maps, struct wined3d_shader_buffer *buffer, const struct wined3d_gl_info *gl_info, DWORD *lconst_map, DWORD *num_clipplanes, struct shader_arb_ctx_priv *ctx) { - IWineD3DBaseShaderImpl* This = (IWineD3DBaseShaderImpl*) iface; DWORD i, next_local = 0; char pshader = shader_is_pshader_version(reg_maps->shader_version.type); unsigned max_constantsF; @@ -725,7 +724,7 @@ static DWORD shader_generate_arb_declarations(IWineD3DBaseShader *iface, } else { - const struct arb_vshader_private *shader_data = This->baseShader.backend_data; + const struct arb_vshader_private *shader_data = shader->baseShader.backend_data; max_constantsF = gl_info->limits.arb_vs_native_constants; /* 96 is the minimum MAX_PROGRAM_ENV_PARAMETERS_ARB value. * Also prevents max_constantsF from becoming less than 0 and @@ -740,7 +739,7 @@ static DWORD shader_generate_arb_declarations(IWineD3DBaseShader *iface, max_constantsF -= reserved_vs_const(shader_data, reg_maps, gl_info); max_constantsF -= count_bits(reg_maps->integer_constants); - for(i = 0; i < This->baseShader.limits.constant_float; i++) + for (i = 0; i < shader->baseShader.limits.constant_float; ++i) { DWORD idx = i >> 5; DWORD shift = i & 0x1f; @@ -794,8 +793,10 @@ static DWORD shader_generate_arb_declarations(IWineD3DBaseShader *iface, /* Load local constants using the program-local space, * this avoids reloading them each time the shader is used */ - if(lconst_map) { - LIST_FOR_EACH_ENTRY(lconst, &This->baseShader.constantsF, local_constant, entry) { + if (lconst_map) + { + LIST_FOR_EACH_ENTRY(lconst, &shader->baseShader.constantsF, local_constant, entry) + { shader_addline(buffer, "PARAM C%u = program.local[%u];\n", lconst->idx, lconst_map[lconst->idx]); next_local = max(next_local, lconst_map[lconst->idx] + 1); @@ -815,7 +816,7 @@ static DWORD shader_generate_arb_declarations(IWineD3DBaseShader *iface, } /* Avoid declaring more constants than needed */ - max_constantsF = min(max_constantsF, This->baseShader.limits.constant_float); + max_constantsF = min(max_constantsF, shader->baseShader.limits.constant_float); /* we use the array-based constants array if the local constants are marked for loading, * because then we use indirect addressing, or when the local constant list is empty, @@ -833,7 +834,7 @@ static DWORD shader_generate_arb_declarations(IWineD3DBaseShader *iface, DWORD idx, mask; idx = i >> 5; mask = 1 << (i & 0x1f); - if (!shader_constant_is_local(This, i) && (reg_maps->constf[idx] & mask)) + if (!shader_constant_is_local(shader, i) && (reg_maps->constf[idx] & mask)) { shader_addline(buffer, "PARAM C%d = program.env[%d];\n",i, i); } @@ -3649,7 +3650,7 @@ static GLuint shader_arb_generate_pshader(IWineD3DPixelShaderImpl *This, } /* Base Declarations */ - next_local = shader_generate_arb_declarations((IWineD3DBaseShader *)This, + next_local = shader_generate_arb_declarations((IWineD3DBaseShaderImpl *)This, reg_maps, buffer, gl_info, lconst_map, NULL, &priv_ctx); for (i = 0, map = reg_maps->bumpmat; map; map >>= 1, ++i) @@ -4121,7 +4122,7 @@ static GLuint shader_arb_generate_vshader(IWineD3DVertexShaderImpl *This, shader_addline(buffer, "TEMP TA;\n"); /* Base Declarations */ - next_local = shader_generate_arb_declarations((IWineD3DBaseShader *)This, + next_local = shader_generate_arb_declarations((IWineD3DBaseShaderImpl *)This, reg_maps, buffer, gl_info, lconst_map, &priv_ctx.vs_clipplanes, &priv_ctx); for(i = 0; i < MAX_CONST_I; i++)

1 0

André Hentschel : msi: Fix test failure on some 2k3 machines.
by Alexandre Julliard 13 Dec '10

13 Dec '10

Module: wine Branch: master Commit: 6e9b8412160d75de5b78227c8c72b1b15094bdd2 URL: http://source.winehq.org/git/wine.git/?a=commit;h=6e9b8412160d75de5b78227c8… Author: André Hentschel <nerv(a)dawncrow.de> Date: Mon Dec 13 12:34:53 2010 +0100 msi: Fix test failure on some 2k3 machines. --- dlls/msi/tests/msi.c | 6 ++++++ 1 files changed, 6 insertions(+), 0 deletions(-) diff --git a/dlls/msi/tests/msi.c b/dlls/msi/tests/msi.c index e08f6bb..bb0d65f 100644 --- a/dlls/msi/tests/msi.c +++ b/dlls/msi/tests/msi.c @@ -7946,6 +7946,11 @@ static void test_MsiOpenProduct(void) /* LocalPackage has just the package name */ hprod = 0xdeadbeef; r = MsiOpenProductA(prodcode, &hprod); + if (r == ERROR_INSTALL_PACKAGE_REJECTED) + { + skip("Not enough rights to perform tests\n"); + goto error; + } ok(r == ERROR_INSTALL_PACKAGE_OPEN_FAILED || r == ERROR_SUCCESS, "Expected ERROR_INSTALL_PACKAGE_OPEN_FAILED or ERROR_SUCCESS, got %d\n", r); if (r == ERROR_SUCCESS) @@ -7968,6 +7973,7 @@ static void test_MsiOpenProduct(void) "Expected ERROR_UNKNOWN_PRODUCT, got %d\n", r); ok(hprod == 0xdeadbeef, "Expected hprod to be unchanged\n"); +error: RegDeleteValueA(props, "LocalPackage"); delete_key(props, "", access & KEY_WOW64_64KEY); RegCloseKey(props);

1 0

Eric Pouech : kernel32: In readline, now print control characters with ^ escape.
by Alexandre Julliard 13 Dec '10

13 Dec '10

Module: wine Branch: master Commit: 8aa3ec414d8ff4a5ebd7ca32fa171c267af2640e URL: http://source.winehq.org/git/wine.git/?a=commit;h=8aa3ec414d8ff4a5ebd7ca32f… Author: Eric Pouech <eric.pouech(a)orange.fr> Date: Sun Dec 12 22:31:49 2010 +0100 kernel32: In readline, now print control characters with ^ escape. --- dlls/kernel32/editline.c | 136 ++++++++++++++++++++++++++++++++++------------ 1 files changed, 101 insertions(+), 35 deletions(-) diff --git a/dlls/kernel32/editline.c b/dlls/kernel32/editline.c index 4dff110..b33798b 100644 --- a/dlls/kernel32/editline.c +++ b/dlls/kernel32/editline.c @@ -57,7 +57,7 @@ typedef struct WCEL_Context { (for consoles that can't change cursor pos) */ unsigned last_max; /* max number of chars written (for consoles that can't change cursor pos) */ - unsigned ofs; /* offset for cursor in current line */ + unsigned ofs; /* offset for cursor in current line */ WCHAR* yanked; /* yanked line */ unsigned mark; /* marked point (emacs mode only) */ CONSOLE_SCREEN_BUFFER_INFO csbi; /* current state (initial cursor, window size, attribute) */ @@ -111,10 +111,27 @@ static inline BOOL WCEL_IsSingleLine(WCEL_Context* ctx, size_t len) return ctx->csbi.dwCursorPosition.X + ctx->len + len <= ctx->csbi.dwSize.X; } -static inline COORD WCEL_GetCoord(WCEL_Context* ctx, int ofs) +static inline int WCEL_CharWidth(WCHAR wch) +{ + return wch < ' ' ? 2 : 1; +} + +static inline int WCEL_StringWidth(const WCHAR* str, int beg, int len) +{ + int i, ofs; + + for (i = 0, ofs = 0; i < len; i++) + ofs += WCEL_CharWidth(str[beg + i]); + return ofs; +} + +static inline COORD WCEL_GetCoord(WCEL_Context* ctx, int strofs) { COORD c; int len = ctx->csbi.dwSize.X - ctx->csbi.dwCursorPosition.X; + int ofs; + + ofs = WCEL_StringWidth(ctx->line, 0, strofs); c.Y = ctx->csbi.dwCursorPosition.Y; if (ofs >= len) @@ -127,35 +144,79 @@ static inline COORD WCEL_GetCoord(WCEL_Context* ctx, int ofs) return c; } +static DWORD WCEL_WriteConsole(WCEL_Context* ctx, DWORD beg, DWORD len) +{ + DWORD i, last, dw, ret = 0; + WCHAR tmp[2]; + + for (i = last = 0; i < len; i++) + { + if (ctx->line[beg + i] < ' ') + { + if (last != i) + { + WriteConsoleW(ctx->hConOut, &ctx->line[beg + last], i - last, &dw, NULL); + ret += dw; + } + tmp[0] = '^'; + tmp[1] = '@' + ctx->line[beg + i]; + WriteConsoleW(ctx->hConOut, tmp, 2, &dw, NULL); + last = i + 1; + ret += dw; + } + } + if (last != len) + { + WriteConsoleW(ctx->hConOut, &ctx->line[beg + last], len - last, &dw, NULL); + ret += dw; + } + return ret; +} + +static inline void WCEL_WriteNChars(WCEL_Context* ctx, char ch, int count) +{ + DWORD dw; + + if (count > 0) + { + while (count--) WriteFile(ctx->hConOut, &ch, 1, &dw, NULL); + } +} + static inline void WCEL_Update(WCEL_Context* ctx, int beg, int len) { - if (!ctx->shall_echo) return; - if (ctx->can_pos_cursor) + int i, last; + WCHAR tmp[2]; + + /* bare console case is handled in CONSOLE_ReadLine (we always reprint the whole string) */ + if (!ctx->shall_echo || !ctx->can_pos_cursor) return; + + for (i = last = beg; i < beg + len; i++) { - WriteConsoleOutputCharacterW(ctx->hConOut, &ctx->line[beg], len, - WCEL_GetCoord(ctx, beg), NULL); - FillConsoleOutputAttribute(ctx->hConOut, ctx->csbi.wAttributes, len, - WCEL_GetCoord(ctx, beg), NULL); + if (ctx->line[i] < ' ') + { + if (last != i) + { + WriteConsoleOutputCharacterW(ctx->hConOut, &ctx->line[last], i - last, + WCEL_GetCoord(ctx, last), NULL); + FillConsoleOutputAttribute(ctx->hConOut, ctx->csbi.wAttributes, i - last, + WCEL_GetCoord(ctx, last), NULL); + } + tmp[0] = '^'; + tmp[1] = '@' + ctx->line[i]; + WriteConsoleOutputCharacterW(ctx->hConOut, tmp, 2, + WCEL_GetCoord(ctx, i), NULL); + FillConsoleOutputAttribute(ctx->hConOut, ctx->csbi.wAttributes, 2, + WCEL_GetCoord(ctx, i), NULL); + last = i + 1; + } } - else + if (last != beg + len) { - char ch; - unsigned i; - DWORD dw; - - /* erase previous chars */ - ch = '\b'; - for (i = beg; i < ctx->last_rub; i++) - WriteFile(ctx->hConOut, &ch, 1, &dw, NULL); - beg = min(beg, ctx->last_rub); - - /* write new chars */ - WriteConsoleW(ctx->hConOut, &ctx->line[beg], ctx->len - beg, &dw, NULL); - /* clean rest of line (if any) */ - ch = ' '; - for (i = ctx->len; i < ctx->last_max; i++) - WriteFile(ctx->hConOut, &ch, 1, &dw, NULL); - ctx->last_rub = max(ctx->last_max, ctx->len); + WriteConsoleOutputCharacterW(ctx->hConOut, &ctx->line[last], i - last, + WCEL_GetCoord(ctx, last), NULL); + FillConsoleOutputAttribute(ctx->hConOut, ctx->csbi.wAttributes, i - last, + WCEL_GetCoord(ctx, last), NULL); } } @@ -887,18 +948,23 @@ WCHAR* CONSOLE_Readline(HANDLE hConsoleIn, BOOL can_pos_cursor) } else if (!ctx.done && !ctx.error) { - char ch; - unsigned i; - DWORD dw; - + DWORD last; /* erase previous chars */ - ch = '\b'; - for (i = 0; i < ctx.last_rub; i++) - WriteFile(ctx.hConOut, &ch, 1, &dw, NULL); + WCEL_WriteNChars(&ctx, '\b', ctx.last_rub); /* write chars up to cursor */ - WriteConsoleW(ctx.hConOut, ctx.line, ctx.ofs, &dw, NULL); - if ((ctx.last_rub = ctx.ofs) > ctx.last_max) ctx.last_max = ctx.ofs; + ctx.last_rub = WCEL_WriteConsole(&ctx, 0, ctx.ofs); + /* write chars past cursor */ + last = ctx.last_rub + WCEL_WriteConsole(&ctx, ctx.ofs, ctx.len - ctx.ofs); + if (last < ctx.last_max) /* ctx.line has been shortened, erase */ + { + WCEL_WriteNChars(&ctx, ' ', ctx.last_max - last); + WCEL_WriteNChars(&ctx, '\b', ctx.last_max - last); + ctx.last_max = last; + } + else ctx.last_max = last; + /* reposition at cursor */ + WCEL_WriteNChars(&ctx, '\b', last - ctx.last_rub); } } if (ctx.error)

1 0

Eric Pouech : kernel32: Proper support for echo-input (enable/disable) when getting console input.
by Alexandre Julliard 13 Dec '10

13 Dec '10

Module: wine Branch: master Commit: 8b4c2ce9c58a43c5501d3f21b0c8d7e01c8bc767 URL: http://source.winehq.org/git/wine.git/?a=commit;h=8b4c2ce9c58a43c5501d3f21b… Author: Eric Pouech <eric.pouech(a)orange.fr> Date: Sun Dec 12 22:31:40 2010 +0100 kernel32: Proper support for echo-input (enable/disable) when getting console input. --- dlls/kernel32/editline.c | 70 +++++++++++++++++++++++++++------------------- 1 files changed, 41 insertions(+), 29 deletions(-) diff --git a/dlls/kernel32/editline.c b/dlls/kernel32/editline.c index 4d02d77..4dff110 100644 --- a/dlls/kernel32/editline.c +++ b/dlls/kernel32/editline.c @@ -66,6 +66,7 @@ typedef struct WCEL_Context { unsigned done : 1, /* to 1 when we're done with editing */ error : 1, /* to 1 when an error occurred in the editing */ can_wrap : 1, /* to 1 when multi-line edition can take place */ + shall_echo : 1, /* to 1 when characters should be echo:ed when keyed-in */ can_pos_cursor : 1; /* to 1 when console can (re)position cursor */ unsigned histSize; unsigned histPos; @@ -128,6 +129,7 @@ static inline COORD WCEL_GetCoord(WCEL_Context* ctx, int ofs) static inline void WCEL_Update(WCEL_Context* ctx, int beg, int len) { + if (!ctx->shall_echo) return; if (ctx->can_pos_cursor) { WriteConsoleOutputCharacterW(ctx->hConOut, &ctx->line[beg], len, @@ -194,34 +196,38 @@ static BOOL WCEL_Grow(WCEL_Context* ctx, size_t len) static void WCEL_DeleteString(WCEL_Context* ctx, int beg, int end) { unsigned str_len = end - beg; - COORD cbeg = WCEL_GetCoord(ctx, ctx->len - str_len); - COORD cend = WCEL_GetCoord(ctx, ctx->len); - CHAR_INFO ci; if (end < ctx->len) memmove(&ctx->line[beg], &ctx->line[end], (ctx->len - end) * sizeof(WCHAR)); /* we need to clean from ctx->len - str_len to ctx->len */ - ci.Char.UnicodeChar = ' '; - ci.Attributes = ctx->csbi.wAttributes; - - if (cbeg.Y == cend.Y) - { - /* partial erase of sole line */ - CONSOLE_FillLineUniform(ctx->hConOut, cbeg.X, cbeg.Y, - cend.X - cbeg.X, &ci); - } - else + if (ctx->shall_echo) { - int i; - /* erase til eol on first line */ - CONSOLE_FillLineUniform(ctx->hConOut, cbeg.X, cbeg.Y, - ctx->csbi.dwSize.X - cbeg.X, &ci); - /* completely erase all the others (full lines) */ - for (i = cbeg.Y + 1; i < cend.Y; i++) - CONSOLE_FillLineUniform(ctx->hConOut, 0, i, ctx->csbi.dwSize.X, &ci); - /* erase from beginning of line until last pos on last line */ - CONSOLE_FillLineUniform(ctx->hConOut, 0, cend.Y, cend.X, &ci); + COORD cbeg = WCEL_GetCoord(ctx, ctx->len - str_len); + COORD cend = WCEL_GetCoord(ctx, ctx->len); + CHAR_INFO ci; + + ci.Char.UnicodeChar = ' '; + ci.Attributes = ctx->csbi.wAttributes; + + if (cbeg.Y == cend.Y) + { + /* partial erase of sole line */ + CONSOLE_FillLineUniform(ctx->hConOut, cbeg.X, cbeg.Y, + cend.X - cbeg.X, &ci); + } + else + { + int i; + /* erase til eol on first line */ + CONSOLE_FillLineUniform(ctx->hConOut, cbeg.X, cbeg.Y, + ctx->csbi.dwSize.X - cbeg.X, &ci); + /* completely erase all the others (full lines) */ + for (i = cbeg.Y + 1; i < cend.Y; i++) + CONSOLE_FillLineUniform(ctx->hConOut, 0, i, ctx->csbi.dwSize.X, &ci); + /* erase from beginning of line until last pos on last line */ + CONSOLE_FillLineUniform(ctx->hConOut, 0, cend.Y, cend.X, &ci); + } } ctx->len -= str_len; WCEL_Update(ctx, 0, ctx->len); @@ -378,7 +384,8 @@ static void WCEL_FindPrevInHist(WCEL_Context* ctx) ctx->ofs = 0; WCEL_InsertString(ctx, data); ctx->ofs = oldofs; - SetConsoleCursorPosition(ctx->hConOut, WCEL_GetCoord(ctx, ctx->ofs)); + if (ctx->shall_echo) + SetConsoleCursorPosition(ctx->hConOut, WCEL_GetCoord(ctx, ctx->ofs)); HeapFree(GetProcessHeap(), 0, data); return; } @@ -630,15 +637,18 @@ static void WCEL_MoveToLastHist(WCEL_Context* ctx) static void WCEL_Redraw(WCEL_Context* ctx) { - COORD c = WCEL_GetCoord(ctx, ctx->len); - CHAR_INFO ci; + if (ctx->shall_echo) + { + COORD c = WCEL_GetCoord(ctx, ctx->len); + CHAR_INFO ci; - WCEL_Update(ctx, 0, ctx->len); + WCEL_Update(ctx, 0, ctx->len); - ci.Char.UnicodeChar = ' '; - ci.Attributes = ctx->csbi.wAttributes; + ci.Char.UnicodeChar = ' '; + ci.Attributes = ctx->csbi.wAttributes; - CONSOLE_FillLineUniform(ctx->hConOut, c.X, c.Y, ctx->csbi.dwSize.X - c.X, &ci); + CONSOLE_FillLineUniform(ctx->hConOut, c.X, c.Y, ctx->csbi.dwSize.X - c.X, &ci); + } } static void WCEL_RepeatCount(WCEL_Context* ctx) @@ -812,6 +822,7 @@ WCHAR* CONSOLE_Readline(HANDLE hConsoleIn, BOOL can_pos_cursor) OPEN_EXISTING, 0, 0 )) == INVALID_HANDLE_VALUE || !GetConsoleScreenBufferInfo(ctx.hConOut, &ctx.csbi)) return NULL; + ctx.shall_echo = (GetConsoleMode(hConsoleIn, &ks) && (ks & ENABLE_ECHO_INPUT)) ? 1 : 0; ctx.can_wrap = (GetConsoleMode(ctx.hConOut, &ks) && (ks & ENABLE_WRAP_AT_EOL_OUTPUT)) ? 1 : 0; ctx.can_pos_cursor = can_pos_cursor; @@ -868,6 +879,7 @@ WCHAR* CONSOLE_Readline(HANDLE hConsoleIn, BOOL can_pos_cursor) else TRACE("Dropped event\n"); /* EPP WCEL_Dump(&ctx, "after func"); */ + if (!ctx.shall_echo) continue; if (ctx.can_pos_cursor) { if (ctx.ofs != ofs)

1 0

Eric Pouech : kernel32: When reading chars out of a bare console, use the Unix API (instead of the Windows one).
by Alexandre Julliard 13 Dec '10

13 Dec '10

Module: wine Branch: master Commit: 4149643fbe1b3c9c92810fc5cdfc10487e807311 URL: http://source.winehq.org/git/wine.git/?a=commit;h=4149643fbe1b3c9c92810fc5c… Author: Eric Pouech <eric.pouech(a)orange.fr> Date: Sun Dec 12 22:31:32 2010 +0100 kernel32: When reading chars out of a bare console, use the Unix API (instead of the Windows one). --- dlls/kernel32/console.c | 56 ++++++++++++++++++++++++++--------------------- 1 files changed, 31 insertions(+), 25 deletions(-) diff --git a/dlls/kernel32/console.c b/dlls/kernel32/console.c index c0af117..97eae17 100644 --- a/dlls/kernel32/console.c +++ b/dlls/kernel32/console.c @@ -41,6 +41,9 @@ #ifdef HAVE_TERMIOS_H # include <termios.h> #endif +#ifdef HAVE_SYS_POLL_H +# include <sys/poll.h> +#endif #include "ntstatus.h" #define WIN32_NO_STATUS @@ -1112,33 +1115,35 @@ static BOOL handle_simple_char(HANDLE conin, unsigned real_inchar) return WriteConsoleInputW(conin, ir, numEvent, &written); } -static enum read_console_input_return bare_console_fetch_input(HANDLE handle, DWORD timeout) +static enum read_console_input_return bare_console_fetch_input(HANDLE handle, int fd, DWORD timeout) { - OVERLAPPED ov; - enum read_console_input_return ret; - char ch; - - /* get the real handle to the console object */ - handle = wine_server_ptr_handle(console_handle_unmap(handle)); + struct pollfd pollfd; + char ch; + enum read_console_input_return ret; - memset(&ov, 0, sizeof(ov)); - ov.hEvent = CreateEventW(NULL, FALSE, FALSE, NULL); + pollfd.fd = fd; + pollfd.events = POLLIN; + pollfd.revents = 0; - if (ReadFile(handle, &ch, 1, NULL, &ov) || - (GetLastError() == ERROR_IO_PENDING && - WaitForSingleObject(ov.hEvent, timeout) == WAIT_OBJECT_0 && - GetOverlappedResult(handle, &ov, NULL, FALSE))) + switch (poll(&pollfd, 1, timeout)) { - ret = handle_simple_char(handle, ch) ? rci_gotone : rci_error; - } - else - { - WARN("Failed read %x\n", GetLastError()); - ret = rci_error; + case 1: + RtlEnterCriticalSection(&CONSOLE_CritSect); + switch (read(fd, &ch, 1)) + { + case 1: ret = handle_simple_char(handle, ch) ? rci_gotone : rci_error; break; + /* actually another thread likely beat us to reading the char + * return gotone, while not perfect, it should work in most of the cases (as the new event + * should be now in the queue) + */ + case 0: ret = rci_gotone; break; + default: ret = rci_error; break; + } + RtlLeaveCriticalSection(&CONSOLE_CritSect); + return ret; + case 0: return rci_timeout; + default: return rci_error; } - CloseHandle(ov.hEvent); - - return ret; } static enum read_console_input_return read_console_input(HANDLE handle, PINPUT_RECORD ir, DWORD timeout) @@ -1149,12 +1154,13 @@ static enum read_console_input_return read_console_input(HANDLE handle, PINPUT_R if ((fd = get_console_bare_fd(handle)) != -1) { put_console_into_raw_mode(fd); - close(fd); if (WaitForSingleObject(GetConsoleInputWaitHandle(), 0) != WAIT_OBJECT_0) { - ret = bare_console_fetch_input(handle, timeout); - if (ret != rci_gotone) return ret; + ret = bare_console_fetch_input(handle, fd, timeout); } + else ret = rci_gotone; + close(fd); + if (ret != rci_gotone) return ret; } else {

1 0

Alexandre Julliard : winex11: Fix event coordinates mapping for the root window in the explorer process.
by Alexandre Julliard 13 Dec '10

13 Dec '10

Module: wine Branch: master Commit: 10aa13a98e4346e8af5e95a20eaefec7223a8c76 URL: http://source.winehq.org/git/wine.git/?a=commit;h=10aa13a98e4346e8af5e95a20… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Mon Dec 13 12:42:47 2010 +0100 winex11: Fix event coordinates mapping for the root window in the explorer process. --- dlls/winex11.drv/event.c | 1 + dlls/winex11.drv/mouse.c | 5 +++++ 2 files changed, 6 insertions(+), 0 deletions(-) diff --git a/dlls/winex11.drv/event.c b/dlls/winex11.drv/event.c index 5447a40..be3033b 100644 --- a/dlls/winex11.drv/event.c +++ b/dlls/winex11.drv/event.c @@ -788,6 +788,7 @@ static void X11DRV_Expose( HWND hwnd, XEvent *xev ) flags |= RDW_ALLCHILDREN; } + else OffsetRect( &rect, virtual_screen_rect.left, virtual_screen_rect.top ); RedrawWindow( hwnd, &rect, 0, flags ); } diff --git a/dlls/winex11.drv/mouse.c b/dlls/winex11.drv/mouse.c index 71b31cc..a763798 100644 --- a/dlls/winex11.drv/mouse.c +++ b/dlls/winex11.drv/mouse.c @@ -239,6 +239,11 @@ static HWND update_mouse_state( HWND hwnd, Window window, int x, int y, unsigned x += data->whole_rect.left - data->client_rect.left; y += data->whole_rect.top - data->client_rect.top; } + if (window == root_window) + { + x += virtual_screen_rect.left; + y += virtual_screen_rect.top; + } pt->x = x; pt->y = y; if (GetWindowLongW( data->hwnd, GWL_EXSTYLE ) & WS_EX_LAYOUTRTL)

1 0

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

2003

2002

2001

wine-commits December 2010