Re: [PATCH v4 0/3] MR3472: secur32: Convert "Negotiate" to an LSA package.

[Hans Leidekker (＠hans)]

Hans Leidekker (@hans) commented about dlls/secur32/lsa.c:
> -static BOOL load_package(const WCHAR *name, struct lsa_package *package, ULONG package_id)
> +static BOOL initialize_package(struct lsa_package *package, void *lsa_mode_init, void *user_mode_init)
>  {
> - NTSTATUS (NTAPI *pSpLsaModeInitialize)(ULONG, PULONG, PSECPKG_FUNCTION_TABLE *, PULONG);
> - NTSTATUS (NTAPI *pSpUserModeInitialize)(ULONG, PULONG, PSECPKG_USER_FUNCTION_TABLE *, PULONG);
> -
> - memset(package, 0, sizeof(*package));
> - NTSTATUS (NTAPI *pSpLsaModeInitialize)(ULONG, PULONG, PSECPKG_FUNCTION_TABLE *, PULONG) = lsa_mode_init;
> - NTSTATUS (NTAPI *pSpUserModeInitialize)(ULONG, PULONG, PSECPKG_USER_FUNCTION_TABLE *, PULONG) = user_mode_init;
> - package->mod = LoadLibraryW(name);
> - if (!package->mod) return FALSE;
> -
> - pSpLsaModeInitialize = (void *)GetProcAddress(package->mod, "SpLsaModeInitialize");
> if (pSpLsaModeInitialize)
> {

While you're at it, you may want to invert this condition to reduce indentation.