Hi, let me just to do some random patch review (would I get some karma for
that, lispy?), this seems like an easy one.

Ganesh Sittampalam <[EMAIL PROTECTED]> writes:
> Mon Sep  1 23:34:20 BST 2008  Ganesh Sittampalam <[EMAIL PROTECTED]>
>   * resolve issue1015: workaround internal API change on Windows in GHC 6.8.3
>
> New patches:
>
> [resolve issue1015: workaround internal API change on Windows in GHC 6.8.3
> Ganesh Sittampalam <[EMAIL PROTECTED]>**20080901223420] hunk 
> ./src/win32/System/Posix/Files.hsc 49
> -    return (FileStatus tp mode mtime size)
> +    return (FileStatus tp mode mtime (fromIntegral size))
Looks like a decent workaround with no collateral damage. Unfortunately, I
don't have Windows to actually try it out.

Yours,
   Petr.

PS: I agree this message might not be that useful, but let's try to form some
good habits, even if they aren't productive right from the start. I propose
cookies for first constructive reply on a patch, awarded at hacking sprints
(shame I can't attend...) -- one cookie per reviewed patch. ; - )

-- 
Peter Rockai | me()mornfall!net | prockai()redhat!com
 http://blog.mornfall.net | http://web.mornfall.net

"In My Egotistical Opinion, most people's C programs should be
 indented six feet downward and covered with dirt."
     -- Blair P. Houghton on the subject of C program indentation
_______________________________________________
darcs-users mailing list
[email protected]
http://lists.osuosl.org/mailman/listinfo/darcs-users

Reply via email to