wininet: Implement InternetCreateUrlA/W with test cases
Juan Lang
juan_lang at yahoo.com
Fri Oct 28 11:48:08 CDT 2005
Hi James,
+/* max port num is 655936 => 6 digits */
Actually, it's 65535. A correct comment would be appreciated ;)
+ *lpdwUrlLength += strlen("://");
You should use sizeof instead to avoid the call to strlen. Same for other
lengths you use, like "@" and ":".
+ if (lpUrlComponents->nPort != 80 ||
This seems a bit funny. Port 80 has no meaning for INTERNET_SCHEME_FTP or
any other non-HTTP protocol, does it?
--Juan
__________________________________
Yahoo! Mail - PC Magazine Editors' Choice 2005
http://mail.yahoo.com
More information about the wine-devel
mailing list