Differential D23697

buffer: Port away from foreach loops over members without calls to owner
ClosedPublic
Actions

Authored by kossebau on Sep 3 2019, 1:30 PM.

Details

Reviewers
cullmann
Group Reviewers
Kate
Commits
R39:b9167245dd3d: buffer: Port away from foreach loops over members without calls to owner
Summary

There is some small risk here:

  • overseen call chains which still call the owner and modify the container
  • other threads might access the same containers, even if class is not designed to be thread-safe, but wrong usages are currently caught mostly by the container copy

GIT_SILENT

Diff Detail

Repository
R39 KTextEditor
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
kossebau created this revision.Sep 3 2019, 1:30 PM
Restricted Application added projects: Kate, Frameworks. · View Herald TranscriptSep 3 2019, 1:30 PM
Restricted Application added subscribers: kde-frameworks-devel, kwrite-devel. · View Herald Transcript
kossebau requested review of this revision.Sep 3 2019, 1:30 PM
kossebau added a comment.Sep 3 2019, 1:32 PM

Also candidate for post 5.62-tagging, for some weeks of testing in master. Like all the following review requests. only the autotest one should be good to go in before, not critical to users.

kossebau edited the summary of this revision. (Show Details)Sep 3 2019, 1:32 PM
cullmann accepted this revision.Sep 7 2019, 7:38 AM
cullmann added a subscriber: cullmann.

Looks OK.

This revision is now accepted and ready to land.Sep 7 2019, 7:38 AM
Closed by commit R39:b9167245dd3d: buffer: Port away from foreach loops over members without calls to owner (authored by kossebau). · Explain WhySep 8 2019, 12:55 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Msrc/buffer/katetextfolding.cpp (14 lines)

Diff 65615

View Options

src/buffer/katetextfolding.cpp

Loading...