Update comments
This commit is contained in:
@@ -787,9 +787,6 @@ namespace AssetProcessor
|
||||
}
|
||||
|
||||
// if we get here, we are good to go in terms of disk space and sources existing, so we make the best attempt we can.
|
||||
// first, we broadcast the name of ALL of the outputs we are about to change:
|
||||
|
||||
// after we do the above notify its important that we do not early exit this function without undoing those locks.
|
||||
|
||||
bool anyFileFailed = false;
|
||||
|
||||
@@ -813,9 +810,6 @@ namespace AssetProcessor
|
||||
AZ_TracePrintf(AssetBuilderSDK::WarningWindow, "Unable to change permission for the file: %s.\n", productAbsolutePath.toUtf8().data());
|
||||
}
|
||||
}
|
||||
|
||||
// once we're done, regardless of success or failure, we 'unlock' those files for further process.
|
||||
// if we failed, also re-trigger them to rebuild (the bool param at the end of the ebus call)
|
||||
|
||||
return !anyFileFailed;
|
||||
}
|
||||
|
||||
@@ -142,8 +142,9 @@ namespace AssetUtilsInternal
|
||||
}
|
||||
}
|
||||
} while (!timer.hasExpired(waitTimeInSeconds * 1000)); //We will keep retrying until the timer has expired the inputted timeout
|
||||
|
||||
// note that this absolute path is a real file system path, and the following API requires normalized paths:
|
||||
|
||||
// once we're done, regardless of success or failure, we 'unlock' those files for further process.
|
||||
// if we failed, also re-trigger them to rebuild (the bool param at the end of the ebus call)
|
||||
QString normalized = AssetUtilities::NormalizeFilePath(outputFile);
|
||||
AssetProcessor::ProcessingJobInfoBus::Broadcast(
|
||||
&AssetProcessor::ProcessingJobInfoBus::Events::EndCacheFileUpdate, normalized.toUtf8().constData(), !operationSucceeded);
|
||||
|
||||
Reference in New Issue
Block a user