Удаление рабочего элемента многопоточного загрузчика текстур C++ из основного потока

Итак, имхо есть два варианта:

1. Вы ждете завершения всех запросов на конкретный объект, прежде чем удалить его.
2. Вы проверяете, существует ли объект, прежде чем выполнять любую запланированную операцию.

У меня недостаточно информации о вашем приложении: как реализована очередь и почему и когда планируются запросы, поэтому я не могу дать отзыв по этому поводу.

Я обнаружил, что мне нужно создать список отмены из защищенного вектора std::mutex. Когда основной поток хочет выйти, он просто добавляет запись в вектор и продолжает работу. Поток загрузки текстур несет дополнительную нагрузку по проверке списка для каждого полученного запроса текстуры, но операция не находится на критическом пути.

Меня все еще интересуют альтернативы/предложения.

Небольшой план темы ниже:

void textureLoaderThreadFn()
{
    log("texture loader thread started");

    while (!mShouldQuit)
    {
        // Wait for texture loader request
        TextureLoaderRequest *textureLoaderRequest = nullptr;
        mTextureRequests->popBack(&textureLoaderRequest);

        // it is possible popBack didnt modify textureLoaderRequest (eg. when cancelled on exit)
        if (textureLoaderRequest != nullptr)
        {
            std::lock_guard<std::mutex> lk(mCancellationListMutex);

            if (std::find(mCancellationList.begin(), mCancellationList.end(), textureLoaderRequest->mFilename) != mCancellationList.end())
            {
                // Cancelled

                // we must reset the isLoading that was set by the main thread,
                // so that the request to load the texture can get put back if need be
                textureLoaderRequest->mContentViewer->mIsLoading = false;

                // remove from cancellation list
                mCancellationList.erase(std::remove(mCancellationList.begin(), mCancellationList.end(), textureLoaderRequest->mFilename), mCancellationList.end());
            }
            else
            {
                // Not cancelled
                <SERVICE TEXTURE REQUEST>
            }

            // dont need this anymore
            delete textureLoaderRequest;
        }
    }
    log("texture loader thread stopped");

    // Empty the queue    
    int count = 0;
    TextureLoaderRequest *textureLoaderRequest = nullptr;
    while (mTextureRequests->tryPopBack(&textureLoaderRequest))
    {
        if (textureLoaderRequest != nullptr)
            delete textureLoaderRequest;
        count++;
    }
    log("texture loader thread purged " + std::to_string(count) + " outstanding texture load requests");
}

Я бы посоветовал вам добавить флаг состояния в средство просмотра контента, чтобы различать 3 статуса; запланированные для окрашивания, окрашиваемые и не запланированные для окрашивания. Основной поток должен удалять средство просмотра контента только в том случае, если его состояние запланировано для окрашивания или не запланировано для окрашивания.

Рабочий поток текстуры меняет статус на цветной, и как только он окрашен, он переводит его в цветной.

Изменение статусов и проверка статуса на возможность его удаления всегда должны осуществляться одним и тем же мьютексом; вы можете установить флаг как частный в средстве просмотра контента и использовать два общедоступных метода: 1) void change_status (status) и 2) bool can_delete().

Обе функции должны начинаться с получения одного и того же мьютекса. 1) используется в различных переходах в основном потоке и в потоке обработки текстур, а 2) используется основным потоком перед удалением средства просмотра содержимого, чтобы вернуть значение true, только если статус не окрашивается.

В потоке работы с текстурами перед выходом вы можете удалить последнее окрашенное содержимое, если оно не было удалено основным потоком (как это могло быть при окрашивании статуса).