Differential D4300

Simpler calculation of longest line limit
ClosedPublic
Actions

Authored by gszymaszek on Jan 26 2017, 8:26 PM.

Details

Reviewers
mwolff
Group Reviewers
KTextEditor
Commits
R39:268af3d2b7ea: Use C++11 log2() instead of log() / log(2)
Summary

As suggested in TODO, I’ve replaced two log() calls with a single log2() call.

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.
gszymaszek updated this revision to Diff 10602.Jan 26 2017, 8:26 PM
gszymaszek retitled this revision from to Simpler calculation of longest line limit.
gszymaszek updated this object.
gszymaszek edited the test plan for this revision. (Show Details)
gszymaszek added a reviewer: KTextEditor.
gszymaszek set the repository for this revision to R39 KTextEditor.
gszymaszek added a project: KTextEditor.
Restricted Application added a project: Frameworks. · View Herald TranscriptJan 26 2017, 8:26 PM
Restricted Application added subscribers: Frameworks, kwrite-devel. · View Herald Transcript
mwolff accepted this revision.Jan 27 2017, 7:41 AM
mwolff added a reviewer: mwolff.
This revision is now accepted and ready to land.Jan 27 2017, 7:41 AM
Closed by commit R39:268af3d2b7ea: Use C++11 log2() instead of log() / log(2) (authored by dhaumann). · Explain WhyFeb 12 2017, 6:07 PM
This revision was automatically updated to reflect the committed changes.
dhaumann mentioned this in R39:268af3d2b7ea: Use C++11 log2() instead of log() / log(2).

Revision Contents
Changeset List

PathPackages
Msrc/document/katedocument.cpp (2 lines)

Diff 11255

View Options

src/document/katedocument.cpp

Loading...