always set the callback back, because even if it were empty it needs to be empty again

Author: devsh

src/nbl/video/utilities/CAssetConverter.cpp

@@ -5344,8 +5344,7 @@ ISemaphore::future_t<IQueue::RESULT> CAssetConverter::convert_impl(SReserveResul
 				retval.set({params.transfer->scratchSemaphore.semaphore,params.transfer->scratchSemaphore.value});
 		}
 		// reset original callback
-		if (bool(origXferStallCallback))
-			params.transfer->overflowCallback = std::move(origXferStallCallback);
+		params.transfer->overflowCallback = std::move(origXferStallCallback);
 
 		// Its too dangerous to leave an Intended Transfer Submit hanging around that needs to be submitted for Compute to make forward progress outside of this utility,
 		// and doing transfer-signals-after-compute-wait timeline sema tricks are not and option because: