Dr. Stephen Henson wrote:
  -     if (len == -1)
  +     if ((len == -1) && !(attrtype & MBSTRING_FLAG))

I do wish you wouldn't use these extra brackets around comparison operators.

        if (len == -1 && !(attrtype & MBSTRING_FLAG))

works just fine and is consistent with most of the rest of the code, and the rest of the world.

--
http://www.apache-ssl.org/ben.html           http://www.links.org/

"There is no limit to what a man can do or how far he can go if he
doesn't mind who gets the credit." - Robert Woodruff
______________________________________________________________________
OpenSSL Project                                 http://www.openssl.org
Development Mailing List                       [email protected]
Automated List Manager                           [EMAIL PROTECTED]

Reply via email to