Page MenuHomePhorge
Differential D177

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

Authored by nannanko on Sat, Aug 24, 3:27 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Sep 14, 3:34 AM
Unknown Object (File)
Thu, Sep 12, 10:10 AM
Unknown Object (File)
Wed, Sep 11, 3:27 PM
Unknown Object (File)
Wed, Sep 11, 10:01 AM
Unknown Object (File)
Wed, Sep 11, 8:01 AM
Unknown Object (File)
Tue, Sep 10, 3:46 PM
Unknown Object (File)
Mon, Sep 9, 10:28 AM
Unknown Object (File)
Mon, Sep 9, 10:20 AM
View All 33 Files
Subscribers
None

Details

Reviewers
tusooa
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
rK kazv
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

nannanko created this revision.Sat, Aug 24, 3:27 AM
Owners added a reviewer: O1: the Kazv Project.Sat, Aug 24, 3:27 AM
Herald added a reviewer: servant. · View Herald TranscriptSat, Aug 24, 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.Sat, Aug 24, 3:43 AM
nannanko requested review of this revision.Sat, Aug 24, 3:43 AM
nannanko edited the summary of this revision. (Show Details)Sat, Aug 24, 6:55 AM
nannanko removed a reviewer: servant.
tusooa requested changes to this revision.Sat, Aug 24, 5:38 PM
tusooa added inline comments.
src/kazv-io-manager.cpp
188–191
This revision now requires changes to proceed.Sat, Aug 24, 5:38 PM
nannanko marked an inline comment as done.Sat, Aug 24, 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.

nannanko updated this revision to Diff 472.Sat, Aug 31, 7:21 AM
nannanko marked an inline comment as done.
  • Fixed the problem that kazv cannot be closed when cached files
nannanko added a comment.Sat, Aug 31, 7:22 AM

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

Harbormaster completed remote builds in B284: Diff 472.Sat, Aug 31, 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.Sat, Aug 31, 7:03 PM
tusooa accepted this revision.Sun, Sep 1, 3:21 PM
This revision is now accepted and ready to land.Sun, Sep 1, 3:21 PM
Closed by commit rK70d40937b597: Fix the problem that kazv cannot be closed when cached files. · Explain WhyWed, Sep 4, 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

PathSizePackages
src/
12 linesO1: the Kazv Project

Diff 477

View Options

src/kazv-io-manager.cpp

Loading...