[PATCH 1/4] kernel32: Support UTF-7 in MultiByteToWideChar.
alexhenrie24 at gmail.com
Tue Dec 4 21:32:41 CST 2012
2012/12/4 Alexandre Julliard <julliard at winehq.org>:
> Alex Henrie <alexhenrie24 at gmail.com> writes:
>> My implementation is modeled after Windows XP (Wine's default target
>> Windows version), which encodes and decodes arbitrary character
>> sequences without normalization. I saw that my submission has already
>> been marked "rejected"--was this why?
> It was rejected because you still haven't addressed any of the issues I
> mentioned in previous reviews. It doesn't look like this is going anywhere.
Well, I went back through the archives and compiled the feedback you've given:
1. Don't use Katayama Hirofumi's code.
2. Don't use iconv.
3. Put the code in libwine.
4. Don't use malloc.
5. Add tests for buffer overflow.
6. Add tests for partial sequences.
7. Add tests for srclen < -1.
8. Add more tests.
9. Split up the patch.
10. Add tests for srclen > 0.
Far from not addressing any of these issues, I feel that I have
addressed all of them. More importantly, my implementation is correct;
it matches Windows XP exactly. (I'd be happy to be proven wrong if
anyone can show me a case where this is not so.) If there is another
issue which I have not addressed, it's because it has not been
articulated to me. I get the impression that you have something in
particular that you're looking for, but I can't tell what it is.
I do want to see this resolved. It would be a pity if Mac and Linux
users miss out because we couldn't come to an agreement...
More information about the wine-devel