-BOOL __cdecl FDIIsCabinet(
- �� HFDI �� hfdi,
- �� INT_PTR �� hf,
- �� PFDICABINETINFO pfdici)
+BOOL __cdecl FDIIsCabinet(HFDI hfdi, INT_PTR hf, PFDICABINETINFO pfdici)
�� {
�� BOOL rv;
�� FDI_Int *fdi = get_fdi_ptr( hfdi );
@@ -749,13 +746,13 @@ BOOL __cdecl FDIIsCabinet(
�� if (!fdi) return FALSE;
�� if (!hf) {
- �� ERR("(!hf)!\n");
+ �� WARN("No file handle\n");
�� SetLastError(ERROR_INVALID_HANDLE);
�� return FALSE;
�� }
�� if (!pfdici) {
- �� ERR("(!pfdici)!\n");
+ �� WARN("No cab info pointer\n");
�� SetLastError(ERROR_BAD_ARGUMENTS);
�� return FALSE;
�� }

This call already has a TRACE earlier, I don't see a point in tracing input parameters multiple times.

Indeed. That would be better to remove them. Thanks.��