May 2010 - Wine-commits - list.winehq.org

Matteo Bruni : d3dx9: Add missing texkill instruction parsing.
by Alexandre Julliard 21 May '10

21 May '10

Module: wine Branch: master Commit: 1d3f462fefb687b8719d1ffcab71073dbf772ae0 URL: http://source.winehq.org/git/wine.git/?a=commit;h=1d3f462fefb687b8719d1ffca… Author: Matteo Bruni <matteo.mystral(a)gmail.com> Date: Thu May 20 01:21:40 2010 +0200 d3dx9: Add missing texkill instruction parsing. --- dlls/d3dx9_36/asmshader.y | 9 ++++++--- dlls/d3dx9_36/tests/asm.c | 5 +++++ 2 files changed, 11 insertions(+), 3 deletions(-) diff --git a/dlls/d3dx9_36/asmshader.y b/dlls/d3dx9_36/asmshader.y index 06a6825..0949ee3 100644 --- a/dlls/d3dx9_36/asmshader.y +++ b/dlls/d3dx9_36/asmshader.y @@ -705,6 +705,11 @@ instruction: INSTR_ADD omods dreg ',' sregs TRACE("DP2ADD\n"); asm_ctx.funcs->instr(&asm_ctx, BWRITERSIO_DP2ADD, $2.mod, $2.shift, 0, &$3, &$5, 3); } + | INSTR_TEXKILL dreg + { + TRACE("TEXKILL\n"); + asm_ctx.funcs->instr(&asm_ctx, BWRITERSIO_TEXKILL, 0, 0, 0, &$2, 0, 0); + } | INSTR_TEXLD omods dreg ',' sregs { TRACE("TEXLD\n"); @@ -769,9 +774,7 @@ dreg_name: REG_TEMP } | REG_INPUT { - asmparser_message(&asm_ctx, "Line %u: Register v%u is not a valid destination register\n", - asm_ctx.line_no, $1); - set_parse_status(&asm_ctx, PARSE_WARN); + $$.regnum = $1; $$.type = BWRITERSPR_INPUT; } | REG_CONSTFLOAT { diff --git a/dlls/d3dx9_36/tests/asm.c b/dlls/d3dx9_36/tests/asm.c index 54894df..79c6c07 100644 --- a/dlls/d3dx9_36/tests/asm.c +++ b/dlls/d3dx9_36/tests/asm.c @@ -1141,6 +1141,11 @@ static void ps_3_0_test(void) { {0xffff0300, 0x0500005d, 0x802f0000, 0x80e40001, 0x80e40002, 0x80e40003, 0x80e40004, 0x0000ffff} }, + { /* shader 10 */ + "ps_3_0\n" + "texkill v0\n", + {0xffff0300, 0x01000041, 0x900f0000, 0x0000ffff} + }, }; exec_tests("ps_3_0", tests, sizeof(tests) / sizeof(tests[0]));

1 0

Matteo Bruni : d3dx9: Use separate opcodes for TEXLDP and TEXLDB.
by Alexandre Julliard 21 May '10

21 May '10

Module: wine Branch: master Commit: a9118ba1715c3a4833d3e45f19a69d418e8417bb URL: http://source.winehq.org/git/wine.git/?a=commit;h=a9118ba1715c3a4833d3e45f1… Author: Matteo Bruni <matteo.mystral(a)gmail.com> Date: Fri May 21 14:56:43 2010 +0200 d3dx9: Use separate opcodes for TEXLDP and TEXLDB. There is no reason we have to strictly follow D3D bytecode format in our intermediate representation. --- dlls/d3dx9_36/asmshader.y | 4 ++-- dlls/d3dx9_36/asmutils.c | 8 ++++---- dlls/d3dx9_36/bytecodewriter.c | 4 ++-- dlls/d3dx9_36/d3dx9_36_private.h | 5 ++--- 4 files changed, 10 insertions(+), 11 deletions(-) diff --git a/dlls/d3dx9_36/asmshader.y b/dlls/d3dx9_36/asmshader.y index 0949ee3..8be8328 100644 --- a/dlls/d3dx9_36/asmshader.y +++ b/dlls/d3dx9_36/asmshader.y @@ -724,12 +724,12 @@ instruction: INSTR_ADD omods dreg ',' sregs | INSTR_TEXLDP omods dreg ',' sregs { TRACE("TEXLDP\n"); - asm_ctx.funcs->instr(&asm_ctx, BWRITERSIO_TEX | ( BWRITERSI_TEXLD_PROJECT << BWRITER_OPCODESPECIFICCONTROL_SHIFT ), $2.mod, $2.shift, 0, &$3, &$5, 2); + asm_ctx.funcs->instr(&asm_ctx, BWRITERSIO_TEXLDP, $2.mod, $2.shift, 0, &$3, &$5, 2); } | INSTR_TEXLDB omods dreg ',' sregs { TRACE("TEXLDB\n"); - asm_ctx.funcs->instr(&asm_ctx, BWRITERSIO_TEX | ( BWRITERSI_TEXLD_BIAS << BWRITER_OPCODESPECIFICCONTROL_SHIFT ), $2.mod, $2.shift, 0, &$3, &$5, 2); + asm_ctx.funcs->instr(&asm_ctx, BWRITERSIO_TEXLDB, $2.mod, $2.shift, 0, &$3, &$5, 2); } | INSTR_DSX omods dreg ',' sregs { diff --git a/dlls/d3dx9_36/asmutils.c b/dlls/d3dx9_36/asmutils.c index e000a5f..bd6bcea 100644 --- a/dlls/d3dx9_36/asmutils.c +++ b/dlls/d3dx9_36/asmutils.c @@ -209,8 +209,8 @@ DWORD d3d9_opcode(DWORD bwriter_opcode) { case BWRITERSIO_COMMENT: return D3DSIO_COMMENT; case BWRITERSIO_END: return D3DSIO_END; - case BWRITERSIO_TEX | ( BWRITERSI_TEXLD_PROJECT << BWRITER_OPCODESPECIFICCONTROL_SHIFT ): return D3DSIO_TEX | D3DSI_TEXLD_PROJECT; - case BWRITERSIO_TEX | ( BWRITERSI_TEXLD_BIAS << BWRITER_OPCODESPECIFICCONTROL_SHIFT ): return D3DSIO_TEX | D3DSI_TEXLD_BIAS; + case BWRITERSIO_TEXLDP: return D3DSIO_TEX | D3DSI_TEXLD_PROJECT; + case BWRITERSIO_TEXLDB: return D3DSIO_TEX | D3DSI_TEXLD_BIAS; default: FIXME("Unhandled BWRITERSIO token %u\n", bwriter_opcode); @@ -484,8 +484,8 @@ const char *debug_print_opcode(DWORD opcode) { case BWRITERSIO_TEXLDL: return "texldl"; case BWRITERSIO_BREAKP: return "breakp"; - case BWRITERSIO_TEX | ( BWRITERSI_TEXLD_PROJECT << BWRITER_OPCODESPECIFICCONTROL_SHIFT ): return "texldp"; - case BWRITERSIO_TEX | ( BWRITERSI_TEXLD_BIAS << BWRITER_OPCODESPECIFICCONTROL_SHIFT ): return "texldb"; + case BWRITERSIO_TEXLDP: return "texldp"; + case BWRITERSIO_TEXLDB: return "texldb"; default: return "unknown"; } diff --git a/dlls/d3dx9_36/bytecodewriter.c b/dlls/d3dx9_36/bytecodewriter.c index 464ec52..a579ae8 100644 --- a/dlls/d3dx9_36/bytecodewriter.c +++ b/dlls/d3dx9_36/bytecodewriter.c @@ -693,8 +693,8 @@ static const struct instr_handler_table ps_3_handlers[] = { {BWRITERSIO_TEXLDL, instr_handler}, {BWRITERSIO_TEX, instr_handler}, - {BWRITERSIO_TEX | ( BWRITERSI_TEXLD_PROJECT << BWRITER_OPCODESPECIFICCONTROL_SHIFT ), instr_handler}, - {BWRITERSIO_TEX | ( BWRITERSI_TEXLD_BIAS << BWRITER_OPCODESPECIFICCONTROL_SHIFT ), instr_handler}, + {BWRITERSIO_TEXLDP, instr_handler}, + {BWRITERSIO_TEXLDB, instr_handler}, {BWRITERSIO_TEXKILL, instr_handler}, {BWRITERSIO_DSX, instr_handler}, {BWRITERSIO_DSY, instr_handler}, diff --git a/dlls/d3dx9_36/d3dx9_36_private.h b/dlls/d3dx9_36/d3dx9_36_private.h index 923a0df..248503d 100644 --- a/dlls/d3dx9_36/d3dx9_36_private.h +++ b/dlls/d3dx9_36/d3dx9_36_private.h @@ -462,6 +462,8 @@ typedef enum _BWRITERSHADER_INSTRUCTION_OPCODE_TYPE { BWRITERSIO_SETP, BWRITERSIO_TEXLDL, BWRITERSIO_BREAKP, + BWRITERSIO_TEXLDP, + BWRITERSIO_TEXLDB, BWRITERSIO_COMMENT, BWRITERSIO_END, @@ -589,9 +591,6 @@ typedef enum _BWRITERDECLUSAGE { BWRITERDECLUSAGE_SAMPLE } BWRITERDECLUSAGE; -#define BWRITER_OPCODESPECIFICCONTROL_SHIFT 16 -#define BWRITER_OPCODESPECIFICCONTROL_MASK (0xff << BWRITER_OPCODESPECIFICCONTROL_SHIFT) - struct bwriter_shader *SlAssembleShader(const char *text, char **messages); DWORD SlWriteBytecode(const struct bwriter_shader *shader, int dxversion, DWORD **result); void SlDeleteShader(struct bwriter_shader *shader);

1 0

Matteo Bruni : d3dx9: Check for oCx register number.
by Alexandre Julliard 21 May '10

21 May '10

Module: wine Branch: master Commit: 961db7964d89e7650a5618a53c5a3cf94b28515a URL: http://source.winehq.org/git/wine.git/?a=commit;h=961db7964d89e7650a5618a53… Author: Matteo Bruni <matteo.mystral(a)gmail.com> Date: Fri May 21 15:53:44 2010 +0200 d3dx9: Check for oCx register number. --- dlls/d3dx9_36/asmparser.c | 2 +- dlls/d3dx9_36/tests/asm.c | 11 +++++++++++ 2 files changed, 12 insertions(+), 1 deletions(-) diff --git a/dlls/d3dx9_36/asmparser.c b/dlls/d3dx9_36/asmparser.c index 3062639..dfd0936 100644 --- a/dlls/d3dx9_36/asmparser.c +++ b/dlls/d3dx9_36/asmparser.c @@ -259,7 +259,7 @@ static const struct allowed_reg_type ps_3_reg_allowed[] = { { BWRITERSPR_MISCTYPE, 2 }, /* vPos and vFace */ { BWRITERSPR_LOOP, 1 }, { BWRITERSPR_LABEL, 2048 }, - { BWRITERSPR_COLOROUT, ~0U }, + { BWRITERSPR_COLOROUT, 4 }, { BWRITERSPR_DEPTHOUT, 1 }, { ~0U, 0 } /* End tag */ }; diff --git a/dlls/d3dx9_36/tests/asm.c b/dlls/d3dx9_36/tests/asm.c index 79c6c07..0b8cd3e 100644 --- a/dlls/d3dx9_36/tests/asm.c +++ b/dlls/d3dx9_36/tests/asm.c @@ -1146,6 +1146,11 @@ static void ps_3_0_test(void) { "texkill v0\n", {0xffff0300, 0x01000041, 0x900f0000, 0x0000ffff} }, + { /* shader 11 */ + "ps_3_0\n" + "add oC3, r0, r1\n", + {0xffff0300, 0x03000002, 0x800f0803, 0x80e40000, 0x80e40001, 0x0000ffff} + }, }; exec_tests("ps_3_0", tests, sizeof(tests) / sizeof(tests[0])); @@ -1236,6 +1241,12 @@ static void failure_test(void) { /* shader 25: _x4 instruction modified not allowed in > ps_1_x */ "ps_3_0\n" "add_x4 r0, r0, r1\n", + /* shader 26: there aren't oCx registers in ps_1_x */ + "ps_1_3\n" + "add oC0, r0, r1\n", + /* shader 27: oC3 is the max in >= ps_2_0 */ + "ps_3_0\n" + "add oC4, r0, r1\n", }; HRESULT hr; unsigned int i;

1 0

Matteo Bruni : d3dx9: Shader assembler vs_2_x support.
by Alexandre Julliard 21 May '10

21 May '10

Module: wine Branch: master Commit: 5050f7c8d2cbd965c15501f73d199ee989ef452e URL: http://source.winehq.org/git/wine.git/?a=commit;h=5050f7c8d2cbd965c15501f73… Author: Matteo Bruni <matteo.mystral(a)gmail.com> Date: Sun May 16 20:57:52 2010 +0200 d3dx9: Shader assembler vs_2_x support. --- dlls/d3dx9_36/asmparser.c | 259 +++++++++++++++++++++ dlls/d3dx9_36/asmshader.y | 6 +- dlls/d3dx9_36/bytecodewriter.c | 473 +++++++++++++++++++++++++++++++++++++- dlls/d3dx9_36/d3dx9_36_private.h | 11 + dlls/d3dx9_36/tests/asm.c | 12 +- 5 files changed, 753 insertions(+), 8 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=5050f7c8d2cbd965c1550…

1 0

Damjan Jovanovic : winemenubuilder: Generate .png even for paletted icons.
by Alexandre Julliard 21 May '10

21 May '10

Module: wine Branch: master Commit: a83a9c1da143ceec7a4189501431b35f178ec78a URL: http://source.winehq.org/git/wine.git/?a=commit;h=a83a9c1da143ceec7a4189501… Author: Damjan Jovanovic <damjan.jov(a)gmail.com> Date: Fri May 21 07:44:22 2010 +0200 winemenubuilder: Generate .png even for paletted icons. --- programs/winemenubuilder/winemenubuilder.c | 146 +++++++++++++++++++++++++++- 1 files changed, 145 insertions(+), 1 deletions(-) diff --git a/programs/winemenubuilder/winemenubuilder.c b/programs/winemenubuilder/winemenubuilder.c index 9f10fdb..eba21ef 100644 --- a/programs/winemenubuilder/winemenubuilder.c +++ b/programs/winemenubuilder/winemenubuilder.c @@ -242,7 +242,7 @@ static void user_warning_fn(png_structp png_ptr, png_const_charp warning_message WINE_WARN("PNG warning: %s\n", wine_dbgstr_an(warning_message, -1)); } -static BOOL SaveIconResAsPNG(const BITMAPINFO *pIcon, const char *png_filename, LPCWSTR commentW) +static BOOL SaveTrueColorIconResAsPNG(const BITMAPINFO *pIcon, const char *png_filename, LPCWSTR commentW) { static const char comment_key[] = "Created from"; FILE *fp; @@ -373,6 +373,150 @@ static BOOL SaveIconResAsPNG(const BITMAPINFO *pIcon, const char *png_filename, HeapFree(GetProcessHeap(), 0, comment.text); return FALSE; } + +static BOOL SavePalettedIconResAsPNG(const BITMAPINFO *pIcon, const char *png_filename, LPCWSTR commentW) +{ + static const char comment_key[] = "Created from"; + FILE *pngFile = NULL; + int i, j; + int nHeight; + int nXORWidthBytes; + int nANDWidthBytes; + BOOL b8BitColors; + int nColors; + const BYTE *pXOR; + const BYTE *pAND; + png_structp png_ptr; + png_infop info_ptr; + png_text comment; + jmp_buf jmpbuf; + unsigned char *row = NULL; + BOOL ret = FALSE; + + comment.text = NULL; + + if (!((pIcon->bmiHeader.biBitCount == 4) || (pIcon->bmiHeader.biBitCount == 8))) + { + WINE_FIXME("Unsupported color depth %d-bit\n", pIcon->bmiHeader.biBitCount); + goto done; + } + + if (!(pngFile = fopen(png_filename, "w"))) + { + WINE_TRACE("Unable to open '%s' for writing: %s\n", png_filename, strerror(errno)); + goto done; + } + + nHeight = pIcon->bmiHeader.biHeight / 2; + nXORWidthBytes = 4 * ((pIcon->bmiHeader.biWidth * pIcon->bmiHeader.biBitCount / 32) + + ((pIcon->bmiHeader.biWidth * pIcon->bmiHeader.biBitCount % 32) > 0)); + nANDWidthBytes = 4 * ((pIcon->bmiHeader.biWidth / 32) + + ((pIcon->bmiHeader.biWidth % 32) > 0)); + b8BitColors = pIcon->bmiHeader.biBitCount == 8; + nColors = pIcon->bmiHeader.biClrUsed ? pIcon->bmiHeader.biClrUsed + : 1 << pIcon->bmiHeader.biBitCount; + pXOR = (const BYTE*) pIcon + sizeof (BITMAPINFOHEADER) + (nColors * sizeof (RGBQUAD)); + pAND = pXOR + nHeight * nXORWidthBytes; + + row = HeapAlloc(GetProcessHeap(), 0, 4 * pIcon->bmiHeader.biWidth); + if (row == NULL) + { + WINE_ERR("out of memory\n"); + goto done; + } + + if (!libpng_handle && !load_libpng()) + { + WINE_WARN("Unable to load libpng\n"); + goto done; + } + + if (!(png_ptr = ppng_create_write_struct(PNG_LIBPNG_VER_STRING, NULL, NULL, NULL)) || + !(info_ptr = ppng_create_info_struct(png_ptr))) + goto done; + + if (setjmp(jmpbuf)) + { + /* All future errors jump here */ + goto done; + } + ppng_set_error_fn(png_ptr, &jmpbuf, user_error_fn, user_warning_fn); + + ppng_init_io(png_ptr, pngFile); + ppng_set_IHDR(png_ptr, info_ptr, pIcon->bmiHeader.biWidth, nHeight, 8, + PNG_COLOR_TYPE_RGB_ALPHA, + PNG_INTERLACE_NONE, + PNG_COMPRESSION_TYPE_DEFAULT, + PNG_FILTER_TYPE_DEFAULT); + + /* Set comment */ + comment.compression = PNG_TEXT_COMPRESSION_NONE; + comment.key = (png_charp)comment_key; + i = WideCharToMultiByte(CP_UNIXCP, 0, commentW, -1, NULL, 0, NULL, NULL); + comment.text = HeapAlloc(GetProcessHeap(), 0, i); + if (comment.text == NULL) + { + WINE_ERR("out of memory\n"); + goto done; + } + WideCharToMultiByte(CP_UNIXCP, 0, commentW, -1, comment.text, i, NULL, NULL); + comment.text_length = i - 1; + ppng_set_text(png_ptr, info_ptr, &comment, 1); + + ppng_write_info(png_ptr, info_ptr); + for (i = 0; i < nHeight; i++) + { + unsigned char *p = row; + for (j = 0; j < pIcon->bmiHeader.biWidth; j++) + { + if MASK(j,i) + { + *p++ = 0; + *p++ = 0; + *p++ = 0; + *p++ = 0; + } + else + { +#define COLOR(x,y) (b8BitColors ? pXOR[(x) + (nHeight - (y) - 1) * nXORWidthBytes] : (x) % 2 ? pXOR[(x) / 2 + (nHeight - (y) - 1) * nXORWidthBytes] & 0xF : (pXOR[(x) / 2 + (nHeight - (y) - 1) * nXORWidthBytes] & 0xF0) >> 4) + *p++ = pIcon->bmiColors[COLOR(j,i)].rgbRed; + *p++ = pIcon->bmiColors[COLOR(j,i)].rgbGreen; + *p++ = pIcon->bmiColors[COLOR(j,i)].rgbBlue; + *p++ = 0xFF; +#undef COLOR + } + } + ppng_write_row(png_ptr, (png_bytep)row); + } + ppng_write_end(png_ptr, info_ptr); + ret = TRUE; + +done: + ppng_destroy_write_struct(&png_ptr, &info_ptr); + if (pngFile != NULL) + fclose(pngFile); + HeapFree(GetProcessHeap(), 0, comment.text); + HeapFree(GetProcessHeap(), 0, row); + return ret; +} + +static BOOL SaveIconResAsPNG(const BITMAPINFO *pIcon, const char *png_filename, LPCWSTR commentW) +{ + switch (pIcon->bmiHeader.biBitCount) + { + case 4: + return SavePalettedIconResAsPNG(pIcon, png_filename, commentW); + case 8: + return SavePalettedIconResAsPNG(pIcon, png_filename, commentW); + case 24: + return SaveTrueColorIconResAsPNG(pIcon, png_filename, commentW); + case 32: + return SaveTrueColorIconResAsPNG(pIcon, png_filename, commentW); + default: + WINE_FIXME("unsupported bpp %d, please report", pIcon->bmiHeader.biBitCount); + return FALSE; + } +} #endif /* SONAME_LIBPNG */ static BOOL SaveIconResAsXPM(const BITMAPINFO *pIcon, const char *szXPMFileName, LPCWSTR commentW)

1 0

Stefan Dösinger : wined3d: Split up the "one" pixel shader helper constant.
by Alexandre Julliard 21 May '10

21 May '10

Module: wine Branch: master Commit: 09d74b748e54955065cf5ea8adf63a0f6f1dd181 URL: http://source.winehq.org/git/wine.git/?a=commit;h=09d74b748e54955065cf5ea8a… Author: Stefan Dösinger <stefan(a)codeweavers.com> Date: Sat May 15 20:51:19 2010 +0200 wined3d: Split up the "one" pixel shader helper constant. --- dlls/wined3d/arb_program_shader.c | 16 ++++++++-------- 1 files changed, 8 insertions(+), 8 deletions(-) diff --git a/dlls/wined3d/arb_program_shader.c b/dlls/wined3d/arb_program_shader.c index b6285b2..57c5964 100644 --- a/dlls/wined3d/arb_program_shader.c +++ b/dlls/wined3d/arb_program_shader.c @@ -1366,7 +1366,7 @@ static void shader_hw_sample(const struct wined3d_shader_instruction *ins, DWORD if (pshader) { gen_color_correction(buffer, dst_str, ins->dst[0].write_mask, - "one", "coefmul.x", priv->cur_ps_args->super.color_fixup[sampler_idx]); + "ps_helper_const.x", "coefmul.x", priv->cur_ps_args->super.color_fixup[sampler_idx]); } } @@ -1405,13 +1405,13 @@ static void shader_arb_get_src_param(const struct wined3d_shader_instruction *in shader_addline(buffer, "ADD T%c, -%s, coefdiv.x;\n", 'A' + tmpreg, regstr); break; case WINED3DSPSM_SIGN: - shader_addline(buffer, "MAD T%c, %s, coefmul.x, -one.x;\n", 'A' + tmpreg, regstr); + shader_addline(buffer, "MAD T%c, %s, coefmul.x, -ps_helper_const.x;\n", 'A' + tmpreg, regstr); break; case WINED3DSPSM_SIGNNEG: - shader_addline(buffer, "MAD T%c, %s, -coefmul.x, one.x;\n", 'A' + tmpreg, regstr); + shader_addline(buffer, "MAD T%c, %s, -coefmul.x, ps_helper_const.x;\n", 'A' + tmpreg, regstr); break; case WINED3DSPSM_COMP: - shader_addline(buffer, "SUB T%c, one.x, %s;\n", 'A' + tmpreg, regstr); + shader_addline(buffer, "SUB T%c, ps_helper_const.x, %s;\n", 'A' + tmpreg, regstr); break; case WINED3DSPSM_X2: shader_addline(buffer, "ADD T%c, %s, %s;\n", 'A' + tmpreg, regstr, regstr); @@ -2192,7 +2192,7 @@ static void pshader_hw_texdepth(const struct wined3d_shader_instruction *ins) /* According to the msdn, the source register(must be r5) is unusable after * the texdepth instruction, so we're free to modify it */ - shader_addline(buffer, "MIN %s.y, %s.y, one.y;\n", dst_name, dst_name); + shader_addline(buffer, "MIN %s.y, %s.y, ps_helper_const.x;\n", dst_name, dst_name); /* How to deal with the special case dst_name.g == 0? if r != 0, then * the r * (1 / 0) will give infinity, which is clamped to 1.0, the correct @@ -2200,7 +2200,7 @@ static void pshader_hw_texdepth(const struct wined3d_shader_instruction *ins) */ shader_addline(buffer, "RCP %s.y, %s.y;\n", dst_name, dst_name); shader_addline(buffer, "MUL TA.x, %s.x, %s.y;\n", dst_name, dst_name); - shader_addline(buffer, "MIN TA.x, TA.x, one.x;\n"); + shader_addline(buffer, "MIN TA.x, TA.x, ps_helper_const.x;\n"); shader_addline(buffer, "MAX result.depth, TA.x, 0.0;\n"); } @@ -2276,7 +2276,7 @@ static void pshader_hw_texm3x2depth(const struct wined3d_shader_instruction *ins */ shader_addline(buffer, "RCP %s.y, %s.y;\n", dst_name, dst_name); shader_addline(buffer, "MUL %s.x, %s.x, %s.y;\n", dst_name, dst_name, dst_name); - shader_addline(buffer, "MIN %s.x, %s.x, one.x;\n", dst_name, dst_name); + shader_addline(buffer, "MIN %s.x, %s.x, ps_helper_const.x;\n", dst_name, dst_name); shader_addline(buffer, "MAX result.depth, %s.x, 0.0;\n", dst_name); } @@ -3485,7 +3485,7 @@ static GLuint shader_arb_generate_pshader(IWineD3DPixelShaderImpl *This, struct if(dcl_td) shader_addline(buffer, "TEMP TD;\n"); /* Used for sRGB writing */ shader_addline(buffer, "PARAM coefdiv = { 0.5, 0.25, 0.125, 0.0625 };\n"); shader_addline(buffer, "PARAM coefmul = { 2, 4, 8, 16 };\n"); - shader_addline(buffer, "PARAM one = { 1.0, 1.0, 1.0, 1.0 };\n"); + shader_addline(buffer, "PARAM ps_helper_const = { 1.0, 0.0, 0.0, 0.0 };\n"); if (reg_maps->shader_version.major < 2) {

1 0

Stefan Dösinger : wined3d: Store a positive 1. 0 in the ARB's vertex shader helper constant.
by Alexandre Julliard 21 May '10

21 May '10

Module: wine Branch: master Commit: 6946de81253d860097dfc29576a86a226f87c730 URL: http://source.winehq.org/git/wine.git/?a=commit;h=6946de81253d860097dfc2957… Author: Stefan Dösinger <stefan(a)codeweavers.com> Date: Sat May 15 20:56:29 2010 +0200 wined3d: Store a positive 1.0 in the ARB's vertex shader helper constant. --- dlls/wined3d/arb_program_shader.c | 6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/dlls/wined3d/arb_program_shader.c b/dlls/wined3d/arb_program_shader.c index 57c5964..a6cafe7 100644 --- a/dlls/wined3d/arb_program_shader.c +++ b/dlls/wined3d/arb_program_shader.c @@ -3981,7 +3981,7 @@ static GLuint shader_arb_generate_vshader(IWineD3DVertexShaderImpl *This, struct shader_addline(buffer, "TEMP TMP_OUT;\n"); if(need_helper_const(gl_info)) { - shader_addline(buffer, "PARAM helper_const = { 2.0, -1.0, %d.0, 0.0 };\n", This->rel_offset); + shader_addline(buffer, "PARAM helper_const = { 2.0, 1.0, %d.0, 0.0 };\n", This->rel_offset); } if(need_mova_const((IWineD3DBaseShader *) This, gl_info)) { shader_addline(buffer, "PARAM mova_const = { 0.5, 0.0, 2.0, 1.0 };\n"); @@ -4032,7 +4032,7 @@ static GLuint shader_arb_generate_vshader(IWineD3DVertexShaderImpl *This, struct */ if (!gl_info->supported[NV_VERTEX_PROGRAM]) { - shader_addline(buffer, "MOV result.color.secondary, -helper_const.wwwy;\n"); + shader_addline(buffer, "MOV result.color.secondary, helper_const.wwwy;\n"); if (gl_info->quirks & WINED3D_QUIRK_SET_TEXCOORD_W && !device->frag_pipe->ffp_proj_control) { @@ -4040,7 +4040,7 @@ static GLuint shader_arb_generate_vshader(IWineD3DVertexShaderImpl *This, struct for(i = 0; i < min(8, MAX_REG_TEXCRD); i++) { if(This->baseShader.reg_maps.texcoord_mask[i] != 0 && This->baseShader.reg_maps.texcoord_mask[i] != WINED3DSP_WRITEMASK_ALL) { - shader_addline(buffer, "MOV result.texcoord[%u].w, -helper_const.y;\n", i); + shader_addline(buffer, "MOV result.texcoord[%u].w, helper_const.y;\n", i); } } }

1 0

Stefan Dösinger : wined3d: Add a function for reading (ps_)helper_const.
by Alexandre Julliard 21 May '10

21 May '10

Module: wine Branch: master Commit: a1710a32f3dcba2fb3ec6d45d45cb76d16e6e6fc URL: http://source.winehq.org/git/wine.git/?a=commit;h=a1710a32f3dcba2fb3ec6d45d… Author: Stefan Dösinger <stefan(a)codeweavers.com> Date: Sat May 15 21:04:06 2010 +0200 wined3d: Add a function for reading (ps_)helper_const. --- dlls/wined3d/arb_program_shader.c | 108 ++++++++++++++++++++++++++++++------- 1 files changed, 88 insertions(+), 20 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=a1710a32f3dcba2fb3ec6…

1 0

Michael Stefaniuc : mcicda: Avoid using long.
by Alexandre Julliard 21 May '10

21 May '10

Module: wine Branch: master Commit: 309e92f449d8de5d70dc24ef912bb77c8ba5bf74 URL: http://source.winehq.org/git/wine.git/?a=commit;h=309e92f449d8de5d70dc24ef9… Author: Michael Stefaniuc <mstefani(a)redhat.de> Date: Fri May 21 12:13:15 2010 +0200 mcicda: Avoid using long. --- dlls/mcicda/mcicda.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/dlls/mcicda/mcicda.c b/dlls/mcicda/mcicda.c index 9f62594..cb8840e 100644 --- a/dlls/mcicda/mcicda.c +++ b/dlls/mcicda/mcicda.c @@ -564,7 +564,7 @@ static DWORD MCICDA_GetDevCaps(UINT wDevID, DWORD dwFlags, static DWORD CDROM_Audio_GetSerial(CDROM_TOC* toc) { - unsigned long serial = 0; + DWORD serial = 0; int i; WORD wMagic; DWORD dwStart, dwEnd;

1 0

Michael Stefaniuc : user32: Avoid using long.
by Alexandre Julliard 21 May '10

21 May '10

Module: wine Branch: master Commit: bf7241c12567c81ba4cc78b58c9550401cee95f7 URL: http://source.winehq.org/git/wine.git/?a=commit;h=bf7241c12567c81ba4cc78b58… Author: Michael Stefaniuc <mstefani(a)redhat.de> Date: Fri May 21 12:12:47 2010 +0200 user32: Avoid using long. --- dlls/user32/dialog.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/dlls/user32/dialog.c b/dlls/user32/dialog.c index 79f5182..aac8a4d 100644 --- a/dlls/user32/dialog.c +++ b/dlls/user32/dialog.c @@ -1374,7 +1374,7 @@ UINT WINAPI GetDlgItemInt( HWND hwnd, INT id, BOOL *translated, { char str[30]; char * endptr; - long result = 0; + LONG_PTR result = 0; if (translated) *translated = FALSE; if (!SendDlgItemMessageA(hwnd, id, WM_GETTEXT, sizeof(str), (LPARAM)str))

1 0