On 8/4/2010 11:25 AM, Elizabeth M Smith wrote:
On 8/4/2010 11:13 AM, Rivera, Rafael wrote:
Looks nice, love the chip name.
Just one note -- I see wide-character strings in use. I suggest
replacing them, and their print functions, with T equivalents, leaving
the choice of ANSI or UNICODE to the magical defines in the headers and
ultimately the human compiling this stuff.
So, from initial glance, I'd replace L"" with _T("") and fwprintf with
_ftprintf.
No way in hell do I want to support anyone doing that - then you'd have
idiots compiling and giving out version that don't support UNICODE
strings and users complaining about it
ICK
UNICODE or nothing.
Thanks,
Elizabeth M Smith
|
_______________________________________________
Mailing list: https://launchpad.net/~coapp-developers
Post to : coapp-developers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~coapp-developers
More help : https://help.launchpad.net/ListHelp
