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