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