[Bug 6842] DrawFocusRect code correction
Wine Bugs
wine-bugs at winehq.org
Thu Dec 7 12:51:49 CST 2006
http://bugs.winehq.org/show_bug.cgi?id=6842
------- Additional Comments From juan_lang at yahoo.com 2006-07-12 12:51 -------
It would be easier to check your code if you sent it as a patch to wine-devel.
We want to be able to see exactly what you've changed, so the patch format helps
with that.
Some initial feedback:
1. This looks like preprocessor abuse - choose one method (to cache or not to
cache) and propose that.
2. Don't use C++-style comments (no //).
3. You have comments on too many lines - one larger comment before the block
that explain the gist of it could replace a lot of these.
--
Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
More information about the wine-bugs
mailing list