What's wrong with my patch?

2009-08-31 Thread Jan de Mooij
Hello. Subject says it all. I suppose the tests I added may fail on other locales, but there is no other way to test my patch? Should i send it without the tests? Thanks in advance. Jan -- Forwarded message -- From: Jan de Mooij jandemo...@gmail.com Date: Mon, Aug 31, 2009 at

Re: What's wrong with my patch?

2009-08-31 Thread Juan Lang
Hi Jan, Subject says it all. Well, you haven't waited nearly long enough. Sending it in today and expecting it to be committed the same day is a bit much. --Juan

Re: What's wrong with my patch?

2009-08-31 Thread Andrew Eikum
Jan de Mooij wrote: Hello. Subject says it all. I suppose the tests I added may fail on other locales, but there is no other way to test my patch? Should i send it without the tests? Thanks in advance. Jan I can't speak much to the patch itself, but the patch file you attached has an

Re: What's wrong with my patch?

2009-08-31 Thread Jan de Mooij
On Mon, Aug 31, 2009 at 6:25 PM, Juan Langjuan.l...@gmail.com wrote: Well, you haven't waited nearly long enough.  Sending it in today and expecting it to be committed the same day is a bit much. I saw that AJ committed patches sent before and after my email, so I assumed he goes through them

Re: What's wrong with my patch?

2009-08-31 Thread Juan Lang
I saw that AJ committed patches sent before and after my email, so I assumed he goes through them in order. It also depends on whether your patch is obviously correct, and your Julliard rank. Since you're new around here, it's likely your patch will be subject to more scrutiny. Plus Andrew

What's wrong with ...

2007-10-29 Thread Christian Eggers
Hello, I've submitted a patch to wine-patches 5 days ago which has not been added yet: Added wrapper dll for ct-api(CardTerminal API) http://www.winehq.org/pipermail/wine-patches/2007-October/045716.html I think that most of the code has already been reviewed: The 1st version has already been

any clues as to what's wrong with patches (system palette entries)?

2007-10-24 Thread Alexander Dorofeyev
Hello. I've sent a patch that changes X11DRV_GetSystemPaletteEntries behavior and a test something like a week ago. I guess they fell into that so called not obviously correct category (or maybe not correct at all one). I'll appreciate any comments or suggestions on possible improvements.

Re: [try 2] Move console input/output codepages to server - what's wrong?

2007-05-07 Thread Kirill K. Smirnov
В сообщении от 30 апреля 2007 16:26 вы написали: Hi, A week ago I've sent a patch which fixes console input/output codepages handling. http://www.winehq.org/pipermail/wine-patches/2007-April/038429.html It is still not applied, so please, explain, what is wrong with it? It even fixes 2

[try 2] Move console input/output codepages to server - what's wrong?

2007-04-30 Thread Kirill K. Smirnov
Hi, A week ago I've sent a patch which fixes console input/output codepages handling. http://www.winehq.org/pipermail/wine-patches/2007-April/038429.html It is still not applied, so please, explain, what is wrong with it? It even fixes 2 tests: $wine kernel32_test.exe.so process before:

Re: [try2] kernel32: Add partial stub for NeedCurrentDirectoryForExePath - what's wrong?

2007-04-25 Thread Kirill K. Smirnov
Now, I am writing tests for this. Windows crashes if I pass NULL pointer to these function. Crash can be easily avoided in wine. Should we crash too? And how the windows crash can be caught? Or just I should not write such test-case? Thanks, -- Kirill

[try2] kernel32: Add partial stub for NeedCurrentDirectoryForExePath - what's wrong?

2007-04-24 Thread Kirill K. Smirnov
Hi, I've sent the patch: http://www.winehq.com/pipermail/wine-patches/2007-April/038205.html about a week ago and it has not been applied. Please, explain, what is wrong with it? Thanks -- Kirill

Re: [try2] kernel32: Add partial stub for NeedCurrentDirectoryForExePath - what's wrong?

2007-04-24 Thread James Hawkins
On 4/24/07, Kirill K. Smirnov [EMAIL PROTECTED] wrote: Hi, I've sent the patch: http://www.winehq.com/pipermail/wine-patches/2007-April/038205.html about a week ago and it has not been applied. Please, explain, what is wrong with it? You haven't sent in any tests for this function. Is

Re: [try2] kernel32: Add partial stub for NeedCurrentDirectoryForExePath - what's wrong?

2007-04-24 Thread Kirill K. Smirnov
On 4/24/07, Kirill K. Smirnov [EMAIL PROTECTED] wrote: Hi, I've sent the patch: http://www.winehq.com/pipermail/wine-patches/2007-April/038205.html about a week ago and it has not been applied. Please, explain, what is wrong with it? You haven't sent in any tests for this function.

Re: [try2] kernel32: Add partial stub for NeedCurrentDirectoryForExePath - what's wrong?

2007-04-24 Thread James Hawkins
On 4/25/07, Kirill K. Smirnov [EMAIL PROTECTED] wrote: On 4/24/07, Kirill K. Smirnov [EMAIL PROTECTED] wrote: Hi, I've sent the patch: http://www.winehq.com/pipermail/wine-patches/2007-April/038205.html about a week ago and it has not been applied. Please, explain, what is wrong

Re: [try2] kernel32: Add partial stub for NeedCurrentDirectoryForExePath - what's wrong?

2007-04-24 Thread Kirill K. Smirnov
For containing backshash. Yes, comment must be better. A comment that states you're looking for a backslash is superfluous; the code makes that obvious. A better comment would be 'check for filename only' or something along those lines. Maybe, 'MSDN claims that if filename contains

Re: [try3]: user32/tests: add tests for WM_SETICON - what's wrong?

2007-02-05 Thread Kirill K. Smirnov
Hi, I've asked before: What's wrong with them? Maybe, unnecessary PeekMessage in tests??? http://www.winehq.org/pipermail/wine-patches/2007-January/035044.html http://www.winehq.org/pipermail/wine-patches/2007-January/035043.html Please, answer! Do not ignore! -- Kirill

Re: [try3]: user32/tests: add tests for WM_SETICON - what's wrong?

2007-01-23 Thread Kirill K. Smirnov
Anything wrong with them again? http://www.winehq.org/pipermail/wine-patches/2007-January/035044.html http://www.winehq.org/pipermail/wine-patches/2007-January/035043.html Thanks, Kirill

[try2] user32/tests: add tests for WM_SETICON - what's wrong?

2007-01-17 Thread Kirill K. Smirnov
Hi, Traditional questions from rejected patch-sender: 1) What's wrong with the patch: http://www.winehq.org/pipermail/wine-patches/2007-January/034949.html 2) What can be done to improve it? -- Kirill

Re: [try2] user32/tests: add tests for WM_SETICON - what's wrong?

2007-01-17 Thread Alexandre Julliard
Kirill K. Smirnov [EMAIL PROTECTED] writes: Traditional questions from rejected patch-sender: 1) What's wrong with the patch: http://www.winehq.org/pipermail/wine-patches/2007-January/034949.html 2) What can be done to improve it? The test fails on Windows XP: msg.c:3944: Test failed

user32: the patches - what's wrong?

2007-01-15 Thread Kirill K. Smirnov
Hi, Traditional questions from rejected patch-sender: 1) What's wrong with the following patches? http://www.winehq.org/pipermail/wine-patches/2007-January/034758.html http://www.winehq.org/pipermail/wine-patches/2007-January/034790.html 2) What can be done to improve them? Those patches let

Re: user32: the patches - what's wrong?

2007-01-15 Thread Alexandre Julliard
Kirill K. Smirnov [EMAIL PROTECTED] writes: Traditional questions from rejected patch-sender: 1) What's wrong with the following patches? http://www.winehq.org/pipermail/wine-patches/2007-January/034758.html http://www.winehq.org/pipermail/wine-patches/2007-January/034790.html 2) What can

Re: [Try2] winhelp: add preliminary *.cnt file support (parser) - what's wrong?

2006-11-22 Thread Kirill K. Smirnov
Hi! Please, tell me what's wrong with the patch? http://www.winehq.org/pipermail/wine-patches/2006-November/032984.html -- Kirill K. Smirnov

winhelp: fix LZ77 decompressor patch: what's wrong?

2006-08-11 Thread HolyLich
Hi, I've sent patch several times, but it has not been applied. Patch is reasonable, MIME type is OK. patch -p0 mbox works ;-) (as recommended at http://winehq.org/site/docs/winedev-guide/style-notes) Please, tell what I must do to this patch to be applied? Thanks a lot -- HolyLich Subject:

Re: winhelp: fix LZ77 decompressor patch: what's wrong?

2006-08-11 Thread Dan Kegel
2006/8/11, HolyLich [EMAIL PROTECTED]: I've sent patch several times, but it has not been applied. Why not use memmove() instead?

Re: winhelp: fix LZ77 decompressor patch: what's wrong?

2006-08-11 Thread HolyLich
В сообщении от Friday 11 August 2006 17:17 вы написали: 2006/8/11, HolyLich [EMAIL PROTECTED]: I've sent patch several times, but it has not been applied. Why not use memmove() instead? Heh, this problem was discussed before

Re: winhelp: fix LZ77 decompressor patch: what's wrong?

2006-08-11 Thread Alexandre Julliard
HolyLich [EMAIL PROTECTED] writes: I've sent patch several times, but it has not been applied. Patch is reasonable, MIME type is OK. patch -p0 mbox works ;-) (as recommended at http://winehq.org/site/docs/winedev-guide/style-notes) Please, tell what I must do to this patch to be applied?

Re: winhelp: fix LZ77 decompressor patch: what's wrong?

2006-08-11 Thread Dan Kegel
2006/8/11, HolyLich [EMAIL PROTECTED]: Why not use memmove() instead? Heh, this problem was discussed before http://www.winehq.org/pipermail/wine-devel/2006-August/050133.html, http://www.winehq.org/pipermail/wine-devel/2006-August/050139.html

What's wrong in my Trash patches?

2006-07-19 Thread Mikołaj Zalewski
I've sent patches to implement the trashing of files (starting from http://www.winehq.org/pipermail/wine-patches/2006-July/028782.html ) and they were rejected. Could someone tell me what's wrong with them? Mikolaj Zalewski

Re: What's wrong in my Trash patches?

2006-07-19 Thread Steven Edwards
On 7/19/06, Mikołaj Zalewski [EMAIL PROTECTED] wrote: I've sent patches to implement the trashing of files (starting from http://www.winehq.org/pipermail/wine-patches/2006-July/028782.html ) and they were rejected. Could someone tell me what's wrong with them? I normally follow wine-patches