Differential D5841

Don't include the same file twice
ClosedPublic
Actions

Authored by ervin on May 13 2017, 4:55 PM.

Details

Reviewers
dfaure
bensi
franckarrecot
Group Reviewers
Zanshin
Commits
R4:33364cf6fe01: Don't include the same file twice
Summary

Found by Krazy.

Diff Detail

Repository
R4 Zanshin
Lint
Lint Skipped
Unit
Unit Tests Skipped
ervin created this revision.May 13 2017, 4:55 PM
dfaure edited edge metadata.May 13 2017, 5:00 PM

... and this is how one ends up spending his saturday clicking on web pages ;)

dfaure accepted this revision.May 13 2017, 5:00 PM
This revision is now accepted and ready to land.May 13 2017, 5:00 PM
ervin added a comment.May 13 2017, 5:10 PM
In D5841#109280, @dfaure wrote:

... and this is how one ends up spending his saturday clicking on web pages ;)

Yep, that's because I broke a toe yesterday (not a joke) and I get to sit more at my desk preparing Zanshin for kdereview...

dfaure added a comment.May 13 2017, 5:55 PM

Ouch, sorry about your toe ! :(

The million of zanshin users rejoice at the renewed activity, though :-)

bensi accepted this revision.May 16 2017, 7:19 AM
Closed by commit R4:33364cf6fe01: Don't include the same file twice (authored by ervin). · Explain WhyJun 2 2017, 12:03 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 14482

View Options

tests/units/presentation/contextpagemodeltest.cpp

Loading...