msi: Move the file sequence check out of ready_media_info to avoid an unnecessary function call.
This commit is contained in:
parent
6ef6512938
commit
d21e1cc05b
|
@ -473,13 +473,6 @@ static UINT ready_media_for_file( MSIPACKAGE *package, struct media_info *mi,
|
|||
UINT rc = ERROR_SUCCESS;
|
||||
BOOL found = FALSE;
|
||||
|
||||
if (file->Sequence <= mi->last_sequence)
|
||||
{
|
||||
set_file_source(package, file, mi->source);
|
||||
TRACE("Media already ready (%u, %u)\n",file->Sequence,mi->last_sequence);
|
||||
return ERROR_SUCCESS;
|
||||
}
|
||||
|
||||
rc = load_media_info(package, file, mi);
|
||||
if (rc != ERROR_SUCCESS)
|
||||
{
|
||||
|
@ -487,12 +480,6 @@ static UINT ready_media_for_file( MSIPACKAGE *package, struct media_info *mi,
|
|||
return ERROR_FUNCTION_FAILED;
|
||||
}
|
||||
|
||||
if (!file->IsCompressed)
|
||||
{
|
||||
set_file_source(package, file, mi->source);
|
||||
return rc;
|
||||
}
|
||||
|
||||
if (mi->cabinet)
|
||||
{
|
||||
TRACE("Source is CAB %s\n", debugstr_w(mi->cabinet));
|
||||
|
@ -522,8 +509,6 @@ static UINT ready_media_for_file( MSIPACKAGE *package, struct media_info *mi,
|
|||
rc = !extract_cabinet_file(package, mi);
|
||||
}
|
||||
|
||||
set_file_source(package, file, mi->source);
|
||||
|
||||
return rc;
|
||||
}
|
||||
|
||||
|
@ -636,21 +621,23 @@ UINT ACTION_InstallFiles(MSIPACKAGE *package)
|
|||
|
||||
mi = msi_alloc_zero( sizeof(struct media_info) );
|
||||
|
||||
/* Pass 2 */
|
||||
LIST_FOR_EACH_ENTRY( file, &package->files, MSIFILE, entry )
|
||||
{
|
||||
if (file->state != msifs_missing && file->state != msifs_overwrite)
|
||||
continue;
|
||||
|
||||
TRACE("Pass 2: %s\n",debugstr_w(file->File));
|
||||
|
||||
if (file->Sequence > mi->last_sequence || mi->is_continuous)
|
||||
{
|
||||
rc = ready_media_for_file( package, mi, file );
|
||||
if (rc != ERROR_SUCCESS)
|
||||
{
|
||||
ERR("Unable to ready media\n");
|
||||
ERR("Failed to ready media\n");
|
||||
rc = ERROR_FUNCTION_FAILED;
|
||||
break;
|
||||
}
|
||||
}
|
||||
|
||||
set_file_source(package, file, mi->source);
|
||||
|
||||
TRACE("file paths %s to %s\n",debugstr_w(file->SourcePath),
|
||||
debugstr_w(file->TargetPath));
|
||||
|
|
Loading…
Reference in New Issue