oops.
Absolutely agree.
Thanks for the comment. Will fix it.

Thanks,
Mikhail

On Fri, 02 Apr 2010 00:29:16 +0700, Alexandre Julliard <julli...@winehq.org> wrote:

"Mikhail Maroukhine" <mik...@yandex.ru> writes:

Hello,

Could you clear a little what do you mean under "stop abusing"?
I've removed incorrect usage of constness in declaration for the
variable that is changed for sure.
I've send "try 2" patch with additional correction and extended log
message.

I mean your new origProtoRef variable (which BTW uses a completely
different naming convention from the rest of the function) to work
around the fact that the argument is modified. Much better to use a
properly named local variable to store results and not modify the
argument.



--
Using Opera's revolutionary e-mail client: http://www.opera.com/mail/


Reply via email to