please don't shoot from the hip ;)
James McKenzie
jjmckenzie51 at earthlink.net
Fri Jul 9 21:15:07 CDT 2010
Misha Koshelev wrote:
> Hi Guys:
>
> I really appreciate all the responses to my work on this bug
> http://bugs.winehq.org/show_bug.cgi?id=22918
>
>
Misha:
Not everyone gets or reads the wine-devel mailing list. It is good to
duplicate what appears in the mailing list into the bug report that it
affects. Don't expect folks to change.
However, if someone is being rude or obnoxious, please feel free to let
them know.
Everyone else:
I'll be the first to bite on this: Please be friendly. I know that
I've been a little uptight the last few days, but it all over how we
code and what we code. Remember, folks are not going to figure out what
you were doing when trying to fix a bug six months from now that was
inadvertantly caused by bad coding practices. Look over your code
before submitting it. Look for tabs instead of spaces, dangling or
unnecessary white space, use of 'obtuse' variables (i, j, k for counters
in for/do/while loops is accepted practice, count is obvious,
count_for_this_loop is unnecessary), constants on the right side of if
statements (why you ask? Try using =, | or & instead of ==, || or &&
once and see the fireworks, switch it around and you get a compiler
error "cannot change constant"). Other things like using MSDN
variables when we could/should use WINE....
Something that I've noticed is that some code is unnecessarily complex.
Simple solutions work best and are easy to maintain. Remember, you
might not be the next to touch your code.
Thank you for reading and please folks, if you want to hammer me, do it
off the list. We don't need this to degrade into a flame war.
James McKenzie
Coding in 'c' since 1981
Former member, WINOS/2
Former member, Project Odinn.
More information about the wine-devel
mailing list