[PATCH] cryptnet/tests: Make the tests usable on win9x

Juan Lang juan.lang at gmail.com
Thu Jul 24 13:01:22 CDT 2008


Hi Detlef, overall your patch looks good to me.  One minor suggestion:

+    /* only newer versions of cryptnet do the cleanup */
+    if(!ret && GetLastError() != CRYPT_E_ASN1_BADTAG &&
+               GetLastError() != OSS_DATA_ERROR) {
+        ok(crl == NULL, "Expected CRL to be NULL\n");
+    }

Rather than only checking crl when a newer version of cryptnet is
available, why not use the broken() macro?  We really do expect crl to
get cleaned up.
--Juan



More information about the wine-devel mailing list