kernel32: fix PROFILE_Load to handle mac line endings
Erik Inge Bolsø
knan-wine at anduin.net
Sun Sep 14 09:56:46 CDT 2008
On Sun, 14 Sep 2008, Michael Karcher wrote:
> This hunk is unneded. PROFILE_isspaceW already checks for \n and \r. The
> already present check for '\n' could be removed, but that should be a
> separate patch. I will send it (together with a cleanup in
> PROFILE_isspaceW) after testing.
Thanks. Indeed, tests still pass. Will remove the hunk and resend.
> [...]
> Wouldn't it be sensible to factor out the common testing code, and just
> pass the string that gets written to the file as parameter? (I didn't
> check line-by-line that it matches, but the comment seems to implicate
> it.) If you refactor, you could also add a test that uses mixed endings
> (some \r, some \n, some \r\n).
I'm just trying for a minimal fix and simple test for \r-terminated
ini files here. If you want to clean it up further, feel free.
--
-erik http://useofwords.blogspot.com/
More information about the wine-devel
mailing list