Paths

Table of Contentst

Differential D177

Fix the problem that kazv cannot be closed when cached files
ClosedPublic
Actions

Authored by nannanko on Aug 24 2024, 3:27 AM.

Tags

None

Referenced Files

None

Subscribers

None

Details

Reviewers

Group Reviewers

O1: the Kazv Project

Commits

rK70d40937b597: Fix the problem that kazv cannot be closed when cached files

Summary

Fixed the problem that kazv cannot be closed when cached files.

Type: fix

Test Plan

Open a room with uncached files and verify kazv starts caching the files.

Close kazv and verify that the cache jobs is killed immediately.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

nannanko created this revision.Aug 24 2024, 3:27 AM

Owners added a reviewer: O1: the Kazv Project.Aug 24 2024, 3:27 AM

Please add a changelog type to the summary, using the following format:

Type: (add|remove|skip|security|fix)

Harbormaster completed remote builds in B280: Diff 467.Aug 24 2024, 3:43 AM

nannanko requested review of this revision.Aug 24 2024, 3:43 AM

nannanko edited the summary of this revision. (Show Details)Aug 24 2024, 6:55 AM

nannanko removed a reviewer: servant.

tusooa requested changes to this revision.Aug 24 2024, 5:38 PM

tusooa added inline comments.

src/kazv-io-manager.cpp
188–191

This revision now requires changes to proceed.Aug 24 2024, 5:38 PM

nannanko marked an inline comment as done.Aug 24 2024, 6:08 PM

nannanko added inline comments.

src/kazv-io-manager.cpp
188–191	It don't work because the cancel() will cause the job to be removed from cacheJobs, so iterators and loops cannot be used reliably.

Fixed the problem that kazv cannot be closed when cached files

Added comment on why loops cannot be used directly on m_d->cacheJobs

Harbormaster completed remote builds in B284: Diff 472.Aug 31 2024, 7:37 AM

tusooa retitled this revision from Fixed the problem that kazv cannot be closed when cached files to Fix the problem that kazv cannot be closed when cached files.Aug 31 2024, 7:03 PM

tusooa accepted this revision.Sep 1 2024, 3:21 PM

This revision is now accepted and ready to land.Sep 1 2024, 3:21 PM

Closed by commit rK70d40937b597: Fix the problem that kazv cannot be closed when cached files. · Explain WhySep 4 2024, 3:18 AM

This revision was automatically updated to reflect the committed changes.

nannanko added a commit: rK70d40937b597: Fix the problem that kazv cannot be closed when cached files.

Revision Contents
Changeset List

Path

Size

Packages

src/

kazv-io-manager.cpp

12 lines

O1: the Kazv Project

Diff 477

src/kazv-io-manager.cpp

Loading...