-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.