Differential D17000

Make sure there is only one MTime per document in the MTimeDB
ClosedPublic
Actions

Authored by bruns on Nov 19 2018, 1:19 AM.

Details

Reviewers
poboiko
ngraham
Group Reviewers
Baloo
Frameworks
Commits
R293:5b24bd42e4b6: Make sure there is only one MTime per document in the MTimeDB
Summary

Previously, every time a file was was modified in place (e.g. appended),
the document was added for the new mtime, but never removed for the
old one.

Depends on: D16999

Diff Detail

Repository
R293 Baloo
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
bruns created this revision.Nov 19 2018, 1:19 AM
Restricted Application added projects: Frameworks, Baloo. · View Herald TranscriptNov 19 2018, 1:19 AM
Restricted Application added a subscriber: kde-frameworks-devel. · View Herald Transcript
bruns requested review of this revision.Nov 19 2018, 1:19 AM
ngraham accepted this revision.Nov 19 2018, 2:44 AM
This revision is now accepted and ready to land.Nov 19 2018, 2:44 AM
Closed by commit R293:5b24bd42e4b6: Make sure there is only one MTime per document in the MTimeDB (authored by bruns). · Explain WhyNov 20 2018, 12:00 AM
This revision was automatically updated to reflect the committed changes.
bruns mentioned this in R293:40ef654f7857: Add unittest for document replacement.

Revision Contents
Changeset List

PathPackages
Msrc/engine/writetransaction.cpp (9 lines)

Diff 45857

View Options

src/engine/writetransaction.cpp

Loading...