[prev in list] [next in list] [prev in thread] [next in thread]
List: wine-devel
Subject: Re: [1/2] user32: Add a test for ToUnicode (try 5)
From: Paul Vriens <paul.vriens.wine () gmail ! com>
Date: 2008-07-31 16:52:32
Message-ID: 4891EDD0.4070601 () gmail ! com
[Download RAW message or body]
Andre Wisplinghoff wrote:
> Wrapping was still wrong. Sorry for that. Just using attachments now.
>
> dlls/user32/tests/input.c | 37 +++++++++++++++++++++++++++++++++++++
> 1 files changed, 37 insertions(+), 0 deletions(-)
>
>
> ------------------------------------------------------------------------
>
>
Just some things that caught my attention:
Could you trim the length for some of the lines somewhat? I also don't think you
need all those comments in the first call to ToUnicode.
There is a C++ comment in the patch, should be avoided.
And as a last one, is it possible to use names instead of magic constants?
--
Cheers,
Paul.
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic