win16 patches
Juan Lang
juan.lang at gmail.com
Wed Mar 31 19:58:24 CDT 2010
Hi C.W., this isn't going to fly. For one thing,
diff --git a/dlls/kernel32/comm.c b/dlls/kernel32/comm.c
index f4edfd3..a85f158 100644
--- a/dlls/kernel32/comm.c
There's no attribution in this patch. It can't get committed without
valid attribution.
For another thing, one patch per email, please.
For another,
- if(!vga_text_console)
+// if(!vga_text_console)
C++ comments aren't allowed, and removing a line by commenting it out
is frowned upon. If it's really not needed, just remove it. But you
need to justify why it's no longer needed.
More errors remain, but I'll stop at these. This set is well below
acceptable yet, I'm afraid.
--Juan
More information about the wine-devel
mailing list